[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: four patches
From: |
Ralf Corsepius |
Subject: |
Re: four patches |
Date: |
Sun, 17 Jul 2005 08:40:18 +0200 |
On Fri, 2005-07-15 at 12:20 +0200, Stepan Kasal wrote:
> Hello Ralf,
>
> On Fri, Jul 15, 2005 at 08:04:56AM +0200, Ralf Corsepius wrote:
> > On Thu, 2005-07-07 at 13:30 +0200, Stepan Kasal wrote:
> > > autoconf-20050629-canon.patch
> >
> > What is the rationale for replacing the
> > 'test -z' with 'test "x$pat" = x' ?
> >
> > IMO, you are trying to fix non-bugs, here, because even info autoconf
> > states:
> > ...
> > Contrary to a common belief, `test -n STRING' and `test -z STRING'
> > *are* portable.
> > ...
>
> and that very paragraph explains that problem might arise if the string
> looks like an operator.
>
> In my patch, the variables were build_alias, host_alias, target_alias.
> All of these three are given by the user on the command line, so they can
> possibly contain any string.
And you are sure '"x=" = x' evaluates correctly on these systems?
I am having strong doubts.
Ralf