[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [NEW PATCH] Add option to grub-probe to accept system devices as arg
From: |
Robert Millan |
Subject: |
Re: [NEW PATCH] Add option to grub-probe to accept system devices as arguments |
Date: |
Mon, 3 Mar 2008 22:06:56 +0100 |
User-agent: |
Mutt/1.5.13 (2006-08-11) |
On Fri, Feb 29, 2008 at 09:33:13AM +0100, Fabian Greffrath wrote:
> >For this one, I think it'd be much better to rework 10_hurd so that code is
> >shared, rather than duplicating the whole blurb.
>
> Yes, we could source out the complete linux/hurd kernel-adding blurbs
> into an external shell library that is shared with 10_* and 30_os-prober.
>
> However, please don't forget these are two different things. 10_* look
> on / and /boot, while os-prober looks on all *other* partitions.
For 10_linux it isn't that important since the boot parameters are so small,
it's 10_hurd that has more reusable stuff (that big blurb).
> >Do you plan to do that later?
>
> Not a concrete plan yet, However, we should first get thinks *working*
> before we care about cosmetics, IMHO. ;)
In that case maybe it's better to leave Hurd support for later, than
duplicating the whole thing.
> >Maybe it's better to echo something here, otherwise it gives the impression
> >OS FooBar was just detected and properly added.
>
> Definitely. I'll add a short warning message.
>
> Furthermore, I am going to modify the script to immediately 'exit 0'
> if it turns out it misses the tools required to update the boot menu
> (i.e. os-prober, linux-boot-prober, grub-probe (>= 20080228)). These
> tests are spread all over the script at the moment.
Ok.
> PS: Shouldn't 30_os-prober be an upstream feature of os-prober?!
Not sure about that. My initial intention was that in general one could plug
in scripts from other system components, but often the code in them depends on
many update-grub interfaces which makes it likely to break if something is
changed.
For now only very simple add-ons exist (memtest86, invaders...). But this
one will probably end up being more intermangled with other GRUB parts (see
that we're already discussing how to reduce code duplication with 10_hurd).
I'd suggest putting it in GRUB, at least for now, if that is fine with you.
--
Robert Millan
<GPLv2> I know my rights; I want my phone call!
<DRM> What use is a phone call… if you are unable to speak?
(as seen on /.)
- Re: [NEW PATCH] Add option to grub-probe to accept system devices as arguments, Robert Millan, 2008/03/03
- Re: [NEW PATCH] Add option to grub-probe to accept system devices as arguments,
Robert Millan <=
- Re: [NEW PATCH] Add option to grub-probe to accept system devices as arguments, Fabian Greffrath, 2008/03/04
- [NEW PATCH] os-prober output parser, Fabian Greffrath, 2008/03/05
- Re: [NEW PATCH] os-prober output parser, Fabian Greffrath, 2008/03/13
- Re: [NEW PATCH] os-prober output parser, Robert Millan, 2008/03/19
- Re: [NEW PATCH] os-prober output parser, Fabian Greffrath, 2008/03/19
- Re: [NEW PATCH] os-prober output parser, Robert Millan, 2008/03/19
- Re: [NEW PATCH] os-prober output parser, Fabian Greffrath, 2008/03/20
- Re: [NEW PATCH] os-prober output parser, Robert Millan, 2008/03/20
- Re: [NEW PATCH] os-prober output parser, Fabian Greffrath, 2008/03/25