This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: latest snapshot won't find C++ header files
- To: law at cygnus dot com
- Subject: Re: latest snapshot won't find C++ header files
- From: Alexandre Oliva <oliva at dcc dot unicamp dot br>
- Date: 11 Oct 1998 15:26:12 -300
- Cc: egcs-bugs at cygnus dot com, egcs-patches at cygnus dot com
- References: <29263.908093664@hurl.cygnus.com>
Jeffrey A Law <law@cygnus.com> writes:
> In message <oriuhvssr2.fsf@araguaia.dcc.unicamp.br>you write:
>> I'm just trying to make the Makefile consistent and homogeneous. The
>> current Makefile produces inconsistent behavior (i.e., different
>> search dirs) depending on whether you enable cpplib or not, it breaks
>> support for --with-gxx-include-dir and --with-local-prefix, and it
>> causes cppp to search for C++ header-files in the wrong directory.
> I made cpplib consistent (I think) with cccp, though apparently
> we're still having trouble with the include files right?!?
Sure, libstdc++ installs itself in include/g++-2, and the current
sources make cppp look for it in include/g++. Furthermore, if you
specify --with-gxx-include-dir and --with-local-prefix, these will
just be ignored, which is not an improvement, IMO. My patch fixes
just these two problems, nothing else. It also introduces the
unsublibdir modification for include-files in a single point, which
makes it easier to fix in the future, if there's any need for that.
The latest version of the patch, that fixes the modification in
cpplib.o too, can be found below. Please look carefully at it this
time :-) Ok to install?
The unsublibdir patch currently installed also presents another
serious failure, that is the assumption that exec_prefix is the same
as prefix or a subdir thereof. IMO, this is too strong an assumption,
but I'm not trying to fix this problem with this patch.
--
Alexandre Oliva
mailto:oliva@dcc.unicamp.br mailto:oliva@gnu.org mailto:aoliva@acm.org
http://www.dcc.unicamp.br/~oliva
Universidade Estadual de Campinas, SP, Brasil
include.diff