D53557-hurd-self-exe-realpath
authorLLVM Packaging Team <pkg-llvm-team@lists.alioth.debian.org>
Sun, 16 Dec 2018 12:07:54 +0000 (12:07 +0000)
committerSylvestre Ledru <sylvestre@debian.org>
Sun, 16 Dec 2018 12:07:54 +0000 (12:07 +0000)
commit815723a7755e34f4dc22f882e635dde8d594bbcf
tree6cb6af8315a41383cee8efcae9f691afffd9de30
parentd4a4d15d8117f2d8d153d32fb6e0798484b8e2c8
D53557-hurd-self-exe-realpath

r345104 | rnk | 2018-10-24 01:35:43 +0200 (mer. 24 oct. 2018) | 25 lignes

[hurd] Make getMainExecutable get the real binary path

On GNU/Hurd, llvm-config is returning bogus value, such as:

$ llvm-config-6.0 --includedir
/usr/include

while it should be:
$ llvm-config-6.0 --includedir
/usr/lib/llvm-6.0/include

This is because getMainExecutable does not get the actual installation
path. On GNU/Hurd, /proc/self/exe is indeed a symlink to the path that
was used to start the program, and not the eventual binary file. Llvm's
getMainExecutable thus needs to run realpath over it to get the actual
place where llvm was installed (/usr/lib/llvm-6.0/bin/llvm-config), and
not /usr/bin/llvm-config-6.0. This will not change the result on Linux,
where /proc/self/exe already points to the eventual file.

Patch by Samuel Thibault!

While making changes here, I reformatted this block a bit to reduce
indentation and match 2 space indent style.

Differential Revision: https://reviews.llvm.org/D53557

Gbp-Pq: Name D53557-hurd-self-exe-realpath.diff
lib/Support/Unix/Path.inc