-
Notifications
You must be signed in to change notification settings - Fork 232
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
@boost//:stacktrace is missing libbacktrace dependency #534
Comments
For reference, here's a PR with a small patch I put together that adds the libbacktrace dependency: spaceandtimelabs/blitzar#78 But it would probably need to be modified to be usable for a more general context. |
Hey, Ryan! I'm definitely not the biggest Linux expert here, but I think this comes down to currently assuming that libbacktrace is installed on the system--when it's not always by default, and not on your guys'. IMO there's room for a better general Bazel thing here for Linux where rules install what they need (see bazelbuild/bazel#17099), maybe selecting on a flag that lets you build and link statically. |
Anyway, this project is mostly community maintained. (And as much as I wish I could just dive in and fix this one, I don't have bandwidth right now--but I figured I'd hop in and try to help some.) If you're down to contribute to the ecosystem, I think getting Bazel into libbacktrace (or into the central registry and adding it here) would be quite valuable, as could building the package install stuff linked above! Thanks, |
Hey Chris, thanks for getting back to me. I agree that having bazel support for libbacktrace would be valuable -- but that might be a bit too big of a project for me to try to tackle right now. However, I think you could fairly easly add a custom build file for libbacktrace in the same way you are doing for zlib and bzip2 here: https://github.com/nelhage/rules_boost/blob/master/boost/boost.bzl#L117-L124 That's something I might be able to put together a PR for if you'd be ok with a simple build like this: https://github.com/spaceandtimelabs/blitzar/blob/main/WORKSPACE#L30-L45 I can probably rework it so that it doesn't use https://github.com/bazelbuild/rules_foreign_cc if you'd prefer to avoid adding that as a dependency. |
I think that'd be quite valuable! Definitely worth avoiding rules_foreign_cc if we easily can, though, since it has a bunch of limitations esp around cross-platform build, as I recall. If you succeed with that, I still think it might be worth seeing if libbacktrace would take the BUILD file! |
I get this error when building
The text was updated successfully, but these errors were encountered: