2011-03-04 Adam Barth <abarth@webkit.org>
authorabarth@webkit.org <abarth@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 5 Mar 2011 00:11:19 +0000 (00:11 +0000)
committerabarth@webkit.org <abarth@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 5 Mar 2011 00:11:19 +0000 (00:11 +0000)
        Reviewed by Dimitri Glazkov.

        Use target_defaults to reduce boilerplate in GYP build system
        https://bugs.webkit.org/show_bug.cgi?id=55790

        This file contains GYP declarations that are shared by all the modules.
        Currently, we use this to set up the configurations, albeit primitively.

        * Source/gyp/common.gypi: Added.
2011-03-04  Adam Barth  <abarth@webkit.org>

        Reviewed by Dimitri Glazkov.

        Use target_defaults to reduce boilerplate in GYP build system
        https://bugs.webkit.org/show_bug.cgi?id=55790

        Instead of setting up the configuration in each target, just defer to
        target_defaults.  Also, removed a define that was redundant with the
        xcconfig.

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

        Reviewed by Dimitri Glazkov.

        Use target_defaults to reduce boilerplate in GYP build system
        https://bugs.webkit.org/show_bug.cgi?id=55790

        Instead of setting up the configuration in each target, just defer to
        target_defaults.  Also, removed a define that was redundant with the
        xcconfig.

        * gyp/JavaScriptGlue.gyp:

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

ChangeLog
Source/JavaScriptCore/ChangeLog
Source/JavaScriptCore/gyp/JavaScriptCore.gyp
Source/JavaScriptGlue/ChangeLog
Source/JavaScriptGlue/gyp/JavaScriptGlue.gyp
Source/gyp/common.gypi [new file with mode: 0644]

index b67a057..ae7bc61 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,15 @@
+2011-03-04  Adam Barth  <abarth@webkit.org>
+
+        Reviewed by Dimitri Glazkov.
+
+        Use target_defaults to reduce boilerplate in GYP build system
+        https://bugs.webkit.org/show_bug.cgi?id=55790
+
+        This file contains GYP declarations that are shared by all the modules.
+        Currently, we use this to set up the configurations, albeit primitively.
+
+        * Source/gyp/common.gypi: Added.
+
 2011-03-03  Kevin Ollivier  <kevino@theolliviers.com>
 
         [wx] Build fix. Ensure the ICU include dir comes before JavaScriptCore/wtf on Win
index 68e612b..75f41dd 100644 (file)
@@ -1,3 +1,16 @@
+2011-03-04  Adam Barth  <abarth@webkit.org>
+
+        Reviewed by Dimitri Glazkov.
+
+        Use target_defaults to reduce boilerplate in GYP build system
+        https://bugs.webkit.org/show_bug.cgi?id=55790
+
+        Instead of setting up the configuration in each target, just defer to
+        target_defaults.  Also, removed a define that was redundant with the
+        xcconfig.
+
+        * gyp/JavaScriptCore.gyp:
+
 2011-03-03  Gavin Barraclough  <barraclough@apple.com>
 
         Reviewed by Sam Weinig.
index 891bf99..4d647a5 100644 (file)
@@ -1,5 +1,6 @@
 {
   'includes': [
+    '../../gyp/common.gypi',
     '../JavaScriptCore.gypi',
   ],
   'xcode_config_file': '<(DEPTH)/JavaScriptCore/Configurations/DebugRelease.xcconfig',
         ['exclude', '.*BSTR.*$'],
         ['exclude', 'jsc.cpp$'],
       ],
-      'configurations': {
-        'Debug': {},
-        'Release': {},
-        'Production': {},
-      },
-      'default_configuration': 'Debug',
-      'defines': [
-        'WEBKIT_VERSION_MIN_REQUIRED=WEBKIT_VERSION_LATEST',
-      ],
       'postbuilds': [
         {
           'postbuild_name': 'Check For Global Initializers',
           'sh', 'generate-derived-sources.sh',
         ],
       }],
-      'configurations': {
-        'Debug': {},
-        'Release': {},
-        'Production': {},
-      },
-      'default_configuration': 'Debug',
     },
     {
       'target_name': 'Update Version',
            'sh', '<(DEPTH)/gyp/update-info-plist.sh', '<(DEPTH)/JavaScriptCore/Info.plist'
           ]
       }],
-      'configurations': {
-        'Debug': {},
-        'Release': {},
-        'Production': {},
-      },
-      'default_configuration': 'Debug',
     },
     {
       'target_name': 'minidom',
         '<@(minidom_files)',
         '$(SDKROOT)/System/Library/Frameworks/CoreFoundation.framework',
       ],
-      'configurations': {
-        'Debug': {},
-        'Release': {},
-        'Production': {},
-      },
-      'default_configuration': 'Debug',
     },
     {
       'target_name': 'testapi',
         '<@(testapi_files)',
         '$(SDKROOT)/System/Library/Frameworks/CoreFoundation.framework',
       ],
-      'configurations': {
-        'Debug': {},
-        'Release': {},
-        'Production': {},
-      },
-      'default_configuration': 'Debug',
     },
     {
       'target_name': 'jsc',
         '$(SDKROOT)/System/Library/Frameworks/CoreFoundation.framework',
         'libedit.dylib',
       ],
-      'configurations': {
-        'Debug': {},
-        'Release': {},
-        'Production': {},
-      },
-      'default_configuration': 'Debug',
     },
   ], # targets
 }
index 4f4f506..ed47e76 100644 (file)
@@ -1,3 +1,16 @@
+2011-03-04  Adam Barth  <abarth@webkit.org>
+
+        Reviewed by Dimitri Glazkov.
+
+        Use target_defaults to reduce boilerplate in GYP build system
+        https://bugs.webkit.org/show_bug.cgi?id=55790
+
+        Instead of setting up the configuration in each target, just defer to
+        target_defaults.  Also, removed a define that was redundant with the
+        xcconfig.
+
+        * gyp/JavaScriptGlue.gyp:
+
 2011-03-03  Geoffrey Garen  <ggaren@apple.com>
 
         Rolled out 80277 and 80280 because they caused event handler layout test
index 3044afb..c3bb711 100644 (file)
@@ -1,5 +1,6 @@
 {
   'includes': [
+    '../../gyp/common.gypi',
     'JavaScriptGlue.gypi',
   ],
   'xcode_config_file': '<(DEPTH)/JavaScriptGlue/Configurations/DebugRelease.xcconfig',
         '$(SDKROOT)/System/Library/Frameworks/AppKit.framework',
       ],
       'xcode_config_file': '../Configurations/JavaScriptGlue.xcconfig',
-      'configurations': {
-        'Debug': {},
-      },
-      'default_configuration': 'Debug',
-      'defines': [
-        'WEBKIT_VERSION_MIN_REQUIRED=WEBKIT_VERSION_LATEST',
-      ],
       'postbuilds': [
         {
           'postbuild_name': 'Check For Global Initializers',
diff --git a/Source/gyp/common.gypi b/Source/gyp/common.gypi
new file mode 100644 (file)
index 0000000..892009c
--- /dev/null
@@ -0,0 +1,10 @@
+{
+  'target_defaults': {
+    'configurations': {
+      'Debug': {},
+      'Release': {},
+      'Production': {},
+    },
+    'default_configuration': 'Debug',
+  },
+}