[syslinux] patch, doc/{syslinux, menu}.txt: Clarifying the DEFAULT directive
Gene Cumm
gene.cumm at gmail.com
Mon Nov 28 13:07:54 PST 2011
On Nov 28, 2011 2:12 PM, "Regid Ichira" <regid23 at yahoo.com> wrote:
>
> --- On Mon, 11/28/11, Jeffrey Hutzelman wrote:
> --- a/doc/menu.txt 2011-04-19 00:24:17.000000000 +0300
> +++ b/doc/menu.txt 2011-11-27 17:36:02.000000000 +0200
> @@ -307,8 +307,8 @@ DEFAULT label
> START.
>
> For backwards compatibility with earlier versions of Syslinux,
> - this directive is ignored unless the configuration file also
> - contains a UI directive.
> + this directive is ignored only by the MENU system, unless the
> + configuration file also contains a UI directive.
This explicit specification is redundant based on doc/menu.txt implying
differences that apply to the simple menu (vesa)menu.c32 modules. I feel
it's unnecessary.
> --- a/doc/syslinux.txt 2011-04-19 00:24:17.000000000 +0300
> +++ b/doc/syslinux.txt 2011-11-27 17:10:54.000000000 +0200
> + in at the "boot:" prompt. A (CLI) label can be stated instead
> + of kernel options... . The LABEL directive is described below.
The "(CLI)" part is unnecessary. The simple menu system requires these
labels. I think the following is clearer:
'A LABEL may be used instead of "kernel options...".'
Or perhaps: 'A LABEL should be used instead of "kernel options...".'
> + doc/menu.txt also describes potential aspects of the DEFAULT
> + directive.
Unnecessary and out of context. I feel a more generic reference to
doc/menu.txt or no reference at all would be more appropriate. If you are
configuring for the simple menu system, doc/syslinux.txt comes first with
doc/menu.txt supplementing for simple menu system specifics.
--
-Gene
More information about the Syslinux
mailing list