[llvm-commits] Patch: improve protability of llvm-config

[Julien Lerouge]

On Wed, Nov 05, 2008 at 08:28:59PM +0100, Sascha Othman wrote:
> Using Perl 5.6.1 for MSYS (from the DTK) as well, with MSYS 1.0.11 here,
> so an outdated MSYS might be the culprit. The benefit of the change
> would be llvm-config being usable from Windows (without MSYS) and other
> platforms which don't provide a *NIX-like shell and tools like 'which'
> and 'pwd'.

I gave a try to 1.0.11 and I got the same error, then I realized the
problem might come from the "/c/" prefix on the path that ends up being
used in realpath (my build folder is not directly in the msys root).

I do not have a "c" folder in my msys root. It's not created by default,
and I was assuming the /c/ thing would be handled transparently.

But it's not for realpath apparently... so I just did "mkdir /c" and
everything seems to work now, even on 1.0.10. 

So, your initial patch is fine assuming all the magic paths like
"/volume" have a real folder associated with them in the msys root.

Thanks,
Julien

-- 
Julien Lerouge
PGP Key Id: 0xB1964A62
PGP Fingerprint: 392D 4BAD DB8B CE7F 4E5F FA3C 62DB 4AA7 B196 4A62
PGP Public Key from: keyserver.pgp.com