I've shortened the patch somewhat and adjusted the log entry somewhat,
hope you don't mind.  I've queued this for maint.

(I would have queued the mode patches for maint as well, but they caused
a conflict there, and I didn't figure porting was that important.)

Thanks again,
Ralf

commit cce60a543d55a27b4a0424c580b58a4fd53ec1be
Author: Stefano Lattarini <stefano.lattar...@gmail.com>
Date:   Fri Dec 4 15:47:23 2009 +0100

    Avoid Solaris sh `set -e' bug in vala4.test.
    
    * tests/vala4.test: New variable $cwd, holding the full path of the
    current directory.  Use it instead of `pwd` command substitution,
    to avoid Heirloom/Solaris Sh bug with `set -e'.
    
    Signed-off-by: Ralf Wildenhues <ralf.wildenh...@gmx.de>

diff --git a/ChangeLog b/ChangeLog
index ca526c8..c970922 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,11 @@
+2009-12-04  Stefano Lattarini  <stefano.lattar...@gmail.com>
+           Ralf Wildenhues  <ralf.wildenh...@gmx.de>
+
+       Avoid Solaris sh `set -e' bug in vala4.test.
+       * tests/vala4.test: New variable $cwd, holding the full path of the
+       current directory.  Use it instead of `pwd` command substitution,
+       to avoid Heirloom/Solaris Sh bug with `set -e'.
+
 2009-11-28  Jim Meyering  <meyer...@redhat.com>
 
        do not put world-writable directories in distribution tarballs
diff --git a/tests/vala4.test b/tests/vala4.test
index 4301d45..7a15ebe 100755
--- a/tests/vala4.test
+++ b/tests/vala4.test
@@ -43,19 +43,24 @@ exit 0
 END
 chmod +x valac
 
+cwd=`pwd`
+
+# Use $cwd instead of `pwd` in the && list below to avoid a bug in
+# the way Solaris/Heirloom Sh handles `set -e'.
+
 libtoolize
 $ACLOCAL
 $AUTOMAKE -a
 $AUTOCONF
-./configure "VALAC=`pwd`/valac"
+./configure "VALAC=$cwd/valac"
 
 sed 's/AM_PROG_VALAC.*/AM_PROG_VALAC([9999.9])/' < configure.in >t
 mv -f t configure.in
 $AUTOCONF --force
-./configure "VALAC=`pwd`/valac" && Exit 1
+./configure "VALAC=$cwd/valac" && Exit 1
 
 sed 's/AM_PROG_VALAC.*/AM_PROG_VALAC([1.2.3])/' < configure.in >t
 mv -f t configure.in
 $AUTOCONF --force
-./configure "VALAC=`pwd`/valac"
+./configure "VALAC=$cwd/valac"
 :


Reply via email to