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
F161158067: D40329.diff
Wed, Jul 1, 2:20 AM
Unknown Object (File)
Mon, Jun 29, 2:52 AM
Unknown Object (File)
Fri, Jun 26, 9:43 PM
Unknown Object (File)
Thu, Jun 25, 6:52 AM
Unknown Object (File)
Mon, Jun 22, 11:57 PM
Unknown Object (File)
Mon, Jun 22, 2:17 AM
Unknown Object (File)
Sat, Jun 20, 8:57 AM
Unknown Object (File)
Fri, Jun 19, 6:48 PM
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...