Re: [patch] avoid '//' prefixes when sysroot is set to '/'

2012-02-07 Thread Matthias Klose
On 26.01.2012 18:57, Joseph S. Myers wrote: On Thu, 26 Jan 2012, Matthias Klose wrote: On 25.01.2012 17:45, Joseph S. Myers wrote: On Wed, 25 Jan 2012, Matthias Klose wrote: This can end up in generation for dependency files, and other files parsing the output. The solution I came up with

Re: [patch] avoid '//' prefixes when sysroot is set to '/'

2012-02-07 Thread Joseph S. Myers
On Wed, 8 Feb 2012, Matthias Klose wrote: there is one more issue, when configuring --with-sysroot=/ --with-gxx-include-dir=/usr/include/c++/4.7 in that the leading / is stripped away in configure.ac. This case needs an explicit check. Ok for the trunk? This looks like a case where any

Re: [patch] avoid '//' prefixes when sysroot is set to '/'

2012-02-07 Thread Matthias Klose
On 08.02.2012 02:01, Joseph S. Myers wrote: On Wed, 8 Feb 2012, Matthias Klose wrote: there is one more issue, when configuring --with-sysroot=/ --with-gxx-include-dir=/usr/include/c++/4.7 in that the leading / is stripped away in configure.ac. This case needs an explicit check. Ok for the

Re: [patch] avoid '//' prefixes when sysroot is set to '/'

2012-01-26 Thread Matthias Klose
On 25.01.2012 17:45, Joseph S. Myers wrote: On Wed, 25 Jan 2012, Matthias Klose wrote: This can end up in generation for dependency files, and other files parsing the output. The solution I came up with is to check for sysroot set to '/' and special case this in two places. Afaics, there are

Re: [patch] avoid '//' prefixes when sysroot is set to '/'

2012-01-26 Thread Joseph S. Myers
On Thu, 26 Jan 2012, Matthias Klose wrote: On 25.01.2012 17:45, Joseph S. Myers wrote: On Wed, 25 Jan 2012, Matthias Klose wrote: This can end up in generation for dependency files, and other files parsing the output. The solution I came up with is to check for sysroot set to '/'