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

libquiche.so undefined references #1761

Open
bagder opened this issue Apr 15, 2024 · 2 comments
Open

libquiche.so undefined references #1761

bagder opened this issue Apr 15, 2024 · 2 comments

Comments

@bagder
Copy link
Contributor

bagder commented Apr 15, 2024

When I run curl's configure to try to use quiche from current git, I get these errors:

/usr/bin/ld: $path/libquiche.so: undefined reference to `__gxx_personality_v0'
/usr/bin/ld: $path/libquiche.so: undefined reference to `vtable for __cxxabiv1::__si_class_type_info'
/usr/bin/ld: $path/libquiche.so: undefined reference to `operator delete(void*, unsigned long)'
/usr/bin/ld: $path/libquiche.so: undefined reference to `vtable for __cxxabiv1::__class_type_info'

($path is me hiding my local long path to make it easier to the eye)

Any clues on what I am missing? (These looks like traces of C++, which I suspected might come from BoringSSL, but I am able to build from current BoringSSL without getting this problem.)

@ghedo
Copy link
Member

ghedo commented Apr 16, 2024

Are you using the BoringSSL version vendored by quiche or a different one? Wonder if it's related to https://boringssl-review.googlesource.com/c/boringssl/+/66288 (which would affect newer BoringSSL versions than what quiche currently uses).

@bagder
Copy link
Contributor Author

bagder commented Apr 18, 2024

Ah yes, I have a separate BoringSSL build. I forgot about mentioning that. I updated to their latest git just a few days ago (April 12, 2024).

I too strongly suspect it has something to do with that C++ism they introduced.

ghedo added a commit that referenced this issue May 8, 2024
This is apparently needed to fix a build failure on Windows x86.

Also fixes linking to C++ runtime for BoringSSL
(also see https://boringssl-review.googlesource.com/c/boringssl/+/67147).

Fixes #1761.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants