Bug 85128 - [patch] loader.conf(5) autoboot_delay incompletly described
Summary: [patch] loader.conf(5) autoboot_delay incompletly described
Status: In Progress
Alias: None
Product: Documentation
Classification: Unclassified
Component: Manual Pages (show other bugs)
Version: Latest
Hardware: Any Any
: Normal Affects Only Me
Assignee: freebsd-doc (Nobody)
URL: https://reviews.freebsd.org/D11887
Keywords: patch
Depends on:
Blocks:
 
Reported: 2005-08-19 18:20 UTC by Gary W. Swearingen
Modified: 2018-08-05 06:33 UTC (History)
2 users (show)

See Also:


Attachments
loader.diff (3.07 KB, patch)
2005-12-30 22:18 UTC, Daniel Gerzo
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Gary W. Swearingen 2005-08-19 18:20:09 UTC
The description of "autoboot_delay" variable in
/boot/defaults/loader.conf does not mention it's "NO" special
value.

A variable of the same name is described in loader(8), but it is by no
means obvious whether it's the same variable or just another set at the
loader prompt.

The variable is only the one in loader.conf that has more than a brief
comment, leading me to wonder if it shouldn't be described in
loader.conf(5) manpage instead, but the first paragraph of its
"Default Settings" section  makes me wonder.

Fix: 

Decide whether full description belongs in loader(8), loader.conf(5),
or /boot/defaults/loader.conf.  I don't know about loader(8), but I'm
thinking a full description should be in loader.conf(5) and referenced
from comment in "/boot/defaults/loader.conf".

I'll write patch after change is agreed on.
Comment 1 Giorgos Keramidas 2005-08-20 02:28:50 UTC
On 2005-08-19 10:17, "Gary W. Swearingen" <garys@opusnet.com> wrote:
> The description of "autoboot_delay" variable in
> /boot/defaults/loader.conf does not mention it's "NO" special
> value.
>
> A variable of the same name is described in loader(8), but it is by no
> means obvious whether it's the same variable or just another set at the
> loader prompt.
>
> The variable is only the one in loader.conf that has more than a brief
> comment, leading me to wonder if it shouldn't be described in
> loader.conf(5) manpage instead, but the first paragraph of its
> "Default Settings" section  makes me wonder.
>
> >Fix:
>
> Decide whether full description belongs in loader(8), loader.conf(5),
> or /boot/defaults/loader.conf.  I don't know about loader(8), but I'm
> thinking a full description should be in loader.conf(5) and referenced
> from comment in "/boot/defaults/loader.conf".

That seems reasonable.  Duplicating the description is not really useful
and having the loader options described in loader.conf(5) seems a good
thing.
Comment 2 Daniel Gerzo 2005-12-30 22:18:13 UTC
Hello,

   Included diff should fix your report, it also adds some more
   information to the autoboot_delay when set to -1.

   discussed with, and help provided by: ceri@

-- 
Cheers,
   Daniel Gerzo
Comment 3 Benedict Reuschling freebsd_committer 2017-08-05 22:08:25 UTC
I've opened a review for this issue here: https://reviews.freebsd.org/D11887