Re: CONF_SUBDIRS in Makefile.am; test suite & DJGPP [PATCH]

[Robert Boehne]

Right you are Albert, I'll go ahead and check this in
as obvious.

Thanks,

Robert

Albert Chin wrote:
> 
> On Fri, Mar 28, 2003 at 11:04:54PM -0600, Robert Boehne wrote:
> > Approved and checking in.
> 
> In tests/defs, you need to quote $DJGPP. If not, "test -z $DJGPP" when
> DJGPP is not set is equivalent to "test -z" on some systems (IRIX 6.5
> in particular).
> 
> --
> albert chin (address@hidden)

Index: ChangeLog
===================================================================
RCS file: /cvsroot/libtool/libtool/ChangeLog,v
retrieving revision 1.1217
diff -u -r1.1217 ChangeLog
--- ChangeLog   2 Apr 2003 16:44:37 -0000       1.1217
+++ ChangeLog   3 Apr 2003 05:43:13 -0000
@@ -1,3 +1,8 @@
+2003-04-02  Robert Boehne  <address@hidden>
+
+       * tests/defs: Quote $DJGPP to prevent problems with some shells.
+       Pointed out by Albert Chin <address@hidden>
+
 2003-04-01  Peter O'Gorman  <address@hidden>
 
        * libtool.m4: Fix the echo test which I broke earlier.
Index: tests/defs
===================================================================
RCS file: /cvsroot/libtool/libtool/tests/defs,v
retrieving revision 1.4
diff -u -r1.4 defs
--- tests/defs  29 Mar 2003 05:06:46 -0000      1.4
+++ tests/defs  3 Apr 2003 05:43:13 -0000
@@ -40,7 +40,7 @@
 
 # Disable usage of config.site for autoconf, unless DJGPP is present.
 # The DJGPP port of autoconf requires config.site, to work correctly.
-if test -z $DJGPP; then
+if test -z "$DJGPP"; then
     CONFIG_SITE=/nonexistent
 fi