From: Jim Wilson <wilson@cygnus.com>
To: linux-ia64@vger.kernel.org
Subject: Re: [Linux-ia64] The 1117 snapshot cpp problem
Date: Thu, 08 Feb 2001 00:20:54 +0000 [thread overview]
Message-ID: <marc-linux-ia64-105590693005121@msgid-missing> (raw)
In-Reply-To: <marc-linux-ia64-105590678205880@msgid-missing>
>It is ok if you use "make bootstrap" since $(includedir) passed down
>is ignored during "make bootstrap" in gcc. But the toolchain may not
>be built with "make bootstrap". I believe
>"includedir=$(includedir)" \
>should not be passed to gcc during any build.
I don't like that idea. It defeats the purpose of having the top level
Makefile, which is to ensure that all tools are compatible with each other.
Binutils for instance uses includedir. It will install bfd.h there. So
if binutils uses includedir and gcc does not, then they are no longer
compatible with each other. binutils will install a header file someplace
where gcc can't find it.
I think this also explains part of the reason why the "Cygnus" tree sets
includedir to $prefix/include. If you configure a tree setting $prefix, and
then install it, it is wrong to install files into /usr/local/include.
Everything has to be installed underneath $prefix. This required creating
a new include directory underneath prefix, and making gcc look in it.
Unfortunately, our mistake here was that we replaced /usr/local/include
which other people are using.
This mistake has already been institutionalized by the gcc-2.95 release.
gcc-2.9 supports /usr/local/include. gcc-2.95 supports $prefix/include.
There are people that want both, so I think we need to support both now.
This requires a bigger change though. We have to document the new directory,
add support for the new directory to cpp and anyplace else that cares
(protoize?), and fix up the Makefile to pass both directory names on the
compile line for cpp. This is a big enough change that I was not able to
have it done by yesterday.
Jim
prev parent reply other threads:[~2001-02-08 0:20 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2001-01-05 17:46 [Linux-ia64] The 1117 snapshot cpp problem H . J . Lu
2001-01-24 21:37 ` Jim Wilson
2001-02-05 19:57 ` H . J . Lu
2001-02-06 12:56 ` Andreas Schwab
2001-02-07 7:06 ` H . J . Lu
2001-02-08 0:20 ` Jim Wilson [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=marc-linux-ia64-105590693005121@msgid-missing \
--to=wilson@cygnus.com \
--cc=linux-ia64@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox