2011-03-21 Adam Barth <abarth@webkit.org>
authorabarth@webkit.org <abarth@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 22 Mar 2011 05:39:44 +0000 (05:39 +0000)
committerabarth@webkit.org <abarth@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 22 Mar 2011 05:39:44 +0000 (05:39 +0000)
        Reviewed by Eric Seidel.

        GYP build should not have include paths that point within the source tree
        https://bugs.webkit.org/show_bug.cgi?id=56788

        Turns out we don't need these include paths anymore now that we have
        header maps working properly.

        * gyp/JavaScriptCore.gyp:
            - Also, remove jsc.cpp from the excluded list because it's not part
              of the jsc_files variable instead of the javascriptcore_files
              variable.
2011-03-21  Adam Barth  <abarth@webkit.org>

        Reviewed by Eric Seidel.

        GYP build should not have include paths that point within the source tree
        https://bugs.webkit.org/show_bug.cgi?id=56788

        Turns out we don't need these include paths anymore now that we have
        header maps working properly.

        * gyp/JavaScriptGlue.gyp:

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

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

index bc79d03c3e96a980ac4dc43ba71bc3e61f5d6aa9..57353e8857b8c6eb7c68105a6d03942dda81ee25 100644 (file)
@@ -1,3 +1,18 @@
+2011-03-21  Adam Barth  <abarth@webkit.org>
+
+        Reviewed by Eric Seidel.
+
+        GYP build should not have include paths that point within the source tree
+        https://bugs.webkit.org/show_bug.cgi?id=56788
+
+        Turns out we don't need these include paths anymore now that we have
+        header maps working properly.
+
+        * gyp/JavaScriptCore.gyp:
+            - Also, remove jsc.cpp from the excluded list because it's not part
+              of the jsc_files variable instead of the javascriptcore_files
+              variable.
+
 2011-03-21  Adam Barth  <abarth@webkit.org>
 
         Reviewed by Eric Seidel.
index 44f0ff3185eed22c2aa169713fc70adb85544ad5..358bf272ed55880da79fdbb8457ca506435157d2 100644 (file)
@@ -7,24 +7,8 @@
   'variables': {
     # FIXME: We should use a header map instead of listing these explicitly.
     'javascriptcore_include_dirs': [
-      '<(DEPTH)', # Some paths in API include JavaScriptCore/
       '<(project_dir)',
-      '<(project_dir)/ForwardingHeaders',
-      '<(project_dir)/API',
-      '<(project_dir)/assembler',
-      '<(project_dir)/collector/handles',
-      '<(project_dir)/bytecode',
-      '<(project_dir)/bytecompiler',
-      '<(project_dir)/debugger',
       '<(project_dir)/icu',
-      '<(project_dir)/interpreter',
-      '<(project_dir)/jit',
-      '<(project_dir)/parser',
-      '<(project_dir)/profiler',
-      '<(project_dir)/runtime',
-      '<(project_dir)/wtf',
-      '<(project_dir)/wtf/unicode',
-      '<(SHARED_INTERMEDIATE_DIR)',
     ],
     'derived_source_files': [
       '<(SHARED_INTERMEDIATE_DIR)/ArrayPrototype.lut.h',
         ['exclude', '(Default|Gtk|Chromium|None|Qt|Win|Wx|Symbian)\\.(cpp|mm|h)$'],
         ['exclude', 'GCActivityCallback\.cpp'],
         ['exclude', '.*BSTR.*$'],
-        ['exclude', 'jsc.cpp$'],
       ],
       'postbuilds': [
         {
index 554ec29137a77d37973ee3dd603c31318a57f0d3..13cb9c1d9dda926ba85cdf66e98b46d7baa04019 100644 (file)
@@ -1,3 +1,15 @@
+2011-03-21  Adam Barth  <abarth@webkit.org>
+
+        Reviewed by Eric Seidel.
+
+        GYP build should not have include paths that point within the source tree
+        https://bugs.webkit.org/show_bug.cgi?id=56788
+
+        Turns out we don't need these include paths anymore now that we have
+        header maps working properly.
+
+        * gyp/JavaScriptGlue.gyp:
+
 2011-03-20  Bill Budge  <bbudge@chromium.org>
 
         Reviewed by Adam Barth.
index c3bb71198ab7982b70220a06b2e77995e731cf24..825fd860f1293092983ee16a38c319cc0221d355 100644 (file)
@@ -12,7 +12,6 @@
         'Update Version'
       ],
       'include_dirs': [
-        '<(DEPTH)/JavaScriptGlue',
         '<(DEPTH)/JavaScriptGlue/ForwardingHeaders',
         '<(DEPTH)/JavaScriptGlue/icu',
         '<(PRODUCT_DIR)/include',