-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
execinfo.h is not available on MUSL #2397
Comments
alpineLinux has libexecinfo |
@radare Good point. Aldo the current backtrace implementation rely's on functions from it, we should have a minimal or different backtrace implementation when it isn't available (or no backtrace in that case) |
|
Ok i manage to compile it by adding the LDFLAGS=-lexecinfo env var but then it fails to do anything because:
|
export VFLAGS="-cc gcc" |
I got tcc to work by replacing what what was put in tmp. $ cat /var/tmp/tcc/bin/tcc
#!/usr/bin/env bash
tcc $@ -lexecinfo What was put in tmp for tcc by the v update process did not work for me anyways on alpine. |
@bogen85 so tcc works with musl with |
@medvednikov well, compiles and links on alpine with -lexecinfo... (and a lot works) but backtrace does not... See #2508 |
With Alpine Linux Manually building and installing https://github.com/ronchaine/libexecinfo fixes the |
I just tried to compile V on chimera linux and got the following error:
After compiling and installing the libexecinfo library (repo has been archived!) the error message when compiling V changed to
Does anyone have any other ideas on how to get this to work reliably on a modern musl based distro? Thank you very much! P.S.: Compiling V on alpine works. |
I think we only tested musl builds on alpine. execinfo is a common source of issues, would be best to avoid this dep. |
V version:
OS:
What did you do?
try to build v for musl,
What did you expect to see?
a working build
What did you see instead?
a failure because the current ifndef is only for BIONIC
in fact execinfo is a gnuism, so it should be guarded like this:
The text was updated successfully, but these errors were encountered: