Page MenuHomeFreeBSD

Add new commands to rc(8): describe and extracommands

Authored by lme on Jul 19 2014, 3:57 PM.



The patch adds to new commands to rc(8). This is what can be done with it:

$ service sshd describe
Secure Shell Daemon

$ service sshd extracommands
configtest keygen reload

It also adds a "descr" variable to sshd's rc script and documents the additional rc options. See how to print them using service(8).

Diff Detail

rS FreeBSD src repository - subversion
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

lme retitled this revision from to Add new commands to rc(8): describe and extracommands.
lme updated this object.
lme edited the test plan for this revision. (Show Details)
lme added a reviewer: hrs.

Can you also update share/man/man8/rc.8? The "Each script is expected to support at least the following arguments" section.

crees added a reviewer: crees.

Really like this idea, think it was discussed a while ago?

This revision is now accepted and ready to land.Apr 23 2016, 11:48 AM
lme edited edge metadata.

Update the rc(8) manpage and add descriptions to most rc scripts in /etc/rc.d/

This revision now requires review to proceed.Apr 23 2016, 3:05 PM
bapt added a reviewer: bapt.
bapt added a subscriber: bapt.

Please commit in 2 separates commits: 1 the tools, 2 adding the descriptions

This revision is now accepted and ready to land.Apr 23 2016, 3:11 PM
brueffer added a reviewer: brueffer.
brueffer added a subscriber: brueffer.
brueffer added inline comments.
13 ↗(On Diff #15533)

Consistency, filesystems or file systems? Not sure which we prefer these days, but pick just one. After going through the whole patch, "filesystems" seems to be more common.

15 ↗(On Diff #15533)


24 ↗(On Diff #15533)

"the network being up"?

13 ↗(On Diff #15533)

Sounds a bit strange, "and" instead of "plus" maybe?

22 ↗(On Diff #15533)

"name service" here, but "name-service" below. Pick one.

16 ↗(On Diff #15533)

lowercase "check"?

15 ↗(On Diff #15533)


14 ↗(On Diff #15533)

Does the script work for wired interfaces as well?

This revision now requires changes to proceed.Apr 23 2016, 3:22 PM
lme edited edge metadata.
  • Integrate most of brueffer's suggestions. While here, change the remaining "file system" to "filesystem". Most of the description comes from the daemon's manpage which are not very consistent.
  • Always allow to run "describe". Having to use "onedescribe" feels just wrong.
  • Add the now keywords "describe", "extracommands" and "oneextracommands" to the tcsh completion examples
brueffer edited edge metadata.
This revision is now accepted and ready to land.Apr 23 2016, 4:00 PM