Chromium Code Reviews| Index: build/android/devil_chromium.py |
| diff --git a/build/android/devil_chromium.py b/build/android/devil_chromium.py |
| index 8472f659bdcfd1881f9bbff155f524236cbeec8d..2a258ff7dcade26fbe91d6283332e54f0b79f208 100644 |
| --- a/build/android/devil_chromium.py |
| +++ b/build/android/devil_chromium.py |
| @@ -14,90 +14,6 @@ if host_paths.DEVIL_PATH not in sys.path: |
| from devil import devil_env |
| -_DEVIL_CONFIG = os.path.abspath( |
| - os.path.join(os.path.dirname(__file__), 'devil_chromium.json')) |
| - |
| -_DEVIL_BUILD_PRODUCT_DEPS = { |
| - 'forwarder_device': [ |
| - { |
| - 'platform': 'android', |
| - 'arch': 'armeabi-v7a', |
| - 'name': 'forwarder_dist', |
| - }, |
| - { |
| - 'platform': 'android', |
| - 'arch': 'arm64-v8a', |
| - 'name': 'forwarder_dist', |
| - }, |
| - { |
| - 'platform': 'android', |
| - 'arch': 'mips', |
| - 'name': 'forwarder_dist', |
| - }, |
| - { |
| - 'platform': 'android', |
| - 'arch': 'mips64', |
| - 'name': 'forwarder_dist', |
| - }, |
| - { |
| - 'platform': 'android', |
| - 'arch': 'x86', |
| - 'name': 'forwarder_dist', |
| - }, |
| - { |
| - 'platform': 'android', |
| - 'arch': 'x86_64', |
| - 'name': 'forwarder_dist', |
| - }, |
| - ], |
| - 'forwarder_host': [ |
| - { |
| - 'platform': 'linux2', |
| - 'arch': 'x86_64', |
| - 'name': 'host_forwarder', |
| - }, |
| - ], |
| - 'md5sum_device': [ |
| - { |
| - 'platform': 'android', |
| - 'arch': 'armeabi-v7a', |
| - 'name': 'md5sum_dist', |
| - }, |
| - { |
| - 'platform': 'android', |
| - 'arch': 'arm64-v8a', |
| - 'name': 'md5sum_dist', |
| - }, |
| - { |
| - 'platform': 'android', |
| - 'arch': 'mips', |
| - 'name': 'md5sum_dist', |
| - }, |
| - { |
| - 'platform': 'android', |
| - 'arch': 'mips64', |
| - 'name': 'md5sum_dist', |
| - }, |
| - { |
| - 'platform': 'android', |
| - 'arch': 'x86', |
| - 'name': 'md5sum_dist', |
| - }, |
| - { |
| - 'platform': 'android', |
| - 'arch': 'x86_64', |
| - 'name': 'md5sum_dist', |
| - }, |
| - ], |
| - 'md5sum_host': [ |
| - { |
| - 'platform': 'linux2', |
| - 'arch': 'x86_64', |
| - 'name': 'md5sum_bin_host', |
| - }, |
| - ], |
| -} |
| - |
| def Initialize(output_directory=None, custom_deps=None): |
| """Initializes devil with chromium's binaries and third-party libraries. |
| @@ -125,29 +41,20 @@ def Initialize(output_directory=None, custom_deps=None): |
| ... |
| } |
| """ |
| + config_files = None |
| + if output_directory: |
| + generated_config_file = os.path.abspath(os.path.join( |
| + output_directory, 'gen', 'devil_chromium.json')) |
| + if os.path.exists(generated_config_file): |
| + config_files = [generated_config_file] |
|
jbudorick
2016/03/24 18:26:26
This should log a warning if generated_config_file
mikecase (-- gone --)
2016/03/28 20:47:01
Added logging
|
| - devil_dynamic_config = { |
| + custom_config = { |
| 'config_type': 'BaseConfig', |
| 'dependencies': {}, |
| } |
| - if output_directory: |
| - output_directory = os.path.abspath(output_directory) |
| - devil_dynamic_config['dependencies'] = { |
| - dep_name: { |
| - 'file_info': { |
| - '%s_%s' % (dep_config['platform'], dep_config['arch']): { |
| - 'local_paths': [ |
| - os.path.join(output_directory, dep_config['name']), |
| - ], |
| - } |
| - for dep_config in dep_configs |
| - } |
| - } |
| - for dep_name, dep_configs in _DEVIL_BUILD_PRODUCT_DEPS.iteritems() |
| - } |
| if custom_deps: |
| - devil_dynamic_config['dependencies'].update(custom_deps) |
| + custom_config['dependencies'].update(custom_deps) |
| devil_env.config.Initialize( |
| - configs=[devil_dynamic_config], config_files=[_DEVIL_CONFIG]) |
| + configs=[custom_config], config_files=config_files) |