All of lore.kernel.org
 help / color / mirror / Atom feed
From: Yann E. MORIN <yann.morin.1998@free.fr>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 3/9] boot/syslinux: remove 'default y' in sub-options
Date: Fri, 25 Apr 2014 16:54:59 +0200	[thread overview]
Message-ID: <20140425145459.GA3273@free.fr> (raw)
In-Reply-To: <5359965A.2040101@mind.be>

Arnout, All,

On 2014-04-25 00:55 +0200, Arnout Vandecappelle spake thusly:
> On 25/04/14 00:29, Yann E. MORIN wrote:
> > From: "Yann E. MORIN" <yann.morin.1998@free.fr>
> > 
> > Remove the 'default y' from sub-options, but ensure at least one
> > is enabled anyway.
> > 
> > We keep the 'default y' to the default option, so that the menuconfig
> > behaviour is not too surprising.
> 
>  The "surprising" part is that isolinux will be unselected when one of
> the other options is selected. If you really want both isolinux and
> pxelinux, that indeed is surprising. But I think you usually want either,
> not both. So in that case the fact that you first have to select pxelinux
> and then unselect isolinux doesn't seem appropriate to me.

Yes, I was wondering what the expected behaviour would have been. I
would expect that something that is forcibly ON by default stays ON,
unless I explicitly disable it.

But the expected usage of syslinux is indeed that either is usually
used, not both (or more, given the following addition of the MBR, too).

So, I don't really care either way. Let's see what others think, I'll
rework the series if needed.

Regards,
Yann E. MORIN.

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'

  reply	other threads:[~2014-04-25 14:54 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-04-24 22:30 [Buildroot] [PATCH 0/9] Bump and improve syslinux (branch yem/adopted) Yann E. MORIN
2014-04-24 22:29 ` [Buildroot] [PATCH 1/9] boot/syslinux: needs an ia32-capable compiler Yann E. MORIN
2014-04-28  5:55   ` Arnout Vandecappelle
2014-04-28 17:51     ` Yann E. MORIN
2014-04-24 22:29 ` [Buildroot] [PATCH 2/9] boot/syslinux: prepare to install non-core images Yann E. MORIN
2014-04-24 22:29 ` [Buildroot] [PATCH 3/9] boot/syslinux: remove 'default y' in sub-options Yann E. MORIN
2014-04-24 22:55   ` Arnout Vandecappelle
2014-04-25 14:54     ` Yann E. MORIN [this message]
2014-04-24 22:29 ` [Buildroot] [PATCH 4/9] boot/syslinux: rewrite options prompts Yann E. MORIN
2014-04-28  5:57   ` Arnout Vandecappelle
2014-04-28 17:36     ` Yann E. MORIN
2014-04-28 19:17       ` Arnout Vandecappelle
2014-04-24 22:30 ` [Buildroot] [PATCH 5/9] package/ucl: new package Yann E. MORIN
2014-04-24 22:30 ` [Buildroot] [PATCH 6/9] package/upx: " Yann E. MORIN
2014-04-28  6:03   ` Arnout Vandecappelle
2014-04-28 17:43     ` Yann E. MORIN
2014-04-24 22:30 ` [Buildroot] [PATCH 7/9] boot/syslinux: bump version Yann E. MORIN
2014-04-28  6:14   ` Arnout Vandecappelle
2014-04-28 17:26     ` Yann E. MORIN
2014-04-24 22:30 ` [Buildroot] [PATCH 8/9] boot/syslinux: add option to install the EFI image Yann E. MORIN
2014-04-28  6:18   ` Arnout Vandecappelle
2014-04-28 17:15     ` Yann E. MORIN
2014-04-28 19:16       ` Arnout Vandecappelle
2014-04-28 19:45         ` Yann E. MORIN
2014-04-28 19:51           ` Arnout Vandecappelle
2014-04-24 22:30 ` [Buildroot] [PATCH 9/9] boot/syslinux: add an option to install an MBR blob Yann E. MORIN
2014-04-28 16:23   ` Arnout Vandecappelle
2014-04-28 19:52     ` Yann E. MORIN
2014-04-28 19:58       ` Frank Hunleth
2014-04-28 20:39         ` Yann E. MORIN
2014-04-28 20:45           ` Frank Hunleth
2014-04-28 20:57       ` Arnout Vandecappelle

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20140425145459.GA3273@free.fr \
    --to=yann.morin.1998@free.fr \
    --cc=buildroot@busybox.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.