Skip to content
This repository has been archived by the owner on Apr 22, 2023. It is now read-only.

node.gyp change breaks build when using system v8 library #6629

Closed
CraZySacX opened this issue Dec 3, 2013 · 4 comments
Closed

node.gyp change breaks build when using system v8 library #6629

CraZySacX opened this issue Dec 3, 2013 · 4 comments
Labels

Comments

@CraZySacX
Copy link

This change in node.gyp

[
'OS=="linux"', {
'ldflags': [
'-Wl,--whole-archive <(PRODUCT_DIR)/obj.target/deps/v8/tools/gyp/libv8_base.a -Wl,--no-whole-archive',
],
}],

breaks the build when using --shared-v8, --shared-v8-includes and --shared-v8-libs as the libv8_base.a file doesn't exist.

@tjfontaine
Copy link

Sorry about that, I will fix that this morning

@tjfontaine
Copy link

@CraZySacX can you try

diff --git a/node.gyp b/node.gyp
index a0a1774..f8f4a9a 100644
--- a/node.gyp
+++ b/node.gyp
@@ -320,7 +320,7 @@
           ],
         }],
         [
-          'OS=="linux"', {
+          'OS=="linux" and node_shared_v8=="false"', {
             'ldflags': [
               '-Wl,--whole-archive <(PRODUCT_DIR)/obj.target/deps/v8/tools/gyp/libv8_base.<(target_arch).a -Wl,--no-whole-archive',
             ],

@ghost ghost assigned tjfontaine Dec 6, 2013
@CraZySacX
Copy link
Author

That fix works.

tjfontaine added a commit that referenced this issue Dec 10, 2013
@tjfontaine
Copy link

thanks fixed in 92bbd60

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants