This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Configure patch
- To: nathan at cs dot bris dot ac dot uk
- Subject: Re: Configure patch
- From: Jeffrey A Law <law at cygnus dot com>
- Date: Thu, 26 Aug 1999 23:48:24 -0600
- cc: egcs-patches at egcs dot cygnus dot com
- Reply-To: law at cygnus dot com
In message <37C55ECE.2985F2D@acm.org>you write:
> This is a multi-part message in MIME format.
> --------------254FCAB97C86E51014C2CA25
> Content-Type: text/plain; charset=us-ascii
> Content-Transfer-Encoding: 7bit
>
> Hi,
> the attached patch replaces some cases of
> if ! <something>; then
> with
> if <something>; then : else
>
> because solaris shell doesn't understand the ! (and as this is checking
> some sparc features, it's rather sun-centric :-)
>
> ok to install?
>
> 1999-08-26 Nathan Sidwell <nathan@acm.org>
>
> * configure.in: Don't use shell ! to negate exit codes
This is fine. Please install it.
Sometimes I wonder if we should sit down and write some docs on how to
write truly portable shell scripts. It's amazing the number of weird
problems with shells that we have to work around.
Don't forget to rebuild configure.
Thanks,
jeff