[Top][All Lists]

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

Re: CVS, bootstrapping and DJGPP

From: Alexandre Duret-Lutz
Subject: Re: CVS, bootstrapping and DJGPP
Date: Mon, 10 Feb 2003 15:10:03 +0100
User-agent: Gnus/5.090008 (Oort Gnus v0.08) Emacs/21.2 (i386-pc-linux-gnu)

>>> "Richard" == Richard Dawe <address@hidden> writes:


 Richard> I found the problem. It was due to $SHELL (in the
 Richard> environment) not being set to bash.
 >> Is this something we should set in Automake?
 >> For instance
 >> $ENV{'SHELL'} = '@SHELL@'
 >> where @SHELL@ is the shell computed by ./configure

 Richard> Yes, but I don't know whether you should do this for
 Richard> all platforms or just DJGPP (by checking for
 Richard> defined($ENV{'DJGPP'}) before defining $ENV{'SHELL'}
 Richard> to be @SHELL@).

Thanks.  I'm installing the following change on HEAD and branch-1-7.

2003-02-10  Alexandre Duret-Lutz  <address@hidden>

        * (BEGIN): Override $ENV{'SHELL'} with ./configure's
        SHELL for DJGPP.
        Suggested by Richard Dawe.

RCS file: /cvs/automake/automake/,v
retrieving revision 1.1365.2.30
diff -u -r1.1365.2.30
--- 6 Feb 2003 16:52:51 -0000       1.1365.2.30
+++ 10 Feb 2003 14:02:35 -0000
@@ -33,6 +33,15 @@
   my $perllibdir = $ENV{'perllibdir'} || 
   unshift @INC, $perllibdir;
+  # Override SHELL.  This is required on DJGPP so that system() uses
+  # bash, not COMMAND.COM which doesn't quote arguments properly.
+  # Other systems aren't expected to use $SHELL when Automake
+  # runs, but it should be safe to drop the `if DJGPP' gard if
+  # it turns up other systems need the same thing.  After all,
+  # if SHELL is used, ./configure's SHELL is always better than
+  # the user's SHELL (which may be something like tcsh).
+  $ENV{'SHELL'} = '@SHELL@' if exists $ENV{'DJGPP'};
 use Automake::Struct;

Alexandre Duret-Lutz

reply via email to

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