Changeset 81647 in webkit


Ignore:
Timestamp:
Mar 21, 2011 10:39:44 PM (13 years ago)
Author:
abarth@webkit.org
Message:

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.

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:
Location:
trunk/Source
Files:
4 edited

Legend:

Unmodified
Added
Removed

  • trunk/Source/JavaScriptCore/ChangeLog

    r81624 r81647  
     12011-03-21  Adam Barth  <abarth@webkit.org>
     2
     3        Reviewed by Eric Seidel.
     4
     5        GYP build should not have include paths that point within the source tree
     6        https://bugs.webkit.org/show_bug.cgi?id=56788
     7
     8        Turns out we don't need these include paths anymore now that we have
     9        header maps working properly.
     10
     11        * gyp/JavaScriptCore.gyp:
     12            - Also, remove jsc.cpp from the excluded list because it's not part
     13              of the jsc_files variable instead of the javascriptcore_files
     14              variable.
     15
    1162011-03-21  Adam Barth  <abarth@webkit.org>
    217

  • trunk/Source/JavaScriptCore/gyp/JavaScriptCore.gyp

    r81624 r81647  
    88    # FIXME: We should use a header map instead of listing these explicitly.
    99    'javascriptcore_include_dirs': [
    10       '<(DEPTH)', # Some paths in API include JavaScriptCore/
    1110      '<(project_dir)',
    12       '<(project_dir)/ForwardingHeaders',
    13       '<(project_dir)/API',
    14       '<(project_dir)/assembler',
    15       '<(project_dir)/collector/handles',
    16       '<(project_dir)/bytecode',
    17       '<(project_dir)/bytecompiler',
    18       '<(project_dir)/debugger',
    1911      '<(project_dir)/icu',
    20       '<(project_dir)/interpreter',
    21       '<(project_dir)/jit',
    22       '<(project_dir)/parser',
    23       '<(project_dir)/profiler',
    24       '<(project_dir)/runtime',
    25       '<(project_dir)/wtf',
    26       '<(project_dir)/wtf/unicode',
    27       '<(SHARED_INTERMEDIATE_DIR)',
    2812    ],
    2913    'derived_source_files': [
    … …  
    10993        ['exclude', 'GCActivityCallback\.cpp'],
    11094        ['exclude', '.*BSTR.*$'],
    111         ['exclude', 'jsc.cpp$'],
    11295      ],
    11396      'postbuilds': [

  • trunk/Source/JavaScriptGlue/ChangeLog

    r81567 r81647  
     12011-03-21  Adam Barth  <abarth@webkit.org>
     2
     3        Reviewed by Eric Seidel.
     4
     5        GYP build should not have include paths that point within the source tree
     6        https://bugs.webkit.org/show_bug.cgi?id=56788
     7
     8        Turns out we don't need these include paths anymore now that we have
     9        header maps working properly.
     10
     11        * gyp/JavaScriptGlue.gyp:
     12
    1132011-03-20  Bill Budge  <bbudge@chromium.org>
    214

  • trunk/Source/JavaScriptGlue/gyp/JavaScriptGlue.gyp

    r80387 r81647  
    1313      ],
    1414      'include_dirs': [
    15         '<(DEPTH)/JavaScriptGlue',
    1615        '<(DEPTH)/JavaScriptGlue/ForwardingHeaders',
    1716        '<(DEPTH)/JavaScriptGlue/icu',
Note: See TracChangeset for help on using the changeset viewer.