[Top][All Lists]
[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