Chromium Code Reviews| Index: chrome/app_installer/app_installer.gypi |
| diff --git a/chrome/app_installer/app_installer.gypi b/chrome/app_installer/app_installer.gypi |
| index f83df523ba6cf1e253922af42c24054a2e1468b1..dd063548e178cf1cc181378c221694cbf360ef7e 100644 |
| --- a/chrome/app_installer/app_installer.gypi |
| +++ b/chrome/app_installer/app_installer.gypi |
| @@ -6,11 +6,9 @@ |
| 'conditions': [ |
| ['OS=="win"', { |
| 'targets': [ |
| - # TODO(jackhou): Add a version resource (using |
| - # version_resource_rules.gypi). |
| { |
| - 'target_name': 'app_installer', |
| - 'type': 'executable', |
| + 'target_name': 'app_installer_util', |
| + 'type': 'static_library', |
| 'dependencies': [ |
| 'installer_util', |
| 'installer_util_strings', |
| @@ -24,6 +22,24 @@ |
| '<(INTERMEDIATE_DIR)', |
| ], |
| 'sources': [ |
| + 'win/app_installer_util.cc', |
| + 'win/app_installer_util.h', |
| + ], |
| + }, |
| + # TODO(jackhou): Add a version resource (using |
| + # version_resource_rules.gypi). |
| + { |
| + 'target_name': 'app_installer', |
| + 'type': 'executable', |
| + 'dependencies': [ |
| + 'app_installer_util', |
| + '../base/base.gyp:base', |
| + ], |
| + 'include_dirs': [ |
| + '..', |
| + '<(INTERMEDIATE_DIR)', |
| + ], |
| + 'sources': [ |
| 'win/app_installer_main.cc', |
| ], |
| 'msvs_settings': { |
| @@ -37,6 +53,30 @@ |
| }, |
| }, |
| }, |
| + { |
| + 'target_name': 'app_installer_unittests', |
|
grt (UTC plus 2)
2014/11/07 03:13:36
in the new world, i think you just need to add thi
jackhou1
2014/11/07 05:37:52
Hmm, adding it to chromium_trybot.json didn't caus
|
| + 'type': 'executable', |
| + 'dependencies': [ |
| + 'app_installer_util', |
| + '../base/base.gyp:base', |
| + '../base/base.gyp:run_all_unittests', |
| + '../testing/gtest.gyp:gtest', |
| + ], |
| + 'include_dirs': [ |
| + '..', |
| + '<(INTERMEDIATE_DIR)', |
| + ], |
| + 'sources': [ |
| + 'win/app_installer_util_unittest.cc', |
| + ], |
| + 'msvs_settings': { |
| + 'VCManifestTool': { |
| + 'AdditionalManifestFiles': [ |
| + 'app_installer/win/app_installer.exe.manifest', |
| + ], |
| + }, |
| + }, |
| + }, |
| ], |
| }], |
| ], |