2011-03-01 Adam Barth <abarth@webkit.org>
authorabarth@webkit.org <abarth@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 2 Mar 2011 00:47:56 +0000 (00:47 +0000)
committerabarth@webkit.org <abarth@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 2 Mar 2011 00:47:56 +0000 (00:47 +0000)
        Reviewed by Dimitri Glazkov.

        Add pre- and post-build actions for JavaScriptCore GYP build
        https://bugs.webkit.org/show_bug.cgi?id=55507

        After this patch, we have all the steps for building the main
        JavaScriptCore framework except the "copy headers" step, which I'll do
        next.

        * gyp/JavaScriptCore.gyp:
2011-03-01  Adam Barth  <abarth@webkit.org>

        Reviewed by Dimitri Glazkov.

        Add pre- and post-build actions for JavaScriptCore GYP build
        https://bugs.webkit.org/show_bug.cgi?id=55507

        Remove the portion of this file that was split out into common.gypi.

        * gyp/JavaScriptGlue.gyp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@80065 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/JavaScriptCore/ChangeLog
Source/JavaScriptCore/gyp/JavaScriptCore.gyp
Source/JavaScriptGlue/ChangeLog
Source/JavaScriptGlue/gyp/JavaScriptGlue.gyp

index acca59f..2f6f6e3 100644 (file)
@@ -1,3 +1,16 @@
+2011-03-01  Adam Barth  <abarth@webkit.org>
+
+        Reviewed by Dimitri Glazkov.
+
+        Add pre- and post-build actions for JavaScriptCore GYP build
+        https://bugs.webkit.org/show_bug.cgi?id=55507
+
+        After this patch, we have all the steps for building the main
+        JavaScriptCore framework except the "copy headers" step, which I'll do
+        next.
+
+        * gyp/JavaScriptCore.gyp:
+
 2011-03-01  Geoffrey Garen  <ggaren@apple.com>
 
         Reviewed by Sam Weinig.
index 498b35b..8f2bf84 100644 (file)
@@ -7,6 +7,10 @@
     {
       'target_name': 'JavaScriptCore',
       'type': 'shared_library',
+      'dependencies': [
+        'Derived Sources',
+        'Update Version',
+      ],
       'include_dirs': [
         '<(DEPTH)', # Some paths in API include JavaScriptCore/
         '<(DEPTH)/JavaScriptCore',
         ['exclude', '.*BSTR.*$'],
         ['exclude', 'jsc.cpp$'],
       ],
-      'actions': [{
-        'action_name': 'generate_derived_sources',
-        'inputs': [],
-        'outputs': [],
-        'action': [
-          'sh', 'generate-derived-sources.sh',
-        ],
-      }],
       'configurations': {
         'Debug': {},
         'Relase': {},
       'defines': [
         'WEBKIT_VERSION_MIN_REQUIRED=WEBKIT_VERSION_LATEST',
       ],
+      'postbuilds': [
+        {
+          'postbuild_name': 'Check For Global Initializers',
+          'action': [
+            'sh', '<(DEPTH)/gyp/run-if-exists.sh', '<(DEPTH)/../Tools/Scripts/check-for-global-initializers'
+          ],
+        },
+        {
+          'postbuild_name': 'Check For Exit Time Destructors',
+          'action': [
+            'sh', '<(DEPTH)/gyp/run-if-exists.sh', '<(DEPTH)/../Tools/Scripts/check-for-exit-time-destructors'
+          ],
+        },
+        {
+          'postbuild_name': 'Check For Weak VTables and Externals',
+          'action': [
+            'sh', '<(DEPTH)/gyp/run-if-exists.sh', '<(DEPTH)/../Tools/Scripts/check-for-weak-vtables-and-externals'
+          ],
+        },
+      ],
       'conditions': [
         ['OS=="mac"', {
           'mac_bundle': 1,
         }],
       ],
     },
+    {
+      'target_name': 'Derived Sources',
+      'type': 'none',
+      'actions': [{
+        'action_name': 'generate_derived_sources',
+        'inputs': [],
+        'outputs': [],
+        'action': [
+          'sh', 'generate-derived-sources.sh',
+        ],
+      }],
+    },
+    {
+      'target_name': 'Update Version',
+      'type': 'none',
+      'actions': [{
+        'action_name': 'Update Info.plist with version information',
+        'inputs': [],
+         'outputs': [],
+         'action': [
+           'sh', '<(DEPTH)/gyp/update-info-plist.sh', '<(DEPTH)/JavaScriptCore/Info.plist'
+          ]
+      }],
+    },
   ], # targets
 }
index cd5a2c5..2340a3b 100644 (file)
@@ -1,3 +1,14 @@
+2011-03-01  Adam Barth  <abarth@webkit.org>
+
+        Reviewed by Dimitri Glazkov.
+
+        Add pre- and post-build actions for JavaScriptCore GYP build
+        https://bugs.webkit.org/show_bug.cgi?id=55507
+
+        Remove the portion of this file that was split out into common.gypi.
+
+        * gyp/JavaScriptGlue.gyp:
+
 2011-02-28  Gavin Barraclough  <barraclough@apple.com>
 
         Reviewed by Sam Weinig & Darin Adler.
index 2a1573c..3044afb 100644 (file)
     {
       'target_name': 'Update Version',
       'type': 'none',
-      'actions': [
-        {
-          'action_name': 'Update Info.plist with version information',
-          'inputs': [],
-          'outputs': [],
-          'action': [
-            'sh', '<(DEPTH)/gyp/update-info-plist.sh', '<(DEPTH)/JavaScriptGlue/Info.plist'
-          ],
-        },
-      ], # actions
+      'actions': [{
+        'action_name': 'Update Info.plist with version information',
+        'inputs': [],
+         'outputs': [],
+         'action': [
+           'sh', '<(DEPTH)/gyp/update-info-plist.sh', '<(DEPTH)/JavaScriptGlue/Info.plist'
+          ]
+      }],
     },
   ], # targets
 }