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 no other places.

I could imagine a sysroot path that isn't just '/' but ends with '/' 
resulting in duplicate '/' in the middle of the path - although that's not 
a correctness issue in the way that '//' at the start could be, maybe the 
best check is actually for '/' at end of sysroot (in which case skip the 
'/' at the start of the path within the sysroot)?  Or remove such a '/' at 
end of sysroot at configure time rather than checking for it later....

-- 
Joseph S. Myers
jos...@codesourcery.com

Reply via email to