PATCH for a rather bad cpplib related bug

[Manfred Hollstein]

On Wed, 7 July 1999, 01:35:43, law@cygnus.com wrote:

 > 
 >   In message < 14206.1175.702991.775614@saturn.hollstein.net >you write:
 >   > This  is a fix   for a  rather  bad bug   in all recent  egcs versions
 >   > (including  the release  branch  for   gcc-2.95!!!) which  have   been
 >   > configured to use the CPPLIB C preprocessor, i.e. `--enable-c-cpplib'.
 >   > 
 >   > Suppose the following:
 >   > 
 >   > $ cat << - > t.c
 >   > #include <ThisFileDoesNotExist.h>
 >   > 
 >   > main () {}
 >   > $ gcc -c t.c
 >   > t.c:1: ThisFileDoesNotExist.h: No such file or directory
 >   > $ 
 >   > 
 >   > BUT, it terminates with an exit status 0!!!!!  This renders nearly all
 >   > configure scripts useless, as they  depend on a  proper exit status of
 >   > the  invoked   compiler.  In  fact, I recognized   this while building
 >   > xemacs-20.4 on  i686-redhat6.0-linux-gnu with  a  gcc-2.95  configured
 >   > with the `--enable-c-cpplib'   option  while  it  was   looking for  a
 >   > non-existing db/db.h, deciding it was there....
 >   > 
 >   > I know, the release branch is frozen  right now, but  I think, this is
 >   > actually a serious bug worth to be fixed for the upcoming release.
 >   > 
 >   > What do you think? OK to install?
 >   > 
 >   > Cheers,
 >   > manfred
 >   > 
 >   > 
 >   > 1999-07-03  Manfred Hollstein  <mhollstein@cygnus.com>
 >   > 
 >   > 	* c-lex.c (finish_parse): If preprocessing has failed, exit with
 >   > 	a failure.
 > Don't you need to fix the other front-ends that conditionally use cpplib
 > (like C++?)

Yep, absolutely true.  Dave Brolley has installed an appropriate patch already.

 > 
 > jeff
 > 
 > 
 > 

manfred