Reviewers: Michael Achenbach,

Description:
Fix builds w/ component=="shared_library" and v8_use_external_startup_data==1.

[email protected]
BUG=

Please review this at https://codereview.chromium.org/794213002/

Base URL: https://chromium.googlesource.com/v8/v8.git@master

Affected files (+14, -12 lines):
  M test/cctest/cctest.gyp
  M test/unittests/unittests.gyp


Index: test/cctest/cctest.gyp
diff --git a/test/cctest/cctest.gyp b/test/cctest/cctest.gyp
index 6e00713e02d087f5c397fdf42ce4f988817a0eb6..9a39945b80e9e9faf24ec09802400bbf949346b1 100644
--- a/test/cctest/cctest.gyp
+++ b/test/cctest/cctest.gyp
@@ -259,13 +259,14 @@
           # cctest can't be built against a shared library, so we need to
           # depend on the underlying static target in that case.
           'conditions': [
-            ['v8_use_snapshot=="true"', {
+ ['v8_use_snapshot=="true" and v8_use_external_startup_data==0', {
               'dependencies': ['../../tools/gyp/v8.gyp:v8_snapshot'],
-            },
-            {
-              'dependencies': [
-                '../../tools/gyp/v8.gyp:v8_nosnapshot',
-              ],
+            }],
+ ['v8_use_snapshot=="true" and v8_use_external_startup_data==1', { + 'dependencies': ['../../tools/gyp/v8.gyp:v8_external_snapshot'],
+            }],
+            ['v8_use_snapshot!="true"', {
+              'dependencies': ['../../tools/gyp/v8.gyp:v8_nosnapshot'],
             }],
           ],
         }, {
Index: test/unittests/unittests.gyp
diff --git a/test/unittests/unittests.gyp b/test/unittests/unittests.gyp
index 8a323f121403478afbbe6fe54b663b4c02b7b8a1..35c1020ce3d18ef9428008a480eeec512e467b9d 100644
--- a/test/unittests/unittests.gyp
+++ b/test/unittests/unittests.gyp
@@ -106,13 +106,14 @@
# compiler-unittests can't be built against a shared library, so we
           # need to depend on the underlying static target in that case.
           'conditions': [
-            ['v8_use_snapshot=="true"', {
+ ['v8_use_snapshot=="true" and v8_use_external_startup_data==0', {
               'dependencies': ['../../tools/gyp/v8.gyp:v8_snapshot'],
-            },
-            {
-              'dependencies': [
-                '../../tools/gyp/v8.gyp:v8_nosnapshot',
-              ],
+            }],
+ ['v8_use_snapshot=="true" and v8_use_external_startup_data==1', { + 'dependencies': ['../../tools/gyp/v8.gyp:v8_external_snapshot'],
+            }],
+            ['v8_use_snapshot!="true"', {
+              'dependencies': ['../../tools/gyp/v8.gyp:v8_nosnapshot'],
             }],
           ],
         }, {


--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to