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

Side by Side Diff: components/nacl/broker/BUILD.gn

Issue 2652273002: Remove all service manifests from data_deps (Closed)
Patch Set: . Created 3 years, 10 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 | « components/leveldb/BUILD.gn ('k') | mash/browser/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 2016 The Chromium Authors. All rights reserved. 1 # Copyright 2016 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/features.gni") 5 import("//build/config/features.gni")
6 import("//build/config/compiler/compiler.gni") 6 import("//build/config/compiler/compiler.gni")
7 import("//services/service_manager/public/service_manifest.gni") 7 import("//services/service_manager/public/service_manifest.gni")
8 8
9 # This file builds nacl64.exe, which is a 64-bit x86 Windows executable 9 # This file builds nacl64.exe, which is a 64-bit x86 Windows executable
10 # used only in the 32-bit x86 Windows build. The :broker code runs both 10 # used only in the 32-bit x86 Windows build. The :broker code runs both
(...skipping 20 matching lines...) Expand all
31 "//mojo/edk/system", 31 "//mojo/edk/system",
32 "//sandbox", 32 "//sandbox",
33 "//services/service_manager/public/cpp", 33 "//services/service_manager/public/cpp",
34 ] 34 ]
35 35
36 if (current_cpu == target_cpu) { 36 if (current_cpu == target_cpu) {
37 deps += [ "//content/public/common" ] 37 deps += [ "//content/public/common" ]
38 } else { 38 } else {
39 deps += [ ":content_dummy" ] 39 deps += [ ":content_dummy" ]
40 } 40 }
41
42 data_deps = [
43 ":nacl_broker_manifest",
44 ]
45 } 41 }
46 42
47 # This exists just to make 'gn check' happy with :broker. It can't depend 43 # This exists just to make 'gn check' happy with :broker. It can't depend
48 # on //content/public/common or anything like that, because that would 44 # on //content/public/common or anything like that, because that would
49 # bring in lots more stuff that should not be in the nacl64.exe build. 45 # bring in lots more stuff that should not be in the nacl64.exe build.
50 source_set("content_dummy") { 46 source_set("content_dummy") {
51 check_includes = false 47 check_includes = false
52 sources = [ 48 sources = [
53 "//content/public/common/sandbox_init.h", 49 "//content/public/common/sandbox_init.h",
54 "//content/public/common/sandboxed_process_launcher_delegate.h", 50 "//content/public/common/sandboxed_process_launcher_delegate.h",
(...skipping 118 matching lines...) Expand 10 before | Expand all | Expand 10 after
173 "//ipc", 169 "//ipc",
174 "//third_party/kasko", 170 "//third_party/kasko",
175 ] 171 ]
176 } 172 }
177 } 173 }
178 174
179 service_manifest("nacl_broker_manifest") { 175 service_manifest("nacl_broker_manifest") {
180 name = "nacl_broker" 176 name = "nacl_broker"
181 source = "nacl_broker_manifest.json" 177 source = "nacl_broker_manifest.json"
182 } 178 }
OLDNEW
« no previous file with comments | « components/leveldb/BUILD.gn ('k') | mash/browser/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698