All of lore.kernel.org
 help / color / mirror / Atom feed
From: Pavel Roskin <proski@gnu.org>
To: The development of GRUB 2 <grub-devel@gnu.org>
Subject: Re: [PATCH] Support FreeBSD in osdetect.lua
Date: Tue, 21 Jul 2009 17:25:31 -0400	[thread overview]
Message-ID: <1248211531.18091.5.camel@mj> (raw)
In-Reply-To: <d7ead6de0907211317h44838027g2bbf40cb5dc227a6@mail.gmail.com>

On Tue, 2009-07-21 at 22:17 +0200, Vladimir 'phcoder' Serbinenko wrote:

> Actually this menu is done to match the default one of FreeBSD loader.
> I'm not a FreeBSD expert so I prefered to stick to the same menu as
> default bootloader.

I see.

> > The implementation could remove the original menu, or it could insert
> > entries immediately below "safe mode".  I don't know if it's possible,
> > but it could be useful.
> >
> Bean proposed something like this but I don't know his progress
> > Actually, I don't want osdetect.lua to be overengineered and overloaded
> > with intimate knowledge of different OSes.  It's an example for others
> > to extend, not a complete solution.
> We put it into our repository this means that we accept improvements.
> I also think that there will be distributions similar to Super GRUB
> Disk and other recovery tools based on osdetect.lua. Unless we provide
> a good reference osdetect.lua there will be a bitrot between such
> distributions every one neglecting some OSes

I don't fee strongly against your change.  However, I would hate to see
five lines for every Linux kernel (although it would be nice to have
safe mode for Windows in the GRUB menu).  So this should not be an
example for other OSes.

Obviously, the zfs part shouldn't go in yet.

-- 
Regards,
Pavel Roskin



  reply	other threads:[~2009-07-21 21:25 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-07-21 13:45 [PATCH] Support FreeBSD in osdetect.lua Vladimir 'phcoder' Serbinenko
2009-07-21 16:02 ` Pavel Roskin
2009-07-21 20:17   ` Vladimir 'phcoder' Serbinenko
2009-07-21 21:25     ` Pavel Roskin [this message]
2009-07-21 21:50       ` Vladimir 'phcoder' Serbinenko
2009-07-22 17:51         ` Robert Millan
2009-07-22 18:02           ` Vladimir 'phcoder' Serbinenko
2009-07-25 16:10             ` Robert Millan
2009-07-25 16:46               ` Vladimir 'phcoder' Serbinenko

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=1248211531.18091.5.camel@mj \
    --to=proski@gnu.org \
    --cc=grub-devel@gnu.org \
    /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.