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
F102959230: D40329.diff
Tue, Nov 19, 5:19 AM
Unknown Object (File)
Mon, Nov 18, 2:18 AM
Unknown Object (File)
Sep 24 2024, 3:34 AM
Unknown Object (File)
Sep 13 2024, 6:18 AM
Unknown Object (File)
Sep 3 2024, 7:09 AM
Unknown Object (File)
Sep 3 2024, 1:58 AM
Unknown Object (File)
Aug 28 2024, 5:53 PM
Unknown Object (File)
Jul 31 2024, 8:49 AM
View All 76 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...