-
Notifications
You must be signed in to change notification settings - Fork 645
[OSRM] Add new binary, second try... #8893
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
[OSRM] Add new binary, second try... #8893
Conversation
|
Error on [ Info: Checking shared library lib/libosrm.so
--
ERROR: could not load library "/cache/build/yggy-amdci7-8/julialang/yggdrasil/O/OSRM/build/x86_64-linux-gnu-cxx11/ZsPx5jB0/x86_64-linux-gnu-libgfortran5-cxx11/destdir/lib/libosrm.so"
| /cache/julia-buildkite-plugin/julia_installs/bin/linux/x64/1.7/julia-1.7-latest-linux-x86_64/bin/../lib/julia/libstdc++.so.6: version `GLIBCXX_3.4.31' not found (required by /cache/build/yggy-amdci7-8/julialang/yggdrasil/O/OSRM/build/x86_64-linux-gnu-cxx11/ZsPx5jB0/x86_64-linux-gnu-libgfortran5-cxx11/destdir/lib/libosrm.so)
| ┌ Warning: lib/libosrm.so cannot be dlopen()'ed |
|
Error on ld64.lld: error: undefined symbol: osrm::extractor::CompressedEdgeContainer::GetBucketReference(unsigned int) const |
That is because it is using GCC 13, so it gets linked against a newer glibcxx version because that is what ships with it. You need to lower the GCC version requested to get around this. |
If I lower GCC below v12, other target builds break. Is there another strategy for dealing with this issue (I don't exactly understand what the problem is...all of the dependencies were built with earlier GCC versions). |
|
Thanks both for working on this! I'm away from the computer now, but the osrm-again branch of my Yggdrasil fork was building last I checked, not sure if you've looked at those changes. |
I've included your cmake patch. this version builds on even more platforms, just seeing if I can get mingw to build; the last failures are risc, etc. due to onetbb not being available for the platform |
|
Does the resent OSRM 6.0 release help here by any chance? |
fingolfin
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jeremiahpslewis thanks for your contribution; are you still interested in completing this? If so, see my comments below, and I think a rebase onto / merge of lastest master into your PR to re-run CI and get up-to-date build failure info, would be in order.
| }, | ||
| "privileged": true, | ||
| "onCreateCommand": "julia -e 'using Pkg; Pkg.precompile();'" | ||
| "onCreateCommand": "julia -e 'using Pkg; Pkg.precompile();'", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think this file should be modified in this PR
| # OSRM requires C++20, which needs a newer SDK | ||
| ArchiveSource("https://github.com/realjf/MacOSX-SDKs/releases/download/v0.0.1/MacOSX12.3.sdk.tar.xz", | ||
| "a511c1cf1ebfe6fe3b8ec005374b9c05e89ac28b3d4eb468873f59800c02b030"), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be switched to the require_macos_sdk helper, see e.g. fddd43f for examples how to do that.
No description provided.