bug-autoconf
[Top][All Lists]
Advanced

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

Re: CVS autoconf testsuite failure on Tru64 unix


From: Paul Eggert
Subject: Re: CVS autoconf testsuite failure on Tru64 unix
Date: Sun, 12 Dec 2004 01:54:38 -0800
User-agent: Gnus/5.1006 (Gnus v5.10.6) Emacs/21.3 (gnu/linux)

Noah Misch <address@hidden> writes:

> -        *[[^\\]]"$at_lf"*) at_reason='an embedded newline' ;;
> +        *\\"$at_lf"*)      at_reason= ;;
> +        *"$at_lf"*)        at_reason='an embedded newline' ;;

I installed your patch, as it's an improvement, but this part of it
still looks wrong to me.  For example, the following shell command
contains an embedded newline, but the code wouldn't catch it:

  foo="bar\\
  baz"

Wouldn't it be safer to report all newlines, even those preceded by
backslash?

(The other Tru64 issues in this area still remain, right?)




reply via email to

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