add patch to Python 3.13.5 + 3.14.2 easyconfigs to fix ctypes when $LD_LIBRARY_PATH is not being set#25106
Conversation
|
@boegelbot please test @ jsc-zen3 |
|
@bedroge: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de PR test command '
Test results coming soon (I hope)... Details- notification for comment with ID 3772914867 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @casparvl |
|
Test report by @boegelbot |
|
@boegelbot please test @ jsc-zen3 |
|
@bedroge: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de PR test command '
Test results coming soon (I hope)... Details- notification for comment with ID 3773201401 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @boegelbot |
|
Need to update the patch for 3.14. |
|
@boegelbot please test @ jsc-zen3 |
|
@bedroge: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de PR test command '
Test results coming soon (I hope)... Details- notification for comment with ID 3773562209 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @boegelbot |
$LD_LIBRARY_PATH is not being set
|
@casparvl Still needs some love? |
|
Looks like it failed because |
|
@boegelbot please test @ jsc-zen3 |
|
@bedroge: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de PR test command '
Test results coming soon (I hope)... Details- notification for comment with ID 3777123340 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @bedroge edit: this failed because I wiped the (temporary) working dir in which EB was started 😅 will try again... |
|
Test report by @boegelbot |
|
Test report by @bedroge edit: this was without filtering |
|
@boegelbot please test @ jsc-zen3 |
|
@bedroge: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de PR test command '
Test results coming soon (I hope)... Details- notification for comment with ID 3777425408 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @bedroge edit: this was with filtering |
|
Test report by @bedroge edit: this is a build on top of EESSI 2025.06 (i.e. with filtering |
|
Test report by @boegelbot |
There was a problem hiding this comment.
This looks fine to me. One final request @bedroge :
Can you run the script that I pasted in #23499 (review) on one of the patched Python installations (i.e. one done while filtering LD_LIBRARY_PATH) to test if the behavior/output is indeed as expected (i.e. identical to what was mentioned in that comment?). If it produces the expected output, I'd say this is ready to go.
|
Adapted your script a bit for my local stack (EESSI doesn't have SDL2 / CUDA yet for 2025.06, which we need for this Python version) where Python was installed with Running it results in: |
|
Also ran a similar script on top of EESSI 2025.06 (without CUDA, and with a Python 3.13.5 reinstalled using EESSI-extend with the easyconfig from this PR): Output: |
See #23499, but this was left out for new Python versions.