This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH][DRIVER] Wrong C++ include paths when configuring with "--with-sysroot=/"

From: Yvan Roux <yvan dot roux at linaro dot org>
To: Joseph Myers <joseph at codesourcery dot com>
Cc: Pavel Kopyl <p dot kopyl at samsung dot com>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>, Yury Gribov <y dot gribov at samsung dot com>, Viacheslav Garbuzov <v dot garbuzov at samsung dot com>, Matthias Klose <doko at ubuntu dot com>
Date: Wed, 15 Jul 2015 10:34:01 +0200
Subject: Re: [PATCH][DRIVER] Wrong C++ include paths when configuring with "--with-sysroot=/"
Authentication-results: sourceware.org; auth=none
References: <55353241 dot 5020105 at samsung dot com> <alpine dot DEB dot 2 dot 10 dot 1505072202090 dot 20024 at digraph dot polyomino dot org dot uk> <CAD57uCeOmaToT+fKV4obWUFUnRAtBmikwMDaZkD5+PwEzcPjkA at mail dot gmail dot com> <alpine dot DEB dot 2 dot 10 dot 1505211657510 dot 17178 at digraph dot polyomino dot org dot uk>

Hi,

(Sorry for the delay I'm just back from a long sick leave)

>> There is this old patch submitted by Matthias on that same issue, if
>> its logic is the right one for you Joseph I can rebase/validate it
>> Joseph.
>>
>> https://gcc.gnu.org/ml/gcc-patches/2012-02/msg00320.html
>
> Yes, that seems better.

I've rebased the patch on trunk, bootstrap is ok and when configuring
with options:
"--with-sysroot=/ --with-gxx-include-dir=/usr/include/c++/4.9.2"
gcc_gxx_include_dir keeps its leading slash.

Is it ok for trunk ?

Thanks,
Yvan


2015-07-15  Yvan Roux  <yvan.roux@linaro.org>
                  Matthias Klose  <doko@ubuntu.com>

       * configure.ac: Move AC_ARG_WITH checks for native-system-header-dir,
       build-sysroot, sysroot from the `Miscenalleous configure options'
       to the `Directories' section and strip trailing `/' from with_sysroot.
       (gcc_gxx_include_dir): Don't strip a `/' sysroot value.
       * configure: Regenerated.

Attachment: sysroot-path.diff
Description: Text document

Follow-Ups:
- Re: [PATCH][DRIVER] Wrong C++ include paths when configuring with "--with-sysroot=/"
  - From: Jeff Law

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]