loader: update autoboot description and move to loader.conf.5
ClosedPublic
Actions

Authored by imp on Aug 5 2017, 10:01 PM.

Details

Reviewers

wblock
bcr
danger

Group Reviewers

manpages

Commits

rGf32c4c3b00c4: loader: update autoboot description and move to loader.conf.5
rG71f6aea4150c: loader: update autoboot description and move to loader.conf.5

Summary

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=85128

In essence, the description of the "autoboot_delay" variable in /boot/defaults/loader.conf does not mention it's "NO" special value.

Test Plan

1) Apply the two patches

Check the output from both man pages

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

bcr created this revision.Aug 5 2017, 10:01 PM

Herald added a subscriber: imp. · View Herald TranscriptAug 5 2017, 10:01 PM

bcr edited the summary of this revision. (Show Details)Aug 5 2017, 10:01 PM

wblock added inline comments.Dec 8 2017, 5:42 PM

sys/boot/forth/loader.conf.5
99 ↗	(On Diff #31629)	s/will be/is/
103 ↗	(On Diff #31629)	s/will be/are/
107 ↗	(On Diff #31629)	no delay is inserted, but a user with console access will be able
110 ↗	(On Diff #31629)	process and escape into the interactive mode by pressing a key on
111 ↗	(On Diff #31629)	s/while/while the/ s/are being/are/
117 ↗	(On Diff #31629)	s/will engage/starts/
119 ↗	(On Diff #31629)	s/for some reason././

Benedict, what are your plans for this? Mind if I take it over?

@imp: No more work done since then. If you want to take over, I'd appreciate it.
You can keep me in the list of reviewers for any textual changes you're making. Thanks!

imp retitled this revision from Bug 85128: loader.conf(5) autoboot_delay incompletly described to loader: update autoboot description and move to loader.conf.5.Jul 9 2021, 6:55 AM

imp added reviewers: bcr, danger.

imp commandeered this revision.Jul 9 2021, 7:02 AM

rework based on files moved
My own take at making things a little clearer

Herald added a subscriber: dab. · View Herald TranscriptJul 9 2021, 7:03 AM

Harbormaster completed remote builds in B40406: Diff 91993.Jul 9 2021, 7:03 AM

Minor question about which keys are valid to press.