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

Side by Side Diff: mojo/shell/BUILD.gn

Issue 757803003: Make gn check //mojo/* pass on linux and android (Closed) Base URL: git@github.com:domokit/mojo.git@gn_check_services
Patch Set: PRESUBMIT.py Created 6 years 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 | « mojo/android/BUILD.gn ('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("//build/config/ui.gni") 5 import("//build/config/ui.gni")
6 import("//mojo/public/mojo.gni") 6 import("//mojo/public/mojo.gni")
7 import("//mojo/public/tools/bindings/mojom.gni") 7 import("//mojo/public/tools/bindings/mojom.gni")
8 8
9 # We don't support building in the component build since mojo apps are 9 # We don't support building in the component build since mojo apps are
10 # inherently components. 10 # inherently components.
(...skipping 46 matching lines...) Expand 10 before | Expand all | Expand 10 after
57 "dynamic_service_runner.h", 57 "dynamic_service_runner.h",
58 "in_process_dynamic_service_runner.cc", 58 "in_process_dynamic_service_runner.cc",
59 "in_process_dynamic_service_runner.h", 59 "in_process_dynamic_service_runner.h",
60 ] 60 ]
61 61
62 deps = [ 62 deps = [
63 "//base", 63 "//base",
64 "//mojo/gles2", 64 "//mojo/gles2",
65 "//mojo/public/cpp/system", 65 "//mojo/public/cpp/system",
66 ] 66 ]
67
68 # This target has to include the public thunk headers, which generally
69 # shouldn't be included without picking an implementation. We are providing
70 # the implementation but the thunk header target cannot declare that we are
71 # permitted to include it since it's in the public SDK and we are not.
72 # Suppress include checking so we can still check the rest of the targets in
73 # this file.
74 check_includes = false
67 } 75 }
68 76
69 source_set("lib") { 77 source_set("lib") {
70 sources = [ 78 sources = [
71 "app_child_process.cc", 79 "app_child_process.cc",
72 "app_child_process.h", 80 "app_child_process.h",
73 "app_child_process_host.cc", 81 "app_child_process_host.cc",
74 "app_child_process_host.h", 82 "app_child_process_host.h",
75 "child_process.cc", 83 "child_process.cc",
76 "child_process.h", 84 "child_process.h",
(...skipping 57 matching lines...) Expand 10 before | Expand all | Expand 10 after
134 "network_application_loader.cc", 142 "network_application_loader.cc",
135 "network_application_loader.h", 143 "network_application_loader.h",
136 ] 144 ]
137 145
138 deps += [ 146 deps += [
139 "//mojo/services/gles2", 147 "//mojo/services/gles2",
140 "//mojo/services/native_viewport:lib", 148 "//mojo/services/native_viewport:lib",
141 "//mojo/services/network:lib", 149 "//mojo/services/network:lib",
142 ] 150 ]
143 } 151 }
152
153 # This target includes some files behind #ifdef OS... guards. Since gn is not
154 # smart enough to understand preprocess includes, it does complains about
155 # these includes when not using the build files for that OS. Suppress checking
156 # so we can enable checking for the rest of the targets in this file.
157 # TODO: Might be better to split the files with OS-specific includes out to a
158 # separate source_set so we can leave checking on for the rest of the target.
159 check_includes = false
144 } 160 }
145 161
146 if (is_android) { 162 if (is_android) {
147 generate_jni("jni_headers") { 163 generate_jni("jni_headers") {
148 sources = [ 164 sources = [
149 "android/apk/src/org/chromium/mojo_shell_apk/MojoMain.java", 165 "android/apk/src/org/chromium/mojo_shell_apk/MojoMain.java",
150 ] 166 ]
151 jni_package = "mojo" 167 jni_package = "mojo"
152 } 168 }
153 169
(...skipping 179 matching lines...) Expand 10 before | Expand all | Expand 10 after
333 349
334 deps -= [ 350 deps -= [
335 ":lib", 351 ":lib",
336 ":external_application_registrar_connection", 352 ":external_application_registrar_connection",
337 ":external_application_registrar_bindings", 353 ":external_application_registrar_bindings",
338 "//mojo/shell/domain_socket", 354 "//mojo/shell/domain_socket",
339 "//mojo/shell/domain_socket:tests", 355 "//mojo/shell/domain_socket:tests",
340 ] 356 ]
341 } 357 }
342 } 358 }
OLDNEW
« no previous file with comments | « mojo/android/BUILD.gn ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698