All of lore.kernel.org
 help / color / mirror / Atom feed
From: Felix Zielcke <fzielcke@z-51.de>
To: The development of GRUB 2 <grub-devel@gnu.org>
Subject: Re: grub-install for FreeBSD
Date: Thu, 24 Sep 2009 20:12:06 +0200	[thread overview]
Message-ID: <1253815926.2845.64.camel@fz.local> (raw)
In-Reply-To: <4ABBB3EC.9050505@castlejones.net>

Am Donnerstag, den 24.09.2009, 11:01 -0700 schrieb Brian R. Jones:
> Robert Millan wrote:
> > On Wed, Sep 23, 2009 at 08:36:01AM +0200, Vladimir 'phcoder' Serbinenko wrote:
> >> Andrey Shuvikov wrote:
> >>> Hello,
> >>>
> >>> I tried to install Grub2 under FreeBSD but grub-install failed because readlink doesn't understand -f and stat doesn't have -c. I edited grub-mkconfig_lib.in to use different commands instead. I'm not sure it's the right way to handle this difference but it worked. The patch is attached.
> >>>
> >>>   
> >> We're aware of this problem. Felix Zielcke has a solution but at this
> >> point it's deemed too intrusive. We'll probably commit it once feature
> >> freeze is over
> > 
> > Alternatively, I could also allow a patch that checks the capabilities of
> > those commands.  It should try readlink first, and fallback to greadlink
> > if necessary.
> > 
> 
> 
> Not sure if Felix's patch covers it, but there is also a problem with the 
> 'sed' syntax in 10_freebsd and 30_os-prober.  It turns out that freebsd 
> 'sed' is quite retarded (at least in 7.2) and doesn't know that '\t' is 
> supposed to be tab.  Replacing 'sed' with 'gsed' or 'perl -p' fixes this issue.

No it doestn't.
Autoconf has an AC_PROG_SED macro which checks for a POSIX sed.
http://www.gnu.org/software/autoconf/manual/html_node/Limitations-of-Usual-Tools.html#sed
Portable sed regular expressions should use ‘\’ only to escape
characters in the string ‘$()*.0123456789[\^n{}’

If I understand that right, POSIX doestn't mandate \t so it wouldn't
help us.

> Where would I go to view Felix's path?  I am concerned because freebsd also 
> has a 'gstat' command that does not work either.  Only by installing the 
> coreutils and pointing explicitly at the coreutils version of gstat did I 
> get this to work.

http://lists.gnu.org/archive/html/grub-devel/2009-08/msg00749.html


-- 
Felix Zielcke
Proud Debian Maintainer




  reply	other threads:[~2009-09-24 18:12 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-09-23  2:01 grub-install for FreeBSD Andrey Shuvikov
2009-09-23  2:18 ` Seth Goldberg
2009-09-23  6:36 ` Vladimir 'phcoder' Serbinenko
2009-09-24 12:09   ` Robert Millan
2009-09-24 18:01     ` Brian R. Jones
2009-09-24 18:12       ` Felix Zielcke [this message]
2009-09-24 18:53       ` Colin Watson
2009-09-24 19:30         ` Brian R. Jones
2009-09-24 20:18       ` Robert Millan
2009-09-25  1:18         ` Brian R. Jones

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=1253815926.2845.64.camel@fz.local \
    --to=fzielcke@z-51.de \
    --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.