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

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

Issue 1451223004: GN: Add depfile for java_cpp_enum (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 1 month 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 | « build/android/gyp/java_cpp_enum.py ('k') | 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/toolchain/toolchain.gni") 8 import("//build/toolchain/toolchain.gni")
9 import("//third_party/android_platform/config.gni") 9 import("//third_party/android_platform/config.gni")
10 import("//tools/grit/grit_rule.gni") 10 import("//tools/grit/grit_rule.gni")
(...skipping 330 matching lines...) Expand 10 before | Expand all | Expand 10 after
341 341
342 assert(defined(invoker.sources)) 342 assert(defined(invoker.sources))
343 forward_variables_from(invoker, 343 forward_variables_from(invoker,
344 [ 344 [
345 "sources", 345 "sources",
346 "testonly", 346 "testonly",
347 "visibility", 347 "visibility",
348 ]) 348 ])
349 349
350 script = "//build/android/gyp/java_cpp_enum.py" 350 script = "//build/android/gyp/java_cpp_enum.py"
351 depfile = "$target_gen_dir/$target_name.d"
pkotwicz 2015/11/18 23:16:24 Nit: Shouldn't |depfile| be |_depfile| ?
pkotwicz 2015/11/18 23:17:50 Never mind, depfile is a parameter of action
351 352
352 _srcjar_path = "${target_gen_dir}/${target_name}.srcjar" 353 _srcjar_path = "${target_gen_dir}/${target_name}.srcjar"
353 _rebased_srcjar_path = rebase_path(_srcjar_path, root_build_dir) 354 _rebased_srcjar_path = rebase_path(_srcjar_path, root_build_dir)
354 _rebased_sources = rebase_path(invoker.sources, root_build_dir) 355 _rebased_sources = rebase_path(invoker.sources, root_build_dir)
355 356
356 args = [ "--srcjar=$_rebased_srcjar_path" ] + _rebased_sources 357 args = [
358 "--depfile",
359 rebase_path(depfile, root_build_dir),
360 "--srcjar=$_rebased_srcjar_path",
361 ] + _rebased_sources
357 outputs = [ 362 outputs = [
363 depfile,
358 _srcjar_path, 364 _srcjar_path,
359 ] 365 ]
360 } 366 }
361 } 367 }
362 368
363 # Declare a target for processing a Jinja template. 369 # Declare a target for processing a Jinja template.
364 # 370 #
365 # Variables 371 # Variables
366 # input: The template file to be processed. 372 # input: The template file to be processed.
367 # output: Where to save the result. 373 # output: Where to save the result.
(...skipping 1782 matching lines...) Expand 10 before | Expand all | Expand 10 after
2150 } 2156 }
2151 2157
2152 android_library(target_name) { 2158 android_library(target_name) {
2153 java_files = [] 2159 java_files = []
2154 srcjar_deps = [ ":${_template_name}__protoc_java" ] 2160 srcjar_deps = [ ":${_template_name}__protoc_java" ]
2155 deps = [ 2161 deps = [
2156 "//third_party/android_protobuf:protobuf_nano_javalib", 2162 "//third_party/android_protobuf:protobuf_nano_javalib",
2157 ] 2163 ]
2158 } 2164 }
2159 } 2165 }
OLDNEW
« no previous file with comments | « build/android/gyp/java_cpp_enum.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698