public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
From: Neil Booth <neil@daikokuya.co.uk>
To: gcc-patches@gcc.gnu.org, Alexandre Oliva <aoliva@redhat.com>,
	Geoff Keating <geoffk@geoffk.org>
Subject: Re: Updates to --with-sysroot support
Date: Fri, 13 Dec 2002 12:20:00 -0000	[thread overview]
Message-ID: <20021213202005.GC27312@daikokuya.co.uk> (raw)
In-Reply-To: <20021112194148.GA7324@nevyn.them.org>

Daniel Jacobowitz wrote:-

> I'm resending this patch with a couple of additions; I think this
> addresses all concerns.
> 
> On Mon, Nov 04, 2002 at 11:57:29AM -0500, Daniel Jacobowitz wrote:
> > I spent a couple days working with this and finally found a solution for
> > (most of) my concerns.  This patch:
> > 
> >  - Fixes an undocumented behavior in cppinit.c which I consider to be a bug.
> >    If -iprefix was used to change the compiler prefix, then first the
> >    standard include directories which begin with the prefix were searched
> >    (using the relocated prefix); then all directories were searched in order
> >    (using the original prefix).  This means that -iprefix'd versions of
> >    GCC_INCLUDE_DIR (that's -iprefix's prefix followed by "include", folks)
> >    would be searched before GPLUSPLUS_INCLUDE_DIR.  Oops.
> >    I changed this so that the order is preserved, and unrelocated versions
> >    are not searched.  This means that people who were using -iprefix and
> >    still expecting to get GCC's <stddef.h> et al. from the normal location
> >    are out of luck; they'll have to specify that path as an -isystem
> >    explicitly.  I think this change is correct but I'd like a second
> >    opinion.
> 
> Neil and Zack seemed to agree with me on this one, whew.

Daniel,

Would you commit your CPP changes to BIB?  I am changing include
handling (you probably already have a merge conflict, but I've not
touched what you touched) and I don't want to wait any longer for
these nice improvements.

Thanks,

Neil.

      parent reply	other threads:[~2002-12-13 20:20 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-11-04  8:56 Daniel Jacobowitz
2002-11-04  9:08 ` Jason R Thorpe
2002-11-04 10:26 ` Hans-Peter Nilsson
2002-11-04 10:30   ` Daniel Jacobowitz
2002-11-06  9:07     ` Alexandre Oliva
2002-11-04 23:29 ` Zack Weinberg
2002-11-05  6:30   ` Daniel Jacobowitz
2002-11-05 10:51   ` Geoff Keating
2002-11-06  9:15 ` Alexandre Oliva
2002-11-06  9:30   ` Daniel Jacobowitz
2002-11-06 10:04     ` Alexandre Oliva
2002-11-06 10:14       ` Daniel Jacobowitz
2002-11-06 12:49         ` Alexandre Oliva
2002-11-06 12:59           ` Daniel Jacobowitz
2002-11-06 13:30             ` Alexandre Oliva
2002-11-12 11:40 ` Daniel Jacobowitz
2002-11-14 13:25   ` Daniel Jacobowitz
2002-12-07  8:44     ` Neil Booth
2002-12-13 12:20   ` Neil Booth [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=20021213202005.GC27312@daikokuya.co.uk \
    --to=neil@daikokuya.co.uk \
    --cc=aoliva@redhat.com \
    --cc=gcc-patches@gcc.gnu.org \
    --cc=geoffk@geoffk.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;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).