Skip to content
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

Add libexecinfo to debug builds for BSD #2826

Merged
merged 1 commit into from
Jul 21, 2018

Conversation

slfritchie
Copy link
Contributor

While investigating #2823 for OpenBSD support and also #2825, I also tried twiddling a bit with FreeBSD and discovered that it isn't possible to compile with config=debug successfully. Running gmake config=debug test verbose=true on ponyc's tag 0.24.0 on both FreeBSD 11 + LLVM 3.9.1 and DragonflyBSD 4.8.1 + LLVM 3.9.1:

[...]
c++ -MMD -MP -march=native -mtune=generic -Werror -Wconversion -Wno-sign-conversion -Wextra -Wall -g -DDEBUG -std=gnu++11 -fno-rtti   -c -o build/debug/obj/tests/libponyrt/util.o test/libponyrt/util.cc  -I src/common/ -I src/libponyrt/ -isystem lib/gtest/
Linking libponyrt.tests
c++ -o build/debug/libponyrt.tests build/debug/obj/tests/libponyrt/ds/fun.o build/debug/obj/tests/libponyrt/ds/hash.o build/debug/obj/tests/libponyrt/ds/list.o build/debug/obj/tests/libponyrt/lang/error.o build/debug/obj/tests/libponyrt/mem/heap.o build/debug/obj/tests/libponyrt/mem/pagemap.o build/debug/obj/tests/libponyrt/mem/pool.o build/debug/obj/tests/libponyrt/util.o -march=native -mtune=generic -L build/debug -lgtest -lponyrt -lpthread -L /usr/local/lib -rdynamic
build/debug/libponyrt.a(ponyassert.o): In function `ponyint_assert_fail':
/usr/home/vagrant/ponyc/src/libponyrt/platform/ponyassert.c:42: undefined reference to `backtrace'
/usr/home/vagrant/ponyc/src/libponyrt/platform/ponyassert.c:43: undefined reference to `backtrace_symbols'
c++: error: linker command failed with exit code 1 (use -v to see invocation)
gmake: *** [Makefile:821: build/debug/libponyrt.tests] Error 1

@slfritchie slfritchie added bug: 4 - in progress changelog - fixed Automatically add "Fixed" CHANGELOG entry on merge labels Jul 16, 2018
@slfritchie slfritchie mentioned this pull request Jul 17, 2018
@SeanTAllen
Copy link
Member

I hope the qemu based BSD ci tests work out. Doubtful though given when CI jobs time out.

@dipinhora
Copy link
Contributor

Merging to include in release 0.24.1.

@dipinhora dipinhora merged commit bf67183 into ponylang:master Jul 21, 2018
ponylang-main added a commit that referenced this pull request Jul 21, 2018
@slfritchie slfritchie deleted the bsd-libexecinfo branch July 23, 2018 20:42
valpackett added a commit to valpackett/ponyc that referenced this pull request Oct 17, 2018
ponylang#2826 restricted -lexecinfo to debug compilers (not even to building something in debug mode).
This doesn't make much sense. I'm not sure why ordinary builds can succeed without -lexecinfo,
but bitcode-runtime builds can't. Either way, if it's always included on non-glibc-Linux, it should
always be included on BSD.
jemc pushed a commit that referenced this pull request Nov 2, 2018
#2826 restricted -lexecinfo to debug compilers (not even to building something in debug mode).
This doesn't make much sense. I'm not sure why ordinary builds can succeed without -lexecinfo,
but bitcode-runtime builds can't. Either way, if it's always included on non-glibc-Linux, it should
always be included on BSD.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog - fixed Automatically add "Fixed" CHANGELOG entry on merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants