Page MenuHomeFreeBSD
Differential D40329

rc(8): document how to reload rc's configuration
ClosedPublic
Actions

Authored by freebsd_igalic.co on May 30 2023, 7:40 AM.
Tags
Referenced Files
F148558281: D40329.diff
Wed, Mar 18, 5:59 PM
F148498575: D40329.id122598.diff
Wed, Mar 18, 6:48 AM
Unknown Object (File)
Tue, Mar 17, 7:02 AM
Unknown Object (File)
Sun, Mar 15, 2:35 PM
Unknown Object (File)
Sun, Mar 15, 9:04 AM
Unknown Object (File)
Sat, Mar 14, 9:20 PM
Unknown Object (File)
Wed, Mar 11, 12:31 PM
Unknown Object (File)
Wed, Mar 11, 10:27 AM
View All Files
Subscribers
debdrup
grahamperrin
imp
manpages

Details

Reviewers
grahamperrin
imp
Group Reviewers
manpages
Commits
rG0c2cdf496834: rc(8): document how to reload rc's configuration
rGfedfe274225a: rc(8): document how to reload rc's configuration
rG0981275c75d5: rc(8): document how to reload rc's configuration
Summary

The fact that rc(8) only reads its configuration once is in the man
page, but how to trigger a reload is only in the code.

Discovered while trying to make cloud-init disable and stop syslogd.

Thanks to RhodiumToad for providing the words.

Sponsored by: The FreeBSD Foundation

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

freebsd_igalic.co created this revision.May 30 2023, 7:40 AM
Herald added a subscriber: imp. · View Herald TranscriptMay 30 2023, 7:40 AM
freebsd_igalic.co requested review of this revision.May 30 2023, 7:40 AM
Harbormaster completed remote builds in B51769: Diff 122598.May 30 2023, 7:40 AM
freebsd_igalic.co added a reviewer: manpages.May 30 2023, 7:41 AM
freebsd_igalic.co added projects: rc, manpages.
freebsd_igalic.co added a subscriber: manpages.
debdrup added a subscriber: debdrup.May 30 2023, 9:28 AM

I only spotted a minor nit.

share/man/man8/rc.8
127

The parenthetical should be before the period.

freebsd_igalic.co updated this revision to Diff 122628.May 30 2023, 9:11 PM

address @debdrup's review

Harbormaster completed remote builds in B51785: Diff 122628.May 30 2023, 9:11 PM
freebsd_igalic.co updated this revision to Diff 122629.May 30 2023, 9:13 PM
freebsd_igalic.co marked an inline comment as done.

arc swallowed my previous commit

Harbormaster completed remote builds in B51786: Diff 122629.May 30 2023, 9:13 PM
grahamperrin requested changes to this revision.May 30 2023, 10:47 PM
grahamperrin added a subscriber: grahamperrin.
grahamperrin added inline comments.
share/man/man8/rc.8
127
128–129
This revision now requires changes to proceed.May 30 2023, 10:47 PM
freebsd_igalic.co updated this revision to Diff 122728.Jun 1 2023, 7:36 PM

address @grahamperrin's review

Harbormaster completed remote builds in B51844: Diff 122728.Jun 1 2023, 7:36 PM
freebsd_igalic.co marked 2 inline comments as done.Jun 1 2023, 7:36 PM
imp accepted this revision.Jun 1 2023, 8:15 PM
grahamperrin accepted this revision.Jun 1 2023, 11:48 PM

I don't know the technology for this change, but the wording is good now.

One further change suggested, not required. Thanks

share/man/man8/rc.8
378
This revision is now accepted and ready to land.Jun 1 2023, 11:48 PM
Closed by commit rG0981275c75d5: rc(8): document how to reload rc's configuration (authored by freebsd_igalic.co, committed by grahamperrin). · Explain WhyJun 2 2023, 6:30 PM
This revision was automatically updated to reflect the committed changes.
grahamperrin added a commit: rG0981275c75d5: rc(8): document how to reload rc's configuration.
grahamperrin added a commit: rGfedfe274225a: rc(8): document how to reload rc's configuration.Jun 10 2023, 6:26 PM
grahamperrin added a commit: rG0c2cdf496834: rc(8): document how to reload rc's configuration.Jun 10 2023, 6:33 PM

Revision Contents
Changeset List

PathSize
share/
man/
man8/
18 lines

Diff 122767

View Options

share/man/man8/rc.8

Loading...