public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
From: Daniel Jacobowitz <drow@mvista.com>
To: Alexandre Oliva <aoliva@redhat.com>
Cc: Hans-Peter Nilsson <hans-peter.nilsson@axis.com>,
	gcc-patches@gcc.gnu.org
Subject: Re: Bugs in sysroot patches resulting in $(local_include)/include always searched, ../-expansion broken
Date: Tue, 04 Mar 2003 19:25:00 -0000	[thread overview]
Message-ID: <20030304192456.GA14297@nevyn.them.org> (raw)
In-Reply-To: <ork7fflz2o.fsf@free.redhat.lsd.ic.unicamp.br>

On Tue, Mar 04, 2003 at 04:19:11PM -0300, Alexandre Oliva wrote:
> On Feb 22, 2003, Daniel Jacobowitz <drow@mvista.com> wrote:
> 
> > The change was that when you run gcc
> > from outside the installation prefix, nothing inside the installation
> > prefix will be searched.
> 
> Not even the original (unrelocated) installation prefix?  This sounds
> wrong to me.  If I read you correctly, this means that the sys-root
> isn't searched during the GCC build process.  Is this the effect of
> your change?  If so, I don't agree it's the right thing to do.

In theory it could mean that, but there's something else involved that
we discussed: we only relocate the sysroot if (the sysroot was inside
the prefix and) the new directory would exist.  If you happen to have
something matching the sysroot location from inside the build tree,
then I assume it's deliberate.

This is pretty important.  Consider:

  /opt/manufacturer/bin/gcc
  /opt/manufacturer/target/usr/include/version2.h

  /opt/manufacturer-old/bin/gcc
  /opt/manufacturer-old/target/usr/include/version1.h

I believe that (even though both were configured for
/opt/manufacturer), the one in /opt/manufacturer-old should not search
there and autoconf checks for version2.h should fail.

-- 
Daniel Jacobowitz
MontaVista Software                         Debian GNU/Linux Developer

  reply	other threads:[~2003-03-04 19:25 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-02-20 15:09 Hans-Peter Nilsson
2003-02-20 15:16 ` Daniel Jacobowitz
2003-02-20 16:24   ` Hans-Peter Nilsson
2003-02-20 18:49     ` Daniel Jacobowitz
2003-02-20 20:21       ` Hans-Peter Nilsson
2003-03-03 17:04       ` Daniel Jacobowitz
2003-03-04 19:18         ` Alexandre Oliva
2003-02-20 19:09 ` Daniel Jacobowitz
2003-02-20 20:36   ` Hans-Peter Nilsson
2003-02-22  2:05   ` Hans-Peter Nilsson
2003-02-22  3:56     ` Daniel Jacobowitz
2003-02-22  4:12       ` Hans-Peter Nilsson
2003-02-22  4:25         ` Daniel Jacobowitz
2003-03-04 19:20           ` Alexandre Oliva
2003-03-04 19:25             ` Daniel Jacobowitz [this message]
2003-03-04 21:18               ` Alexandre Oliva
2003-03-04 21:32                 ` Daniel Jacobowitz
2003-03-04 21:58                   ` Alexandre Oliva

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=20030304192456.GA14297@nevyn.them.org \
    --to=drow@mvista.com \
    --cc=aoliva@redhat.com \
    --cc=gcc-patches@gcc.gnu.org \
    --cc=hans-peter.nilsson@axis.com \
    /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).