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

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

Issue 2738743003: Making the AIDL GN target support multiple include directories. (Closed)
Patch Set: Making the AIDL GN tag support multiple include directories. Created 3 years, 9 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 | « base/test/BUILD.gn ('k') | chrome/android/webapk/libs/runtime_library/BUILD.gn » ('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 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 # Do not add any imports to non-//build directories here. 5 # Do not add any imports to non-//build directories here.
6 # Some projects (e.g. V8) do not have non-build directories DEPS'ed in. 6 # Some projects (e.g. V8) do not have non-build directories DEPS'ed in.
7 import("//build/config/android/config.gni") 7 import("//build/config/android/config.gni")
8 import("//build/config/android/internal_rules.gni") 8 import("//build/config/android/internal_rules.gni")
9 import("//build/config/compiler/compiler.gni") 9 import("//build/config/compiler/compiler.gni")
10 import("//build/config/dcheck_always_on.gni") 10 import("//build/config/dcheck_always_on.gni")
(...skipping 2602 matching lines...) Expand 10 before | Expand all | Expand 10 after
2613 rebased_imports = rebase_path(imports, root_build_dir) 2613 rebased_imports = rebase_path(imports, root_build_dir)
2614 args = [ 2614 args = [
2615 "--depfile", 2615 "--depfile",
2616 rebase_path(depfile, root_build_dir), 2616 rebase_path(depfile, root_build_dir),
2617 "--aidl-path", 2617 "--aidl-path",
2618 rebase_path(aidl_path, root_build_dir), 2618 rebase_path(aidl_path, root_build_dir),
2619 "--imports=$rebased_imports", 2619 "--imports=$rebased_imports",
2620 "--srcjar", 2620 "--srcjar",
2621 rebase_path(srcjar_path, root_build_dir), 2621 rebase_path(srcjar_path, root_build_dir),
2622 ] 2622 ]
2623 if (defined(invoker.import_include) && invoker.import_include != "") { 2623 if (defined(invoker.import_include) && invoker.import_include != []) {
2624 # TODO(cjhopman): aidl supports creating a depfile. We should be able to 2624 # TODO(cjhopman): aidl supports creating a depfile. We should be able to
2625 # switch to constructing a depfile for the overall action from that 2625 # switch to constructing a depfile for the overall action from that
2626 # instead of having all the .java files in the include paths as inputs. 2626 # instead of having all the .java files in the include paths as inputs.
2627 rebased_import_includes = 2627 rebased_import_paths = []
2628 rebase_path([ invoker.import_include ], root_build_dir) 2628 foreach(import_path, invoker.import_include) {
2629 args += [ "--includes=$rebased_import_includes" ] 2629 _rebased_import_path = []
2630 2630 _rebased_import_path += rebase_path([ import_path ], root_build_dir)
2631 _java_files_build_rel = 2631 rebased_import_paths += _rebased_import_path
2632 exec_script("//build/android/gyp/find.py", 2632 _java_files_build_rel = []
2633 rebase_path([ invoker.import_include ], root_build_dir), 2633 _java_files_build_rel = exec_script("//build/android/gyp/find.py",
2634 "list lines") 2634 _rebased_import_path,
2635 _java_files = rebase_path(_java_files_build_rel, ".", root_build_dir) 2635 "list lines")
2636 inputs += _java_files 2636 inputs += rebase_path(_java_files_build_rel, ".", root_build_dir)
2637 }
2638 args += [ "--includes=$rebased_import_paths" ]
2637 } 2639 }
2638 args += rebase_path(sources, root_build_dir) 2640 args += rebase_path(sources, root_build_dir)
2639 } 2641 }
2640 } 2642 }
2641 2643
2642 # Compile a protocol buffer to java. 2644 # Compile a protocol buffer to java.
2643 # 2645 #
2644 # This generates java files from protocol buffers and creates an Android libra ry 2646 # This generates java files from protocol buffers and creates an Android libra ry
2645 # containing the classes. 2647 # containing the classes.
2646 # 2648 #
(...skipping 244 matching lines...) Expand 10 before | Expand all | Expand 10 after
2891 # because in practice they seem to contain classes required to be in the 2893 # because in practice they seem to contain classes required to be in the
2892 # classpath. 2894 # classpath.
2893 deps += _subjar_targets 2895 deps += _subjar_targets
2894 } 2896 }
2895 if (defined(_res_target_name)) { 2897 if (defined(_res_target_name)) {
2896 deps += [ ":$_res_target_name" ] 2898 deps += [ ":$_res_target_name" ]
2897 } 2899 }
2898 } 2900 }
2899 } 2901 }
2900 } 2902 }
OLDNEW
« no previous file with comments | « base/test/BUILD.gn ('k') | chrome/android/webapk/libs/runtime_library/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698