autoconf-patches
[Top][All Lists]
Advanced

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

Re: [PATCH] tests: don't silently skip the sort-version tests


From: Eric Blake
Subject: Re: [PATCH] tests: don't silently skip the sort-version tests
Date: Thu, 14 Jan 2010 06:58:38 -0700
User-agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1.23) Gecko/20090812 Thunderbird/2.0.0.23 Mnenhy/0.7.6.666

According to Eric Blake on 1/13/2010 6:10 AM:
>> by contrast, bash warns but still exits successfully:
>>   $ printf 'cat<<-x\n foo\n x\n'|bash && echo you lose
>>   bash: line 3: warning: here-document at line 1 delimited by \
>>     end-of-file (wanted `x')
>>    foo
>>    x
>>   you lose
> 
> We should mention this in the autoconf manual, so I'm pushing this.  I'm
> not (yet) sure whether the warning about unterminated here-doc was
> introduced in bash 4.0 or bash 4.1, but I can also confirm that bash 3.2
> is silent whereas bash 4.1 warns (but the warning is longer than 80
> columns, so it is not a nice fit in the manual).

I pushed too soon; this revision makes it more apparent that bash had a
zero exit status after an unterminated here-doc:

-- 
Don't work too hard, make some time for fun as well!

Eric Blake             address@hidden
From 56f5b9bc6a3a4a4dccea38005e211066d1ba96cd Mon Sep 17 00:00:00 2001
From: Eric Blake <address@hidden>
Date: Wed, 13 Jan 2010 09:46:35 -0700
Subject: [PATCH] Fix previous example.

* doc/autoconf.texi (Here-Documents): Touch up the example to
match output to sample command line.

Signed-off-by: Eric Blake <address@hidden>
---
 ChangeLog         |    4 ++++
 doc/autoconf.texi |    6 ++++--
 2 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index 3fa0c5d..1719deb 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,9 @@
 2010-01-13  Eric Blake  <address@hidden>

+       Fix previous example.
+       * doc/autoconf.texi (Here-Documents): Touch up the example to
+       match output to sample command line.
+
        Document here-doc pitfall.
        * doc/autoconf.texi (Here-Documents): Mention problem with <<-
        operator.
diff --git a/doc/autoconf.texi b/doc/autoconf.texi
index 31550fd..1549f89 100644
--- a/doc/autoconf.texi
+++ b/doc/autoconf.texi
@@ -14698,13 +14698,15 @@ Here-Documents
 here-documents).

 @example
-$ @kbd{printf 'cat <<-x\n\t1\n\t 2\n\tx\n' | bash && echo oops}
+$ @kbd{printf 'cat <<-x\n\t1\n\t 2\n\tx\n' | bash && echo done}
 1
  2
-$ @kbd{printf 'cat <<-x\n 1\n  2\n x\n' | bash-3.2 && echo oops}
+done
+$ @kbd{printf 'cat <<-x\n 1\n  2\n x\n' | bash-3.2 && echo done}
  1
   2
  x
+done
 @end example

 @node File Descriptors
-- 
1.6.4.2

Attachment: signature.asc
Description: OpenPGP digital signature


reply via email to

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