macos: use GAUDI_LIBRARY_PATH instead of LD_...

This is a quick, macOS-only implementation of the suggestion in #251 , finally fixing Gaudi on macOS for me (after setting export GAUDI_LIBRARY_PATH=$LD_LIBRARY_PATH)

Should be safe to merge as the changes are only inside APPLE ifdefs, but does make the code a bit more complicated.

changed the description

added 1 commit

• 217be1b6 - pre-commit fixes

Compare with previous version

added 1 commit

• ad9d43ea - fix import

Compare with previous version

It looks mostly OK to me, apart from a couple of minor changes.

I'm not too convinced of the relative RPATH, but I have to think a bit more about it.

changed milestone to %v36r13

added lhcb-gaudi-head label

assigned to @clemenci

I was thinking that we may use the same logic on all platforms, may be calling the environment variable GAUDI_PLUGIN_PATH. If the library cannot be found in one of the directories listed there, then we can fallback on LD_LIBRARY_PATH.

• [2023-05-24 00:04] Validation started with lhcb-gaudi-head#3572
• [2023-05-24 00:26] Validation started with lhcb-run2-gaudi-head#529
• [2023-05-25 00:03] Validation started with lhcb-gaudi-head#3573
• [2023-05-25 00:20] Validation started with lhcb-run2-gaudi-head#530
• [2023-05-26 00:03] Validation started with lhcb-gaudi-head#3574
• [2023-05-26 00:28] Validation started with lhcb-run2-gaudi-head#531
• [2023-05-27 00:03] Validation started with lhcb-gaudi-head#3575
• [2023-05-27 00:24] Validation started with lhcb-run2-gaudi-head#532
• [2023-05-28 00:03] Validation started with lhcb-gaudi-head#3576
• [2023-05-28 00:22] Validation started with lhcb-run2-gaudi-head#533
• [2023-05-29 00:03] Validation started with lhcb-gaudi-head#3577
• [2023-05-29 00:21] Validation started with lhcb-run2-gaudi-head#534
• [2023-05-30 00:03] Validation started with lhcb-gaudi-head#3578
• [2023-05-30 00:20] Validation started with lhcb-run2-gaudi-head#535

resolved all threads

resolved all threads

added 1 commit

• 87026e0c - Apply 2 suggestion(s) to 2 file(s)

Compare with previous version

added 1 commit

• 7c249859 - use naming GAUDI_PLUGIN_PATH instead

Compare with previous version

resolved all threads

mentioned in issue #267

approved this merge request

mentioned in commit 70fe70b5

merged