Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(324)

Side by Side Diff: chrome/installer/mini_installer/BUILD.gn

Issue 2084093003: Add input dependencies on installer test scripts (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | chrome/test/mini_installer/mini_installer_test.gni » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright 2015 The Chromium Authors. All rights reserved. 1 # Copyright 2015 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 import("//build/config/features.gni") 5 import("//build/config/features.gni")
6 import("//build/config/ui.gni") 6 import("//build/config/ui.gni")
7 import("//build_overrides/v8.gni") 7 import("//build_overrides/v8.gni")
8 import("//chrome/test/mini_installer/mini_installer_test.gni")
8 import("//chrome/version.gni") 9 import("//chrome/version.gni")
9 import("//third_party/icu/config.gni") 10 import("//third_party/icu/config.gni")
10 import("//ui/base/ui_features.gni") 11 import("//ui/base/ui_features.gni")
11 12
12 config("mini_installer_compiler_flags") { 13 config("mini_installer_compiler_flags") {
13 # Disable buffer security checking. 14 # Disable buffer security checking.
14 cflags = [ "/GS-" ] 15 cflags = [ "/GS-" ]
15 } 16 }
16 17
17 source_set("lib") { 18 source_set("lib") {
(...skipping 13 matching lines...) Expand all
31 "mini_string.cc", 32 "mini_string.cc",
32 "mini_string.h", 33 "mini_string.h",
33 "pe_resource.cc", 34 "pe_resource.cc",
34 "pe_resource.h", 35 "pe_resource.h",
35 "regkey.cc", 36 "regkey.cc",
36 "regkey.h", 37 "regkey.h",
37 ] 38 ]
38 39
39 configs += [ ":mini_installer_compiler_flags" ] 40 configs += [ ":mini_installer_compiler_flags" ]
40 41
41 deps = [ 42 # This input dependency on the mini installer test Python files will force a
42 #"test_installer_sentinel", TODO(GYP) bug 521052. 43 # recompilation when any of the Python files have changed. This will then
43 ] 44 # invalidate the build and trigger running the test on the bots even when
45 # no other Chrome files have changed. These have to be inputs rather than a
46 # dependency because a depedency won't actually trigger recompiles in
brucedawson 2016/06/21 19:59:26 depedency->dependency
47 # incremental build cases unless a source file actually references one of the
48 # dependency outputs.
49 inputs = mini_installer_test_files
44 } 50 }
45 51
46 process_version("version") { 52 process_version("version") {
47 template_file = "mini_installer_exe_version.rc.version" 53 template_file = "mini_installer_exe_version.rc.version"
48 output = "$target_gen_dir/mini_installer_version.rc" 54 output = "$target_gen_dir/mini_installer_version.rc"
49 } 55 }
50 56
51 source_set("unit_tests") { 57 source_set("unit_tests") {
52 testonly = true 58 testonly = true
53 59
(...skipping 163 matching lines...) Expand 10 before | Expand all | Expand 10 after
217 args = [ 223 args = [
218 "--out", 224 "--out",
219 "$target_name.exe", 225 "$target_name.exe",
220 ] 226 ]
221 deps = [ 227 deps = [
222 ":mini_installer", 228 ":mini_installer",
223 "//chrome/installer/test:alternate_version_generator", 229 "//chrome/installer/test:alternate_version_generator",
224 ] 230 ]
225 } 231 }
226 } 232 }
OLDNEW
« no previous file with comments | « no previous file | chrome/test/mini_installer/mini_installer_test.gni » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698