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

Side by Side Diff: build/config/android/rules.gni

Issue 1922613004: jinja_template should forward "deps" to the underlying action. (Closed) Base URL: https://chromium.googlesource.com/chromium/src@master
Patch Set: Created 4 years, 8 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 | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright 2014 The Chromium Authors. All rights reserved. 1 # Copyright 2014 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("//base/android/linker/config.gni") 5 import("//base/android/linker/config.gni")
6 import("//build/config/android/config.gni") 6 import("//build/config/android/config.gni")
7 import("//build/config/android/internal_rules.gni") 7 import("//build/config/android/internal_rules.gni")
8 import("//build/config/sanitizers/sanitizers.gni") 8 import("//build/config/sanitizers/sanitizers.gni")
9 import("//build/toolchain/toolchain.gni") 9 import("//build/toolchain/toolchain.gni")
10 import("//third_party/android_platform/config.gni") 10 import("//third_party/android_platform/config.gni")
(...skipping 370 matching lines...) Expand 10 before | Expand all | Expand 10 after
381 # output = "$target_gen_dir/AndroidManifest.xml" 381 # output = "$target_gen_dir/AndroidManifest.xml"
382 # } 382 # }
383 template("jinja_template") { 383 template("jinja_template") {
384 set_sources_assignment_filter([]) 384 set_sources_assignment_filter([])
385 forward_variables_from(invoker, [ "testonly" ]) 385 forward_variables_from(invoker, [ "testonly" ])
386 386
387 assert(defined(invoker.input)) 387 assert(defined(invoker.input))
388 assert(defined(invoker.output)) 388 assert(defined(invoker.output))
389 389
390 action(target_name) { 390 action(target_name) {
391 forward_variables_from(invoker, [ "visibility" ]) 391 forward_variables_from(invoker,
392 [
393 "visibility",
394 "deps",
395 ])
392 396
393 sources = [ 397 sources = [
394 invoker.input, 398 invoker.input,
395 ] 399 ]
396 script = "//build/android/gyp/jinja_template.py" 400 script = "//build/android/gyp/jinja_template.py"
397 depfile = "$target_gen_dir/$target_name.d" 401 depfile = "$target_gen_dir/$target_name.d"
398 402
399 outputs = [ 403 outputs = [
400 depfile, 404 depfile,
401 invoker.output, 405 invoker.output,
(...skipping 2000 matching lines...) Expand 10 before | Expand all | Expand 10 after
2402 "--target", 2406 "--target",
2403 rebase_path(invoker.target, root_build_dir), 2407 rebase_path(invoker.target, root_build_dir),
2404 "--output-directory", 2408 "--output-directory",
2405 rebase_path(root_out_dir, root_build_dir), 2409 rebase_path(root_out_dir, root_build_dir),
2406 ] 2410 ]
2407 if (defined(invoker.flag_name)) { 2411 if (defined(invoker.flag_name)) {
2408 args += [ "--flag-name=${invoker.flag_name}" ] 2412 args += [ "--flag-name=${invoker.flag_name}" ]
2409 } 2413 }
2410 } 2414 }
2411 } 2415 }
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698