[Fedora-livecd-list] [PATCH 6/6] Several Kernels

[Jeroen van Meeuwen]

Jeremy Katz wrote:
> On Sun, 2007-08-05 at 20:40 +0200, Jeroen van Meeuwen wrote:
>> A patch which allows multiple kernels to be installed on the live media,
>> giving boot options in the syslinux menu for all of them.
> 
> This part looks mostly reasonable...
> 
>> In addition, this patch adds an optional timeout parameter for the boot
>> menu. It also adds a cli parameter for appending boot options.
> 
> I think a better approach for this, though, is to extend the kickstart
> bootloader syntax a little bit.  The proliferation of command-line
> options for this is a little ridiculous.  Also, it leads to the images
> not being reproducible just from the kickstart config
> 

It's never reproducible from kickstart config as kickstart is not
allowing you to set specific version numbers, nor filesystem
compression, prelinking, ignoring deleted files, etc.

I'd rather have some error, bug or dis-functionality that I can fix. I
can't read your mind and think of the better approach without seeing the
code live in some upstream source repository. If it's reasonable, like
the previous patch, you get to apply it and _then_ think of a better
way, then apply that. Just a cloud of better ideas doesn't make it rain.

Kind regards,

Jeroen van Meeuwen
-kanarip