grub-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] Fix target tool check logic


From: Pavel Roskin
Subject: Re: [PATCH] Fix target tool check logic
Date: Tue, 14 Apr 2009 20:31:35 -0400

On Wed, 2009-04-15 at 00:45 +0900, Yoshinori K. Okuji wrote:
> On Saturday 11 April 2009 22:16:58 Pavel Roskin wrote:
> > Quoting "Yoshinori K. Okuji" <address@hidden>:
> > > "test -n" should be avoided. Maybe this is not necessary nowadays, but my
> > > old lesson was to use "test x$target_alias != x" instead for portability.
> > > Well, "!=" was not very portable, either, maybe.
> >
> > I believe both "-n" and "!=" are found in Autoconf sources that are
> > turned into  configure scripts.  Anyway, I'll use the syntax you want.
> 
> Even if this looks obsolete, I think it is better to follow the 
> chapter "Limitations of Builtins" in the autoconf manual:

Thanks.  The Autoconf code I was referring to didn't involve any
possibility of pathological arguments.  But we are dealing with user
input here (target_alias comes from the command line), so you are right,
it's better to err on the safe side.

-- 
Regards,
Pavel Roskin




reply via email to

[Prev in Thread] Current Thread [Next in Thread]