[issue45371] distutil's runtime_library_dir (rpath) option doesn't work with clang

2021-10-06 Thread Filipe Laíns
Filipe Laíns added the comment: The fix that was merged seems a bit hacky to me -- it changes the _is_gcc check so that it returns True on clang. Even though distutils is deprecated and no longer synced externally, if someone needs to issue a similar fix, this might trip them over. I would r

[issue45371] distutil's runtime_library_dir (rpath) option doesn't work with clang

2021-10-06 Thread Christian Heimes
Change by Christian Heimes : -- resolution: fixed -> stage: resolved -> needs patch status: closed -> open ___ Python tracker ___ _

[issue45371] distutil's runtime_library_dir (rpath) option doesn't work with clang

2021-10-05 Thread Christian Heimes
Christian Heimes added the comment: Thanks for confirming that the fix works for you. The fix will be in 3.10.1, which should be released in early December. -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python

[issue45371] distutil's runtime_library_dir (rpath) option doesn't work with clang

2021-10-05 Thread miss-islington
miss-islington added the comment: New changeset 3ce5e07e9a4b78302b69f898527396ff7b5fd448 by Miss Islington (bot) in branch '3.9': bpo-45371: Fix distutils' rpath support for clang (GH-28732) https://github.com/python/cpython/commit/3ce5e07e9a4b78302b69f898527396ff7b5fd448 -- __

[issue45371] distutil's runtime_library_dir (rpath) option doesn't work with clang

2021-10-05 Thread miss-islington
miss-islington added the comment: New changeset 3733dddecaa332966e200718d4993545f8e52247 by Miss Islington (bot) in branch '3.10': [3.10] bpo-45371: Fix distutils' rpath support for clang (GH-28732) (GH-28733) https://github.com/python/cpython/commit/3733dddecaa332966e200718d4993545f8e52247

[issue45371] distutil's runtime_library_dir (rpath) option doesn't work with clang

2021-10-05 Thread miss-islington
Change by miss-islington : -- pull_requests: +27082 pull_request: https://github.com/python/cpython/pull/28734 ___ Python tracker ___ __

[issue45371] distutil's runtime_library_dir (rpath) option doesn't work with clang

2021-10-05 Thread miss-islington
Change by miss-islington : -- pull_requests: +27081 pull_request: https://github.com/python/cpython/pull/28733 ___ Python tracker ___ __

[issue45371] distutil's runtime_library_dir (rpath) option doesn't work with clang

2021-10-05 Thread miss-islington
miss-islington added the comment: New changeset ef6196028f966f22d82930b66e1371e75c5df2f7 by Christian Heimes in branch 'main': bpo-45371: Fix distutils' rpath support for clang (GH-28732) https://github.com/python/cpython/commit/ef6196028f966f22d82930b66e1371e75c5df2f7 -- nosy: +mis

[issue45371] distutil's runtime_library_dir (rpath) option doesn't work with clang

2021-10-05 Thread Chris Hills
Chris Hills added the comment: I tested this both with and without LDFLAGS="-Wl,-rpath -Wl,/home/chaz/.local/local/python3.10.0/lib", and in both cases this patch works as expected. Thank you! -- nosy: +chaz6 ___ Python tracker

[issue45371] distutil's runtime_library_dir (rpath) option doesn't work with clang

2021-10-05 Thread Christian Heimes
Christian Heimes added the comment: $ CC=clang ./configure --with-openssl=/home/heimes/dev/python/multissl/openssl/3.0.0 --with-openssl-rpath=auto $ make $ readelf -d build/lib.linux-x86_64-3.11/_ssl.cpython-311-x86_64-linux-gnu.so | grep RUNPATH 0x001d (RUNPATH)Libr

[issue45371] distutil's runtime_library_dir (rpath) option doesn't work with clang

2021-10-05 Thread Christian Heimes
Change by Christian Heimes : -- keywords: +patch pull_requests: +27080 stage: -> patch review pull_request: https://github.com/python/cpython/pull/28732 ___ Python tracker ___

[issue45371] distutil's runtime_library_dir (rpath) option doesn't work with clang

2021-10-05 Thread Christian Heimes
New submission from Christian Heimes : Chris Hills reported in bpo-43466 that the new --with-openssl-rpath=auto does not work with clang. It turns out to be a bug in distutils. UnixCCompiler.runtime_library_dir_option() does not detect clang correctly and emits wrong option for rpath. --