Open maleadt opened 4 years ago
Yeah, there's not really going to be a way to fix this in BinaryProvider; we have to set the LD_LIBRARY_PATH
for some executable products, and as long as the interface that BinaryProvider
exports is just a string pointing to the executable (as opposed to something more advanced like a function that you call with a do
block, as in JLL packages) we can't stop setting LD_LIBRARY_PATH
.
As a workaround, you can manually edit the autogenerated build.jl
files to remove the setting of LD_LIBRARY_PATH
if your binaries don't require anything within the Julia library directory.
Yeah, I realized that when looking into the problem a bit more... I'm surprised this hasn't been a problem before, with the Julia library path providing widely-used libraries like libz
or (here) libcurl
.
libz is very stable, so you rarely need to worry about that. (It's quite backwards-compatible).
libcurl is less stable, but it's usually pretty backwards-compatible. Are you running on a super-new distribution? In the past, we've generally been shipping newer versions than the OS provides (Which is often okay) but in this case perhaps the OS version is actually the newer one.
This happened in NewPkgEval, which is running ubuntu:latest
(18.04), so nothing particularly new.
Isn't the issue in this case not backwards compatibility, but the fact that the libcurl
included with the current GDAL build are not full featured builds?
This is probably what @SimonDanisch mentioned in https://github.com/JuliaPackaging/BinaryProvider.jl/issues/129#issuecomment-455777442
The linker warning is telling; this is due to a different libcurl getting loaded:
This is due to modifying LD_LIBRARY_PATH in a BinaryProvider-generated deps script:
https://github.com/JuliaPackaging/BinaryProvider.jl/blob/d57da3f92ad843ea8529a666ca4ea2151e9e0c64/src/Products.jl#L480-L502
Spotted in PkgEval runs: https://github.com/JuliaCI/NanosoldierReports/blob/f062a47c6bb3b5dd7c7bd29e7a3cc53cc6feeedd/pkgeval/by_hash/c32d166_vs_2e6715c/logs/GeoArrays/1.4.0-DEV-015cd56127.log#L290-L294 https://github.com/JuliaCI/NanosoldierReports/blob/f062a47c6bb3b5dd7c7bd29e7a3cc53cc6feeedd/pkgeval/by_hash/c32d166_vs_2e6715c/logs/DensityRatioEstimation/1.4.0-DEV-015cd56127.log#L336-L349
cc @KristofferC