Changeset 139895 in webkit


Ignore:
Timestamp:
Jan 16, 2013, 10:37:48 AM (12 years ago)
Author:
commit-queue@webkit.org
Message:

[Chromium] Remove hardcoded chromium_*.jar in gyp files
​https://bugs.webkit.org/show_bug.cgi?id=104049

Patch by Chris Hopman <​cjhopman@chromium.org> on 2013-01-16
Reviewed by Eric Seidel.

Targets with dependencies (direct/indirect) on a java target receive
the chromium_*.jar paths in the variable input_jars_paths. Targets
should use that rather than hardcoding where they think the jar will
be. These can be passed directly to ant as INPUT_JARS_PATHS rather
than as --jars to generate_native_test.py.

Source/WebKit/chromium:

  • WebKitUnitTests.gyp:

Tools:

  • DumpRenderTree/DumpRenderTree.gyp/DumpRenderTree.gyp:
  • TestWebKitAPI/TestWebKitAPI.gyp/TestWebKitAPI.gyp:
Location:
trunk
Files:
5 edited

Legend:

Unmodified
Added
Removed

  • trunk/Source/WebKit/chromium/ChangeLog

    r139856 r139895  
     12013-01-16  Chris Hopman  <cjhopman@chromium.org>
     2
     3        [Chromium] Remove hardcoded chromium_*.jar in gyp files
     4        https://bugs.webkit.org/show_bug.cgi?id=104049
     5
     6        Reviewed by Eric Seidel.
     7
     8        Targets with dependencies (direct/indirect) on a java target receive
     9        the chromium_*.jar paths in the variable input_jars_paths. Targets
     10        should use that rather than hardcoding where they think the jar will
     11        be. These can be passed directly to ant as INPUT_JARS_PATHS rather
     12        than as --jars to generate_native_test.py.
     13
     14        * WebKitUnitTests.gyp:
     15
    1162013-01-16  Dominic Cooney   <dominicc@chromium.org>
    217

  • trunk/Source/WebKit/chromium/WebKitUnitTests.gyp

    r138044 r139895  
    151151                'variables': {
    152152                    'input_shlib_path': '<(SHARED_LIB_DIR)/<(SHARED_LIB_PREFIX)webkit_unit_tests<(SHARED_LIB_SUFFIX)',
    153                     'input_jars_paths': [
    154                         '<(PRODUCT_DIR)/lib.java/chromium_base.jar',
    155                         '<(PRODUCT_DIR)/lib.java/chromium_net.jar',
    156                     ],
    157153                    'conditions': [
    158154                        ['inside_chromium_build==1', {
    … …  
    182178                        '--native_library',
    183179                        '<(input_shlib_path)',
    184                         '--jars',
    185                         '">@(input_jars_paths)"',
    186180                        '--output',
    187181                        '<(PRODUCT_DIR)/webkit_unit_tests_apk',
    … …  
    205199                        '--ant-args',
    206200                        '-DCHROMIUM_SRC=<(ant_build_to_chromium_src)',
     201                        '--ant-args',
     202                        '-DINPUT_JARS_PATHS=>@(input_jars_paths)',
    207203                        '--app_abi',
    208204                        '<(android_app_abi)',

  • trunk/Tools/ChangeLog

    r139875 r139895  
     12013-01-16  Chris Hopman  <cjhopman@chromium.org>
     2
     3        [Chromium] Remove hardcoded chromium_*.jar in gyp files
     4        https://bugs.webkit.org/show_bug.cgi?id=104049
     5
     6        Reviewed by Eric Seidel.
     7
     8        Targets with dependencies (direct/indirect) on a java target receive
     9        the chromium_*.jar paths in the variable input_jars_paths. Targets
     10        should use that rather than hardcoding where they think the jar will
     11        be. These can be passed directly to ant as INPUT_JARS_PATHS rather
     12        than as --jars to generate_native_test.py.
     13
     14        * DumpRenderTree/DumpRenderTree.gyp/DumpRenderTree.gyp:
     15        * TestWebKitAPI/TestWebKitAPI.gyp/TestWebKitAPI.gyp:
     16
    1172013-01-16  Dan Carney  <dcarney@google.com>
    218

  • trunk/Tools/DumpRenderTree/DumpRenderTree.gyp/DumpRenderTree.gyp

    r138044 r139895  
    473473                'variables': {
    474474                    'input_shlib_path': '<(SHARED_LIB_DIR)/<(SHARED_LIB_PREFIX)DumpRenderTree<(SHARED_LIB_SUFFIX)',
    475                     'input_jars_paths': [
    476                         '<(PRODUCT_DIR)/lib.java/chromium_base.jar',
    477                         '<(PRODUCT_DIR)/lib.java/chromium_net.jar',
    478                         '<(PRODUCT_DIR)/lib.java/chromium_media.jar',
    479                     ],
    480475                    'conditions': [
    481476                        ['inside_chromium_build==1', {
    … …  
    496491                        '<(chromium_src_dir)/testing/android/generate_native_test.py',
    497492                        '<(input_shlib_path)',
    498                         '<@(input_jars_paths)',
     493                        '>@(input_jars_paths)',
    499494                    ],
    500495                    'outputs': [
    … …  
    505500                        '--native_library',
    506501                        '<(input_shlib_path)',
    507                         '--jars',
    508                         '"<@(input_jars_paths)"',
    509502                        '--output',
    510503                        '<(PRODUCT_DIR)/DumpRenderTree_apk',
    … …  
    528521                        '--ant-args',
    529522                        '-DCHROMIUM_SRC=<(ant_build_to_chromium_src)',
     523                        '--ant-args',
     524                        '-DINPUT_JARS_PATHS=>@(input_jars_paths)',
    530525                        '--app_abi',
    531526                        '<(android_app_abi)',

  • trunk/Tools/TestWebKitAPI/TestWebKitAPI.gyp/TestWebKitAPI.gyp

    r138044 r139895  
    107107                'variables': {
    108108                    'input_shlib_path': '<(SHARED_LIB_DIR)/<(SHARED_LIB_PREFIX)TestWebKitAPI<(SHARED_LIB_SUFFIX)',
    109                     'input_jars_paths': [
    110                         '<(PRODUCT_DIR)/lib.java/chromium_base.jar',
    111                     ],
    112109                    'conditions': [
    113110                        ['inside_chromium_build==1', {
    … …  
    128125                        '<(chromium_src_dir)/testing/android/generate_native_test.py',
    129126                        '<(input_shlib_path)',
    130                         '<@(input_jars_paths)',
     127                        '>@(input_jars_paths)',
    131128                    ],
    132129                    'outputs': [
    … …  
    137134                        '--native_library',
    138135                        '<(input_shlib_path)',
    139                         '--jars',
    140                         '"<@(input_jars_paths)"',
    141136                        '--output',
    142137                        '<(PRODUCT_DIR)/TestWebKitAPI_apk',
    … …  
    160155                        '--ant-args',
    161156                        '-DCHROMIUM_SRC=<(ant_build_to_chromium_src)',
     157                        '--ant-args',
     158                        '-DINPUT_JARS_PATHS=>@(input_jars_paths)',
    162159                        '--app_abi',
    163160                        '<(android_app_abi)',
Note: See TracChangeset for help on using the changeset viewer.