-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
node: undefined reference to absl::lts_20230802 #25483
Comments
Hi @vortexilation make package/feeds/packages/node/host/{clean,compile}
make package/feeds/packages/node/{clean,compile} |
@nxhack Thanks for joining in this issue. |
(It's just a guess...) Perhaps you should prepare separate host-build and target-build versions of the linking library. |
If the target is not x86_64, it seems like you can clearly see where the error is. |
Your suggestion is for protobuf?, can you elaborate more or give an example?.
The target is x86_64, just like the build host. |
More proper fixes :
Now it doesn't collide anymore with other compiled abseil-cpp. @vidplace7 Ping |
+ ['OS != "aix" and OS != "os400" and OS != "mac" and OS != "ios"', {
+ 'ldflags': [
+ '-Wl,--whole-archive',
+ '<(obj_dir)/tools/v8_gypfiles/<(STATIC_LIB_PREFIX)v8_abseil<(STATIC_LIB_SUFFIX)',
+ ],
+ }], This part didn't seem to be necessary in my testing. Submitted a slightly modified patch-change as #25582 👍 for @nxhack's review. |
Maintainer: @nxhack
Environment: x86_64, glibc
Description:
Posting it in here also.
It seems that node will failed to compile if there is already compiled abseil-cpp in $(STAGING_DIR_HOSTPKG)/lib dir with the following error.
Abseil-cpp was compiled from bundled in protobuf 25.3, here is the draft Makefile :
Node 22.11.0 will able to compile with the following patch :
but will segmentation fault when running npm or compiling node-yarn :
The text was updated successfully, but these errors were encountered: