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

Side by Side Diff: mojo/public/tools/bindings/mojom.gni

Issue 2328873004: mojom GNI: Implicit variant inheritance for deps (Closed)
Patch Set: Created 4 years, 3 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 mojom_generator_root = "//mojo/public/tools/bindings" 5 mojom_generator_root = "//mojo/public/tools/bindings"
6 mojom_generator_script = "$mojom_generator_root/mojom_bindings_generator.py" 6 mojom_generator_script = "$mojom_generator_root/mojom_bindings_generator.py"
7 mojom_generator_sources = [ 7 mojom_generator_sources = [
8 "$mojom_generator_root/generators/mojom_cpp_generator.py", 8 "$mojom_generator_root/generators/mojom_cpp_generator.py",
9 "$mojom_generator_root/generators/mojom_js_generator.py", 9 "$mojom_generator_root/generators/mojom_js_generator.py",
10 "$mojom_generator_root/generators/mojom_java_generator.py", 10 "$mojom_generator_root/generators/mojom_java_generator.py",
(...skipping 384 matching lines...) Expand 10 before | Expand all | Expand 10 after
395 } 395 }
396 if (defined(invoker.sources) && !defined(bindings_configuration.variant)) { 396 if (defined(invoker.sources) && !defined(bindings_configuration.variant)) {
397 data = process_file_template(enabled_sources, generator_js_outputs) 397 data = process_file_template(enabled_sources, generator_js_outputs)
398 } 398 }
399 399
400 public_deps = [ 400 public_deps = [
401 ":${cpp_sources_target_name}", 401 ":${cpp_sources_target_name}",
402 "//mojo/public/cpp/bindings", 402 "//mojo/public/cpp/bindings",
403 ] 403 ]
404 if (defined(invoker.deps)) { 404 if (defined(invoker.deps)) {
405 public_deps += invoker.deps 405 foreach(dep, invoker.deps) {
406 public_deps +=
407 [ get_label_info(dep, "label_no_toolchain") + variant_suffix ]
408 }
406 } 409 }
407 if (defined(invoker.public_deps)) { 410 if (defined(invoker.public_deps)) {
408 public_deps += invoker.public_deps 411 foreach(dep, invoker.public_deps) {
412 public_deps +=
413 [ get_label_info(dep, "label_no_toolchain") + variant_suffix ]
414 }
409 } 415 }
410 416
411 deps = [] 417 deps = []
412 if (defined(invoker.sources)) { 418 if (defined(invoker.sources)) {
413 public_deps += [ ":$generator_target_name" ] 419 public_deps += [ ":$generator_target_name" ]
414 } 420 }
415 } 421 }
416 422
417 # The generated C++ source files. The main reason to introduce this target 423 # The generated C++ source files. The main reason to introduce this target
418 # is so that mojo/public/cpp/bindings can depend on mojom interfaces without 424 # is so that mojo/public/cpp/bindings can depend on mojom interfaces without
(...skipping 101 matching lines...) Expand 10 before | Expand all | Expand 10 after
520 full_name = get_label_info(d, "label_no_toolchain") 526 full_name = get_label_info(d, "label_no_toolchain")
521 deps += [ "${full_name}_java" ] 527 deps += [ "${full_name}_java" ]
522 } 528 }
523 529
524 srcjar_deps = [ ":$java_srcjar_target_name" ] 530 srcjar_deps = [ ":$java_srcjar_target_name" ]
525 run_findbugs_override = false 531 run_findbugs_override = false
526 } 532 }
527 } 533 }
528 } 534 }
529 } 535 }
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