[Buildroot] [PATCH 12/12] fs/iso9660: add support for grub2

Yann E. MORIN yann.morin.1998 at free.fr
Sat Jun 6 09:18:46 UTC 2015


Thomas, All,

On 2015-06-06 02:59 +0200, Thomas Petazzoni spake thusly:
> Dear Yann E. MORIN,
> 
> On Sat, 6 Jun 2015 01:07:56 +0200, Yann E. MORIN wrote:
> 
> > > -comment "iso image needs a Linux kernel and grub or isolinux to be built"
> > > +comment "iso image needs a Linux kernel and grub(2) or isolinux to be built"
> > 
> > I ghave to admit that reading this comment using the usual associativity
> > of 'and' and 'or' makes it sound that we either want
> >   - a kernel and grub/grub2,
> >   - or isolinux
> 
> True. Any suggestion to improve that?
> 
> comment "iso image needs (a Linux kernel && (grub(2) || isolinux))"
> 
> is probably a bit hard to read, no ? :-)

    iso image needs a Linux kernel and one of grub, grub2 or isolinux

> > > +menuentry "Buildroot" {
> > > +	linux __KERNEL_PATH__ root=/dev/sr0 console=tty1
> > 
> > Why do you forcibly pass the console setting?
> 
> That's a good question. We don't have that for the grub and isolinux
> case, so I'm not sure why I added that for the grub2 case. I'll kill
> that and retest. Thanks for noticing!

He! :-)

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.  |
'------------------------------^-------^------------------^--------------------'


More information about the buildroot mailing list