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

Side by Side Diff: components/cronet/ios/BUILD.gn

Issue 2115583002: GN Rule to build Cronet Dynamic Framework for iOS. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix the case. Created 4 years, 5 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 | components/cronet/tools/package_ios.py » ('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 2015 The Chromium Authors. All rights reserved. 1 # Copyright 2015 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/buildflag_header.gni") 5 import("//build/buildflag_header.gni")
6 import("//build/config/ios/rules.gni") 6 import("//build/config/ios/rules.gni")
7 import("//build/config/mac/symbols.gni")
8 import("//build/mac/tweak_info_plist.gni")
7 import("//build/util/version.gni") 9 import("//build/util/version.gni")
8 import("//chrome/version.gni") 10 import("//chrome/version.gni")
9 import("//testing/test.gni") 11 import("//testing/test.gni")
10 import("//url/features.gni") 12 import("//url/features.gni")
11 13
12 assert(!is_component_build, "Cronet requires static library build.") 14 assert(!is_component_build, "Cronet requires static library build.")
13 15
14 declare_args() { 16 declare_args() {
15 cronet_enable_data_reduction_proxy_support = false 17 cronet_enable_data_reduction_proxy_support = false
16 } 18 }
(...skipping 29 matching lines...) Expand all
46 "cronet_c_for_grpc.h", 48 "cronet_c_for_grpc.h",
47 "cronet_environment.cc", 49 "cronet_environment.cc",
48 "cronet_environment.h", 50 "cronet_environment.h",
49 ] 51 ]
50 52
51 if (!use_platform_icu_alternatives) { 53 if (!use_platform_icu_alternatives) {
52 deps += [ "//base:i18n" ] 54 deps += [ "//base:i18n" ]
53 } 55 }
54 } 56 }
55 57
58 # Tweak |info_plist| with current version and revision.
59 tweak_info_plist("tweak_cronet_plist") {
60 info_plist = "Info.plist"
61 args = []
62 }
63
64 # Compile plist into |output_name|.
65 ios_info_plist("compile_cronet_plist") {
66 executable_name = "Cronet"
67 output_name = "$target_gen_dir/Info.plist"
68 info_plist_target = ":tweak_cronet_plist"
69 }
70
71 # Bundle plist into Cronet.Framework.
72 bundle_data("cronet_info_plist") {
73 deps = [
74 ":compile_cronet_plist",
75 ]
76
77 sources = [
78 "$target_gen_dir/Info.plist",
79 ]
80 outputs = [
81 "{{bundle_root_dir}}/Info.plist",
82 ]
83 }
84
56 ios_framework_bundle("cronet_framework") { 85 ios_framework_bundle("cronet_framework") {
57 output_name = "Cronet" 86 output_name = "Cronet"
58 87
59 deps = [ 88 deps = [
89 ":compile_cronet_plist",
90 ":cronet_info_plist",
60 ":cronet_sources", 91 ":cronet_sources",
61 "//base", 92 "//base",
62 "//net:net", 93 "//net:net",
63 ] 94 ]
64 95
65 info_plist = "Info.plist"
66
67 libs = [ "UIKit.Framework" ] 96 libs = [ "UIKit.Framework" ]
68 97
69 public_headers = [ 98 public_headers = [
70 "Cronet.h", 99 "Cronet.h",
71 "cronet_c_for_grpc.h", 100 "cronet_c_for_grpc.h",
72 ] 101 ]
73 102
74 sources = [ 103 sources = [
75 "Cronet.h", 104 "Cronet.h",
76 "Cronet.mm", 105 "Cronet.mm",
77 ] 106 ]
107
108 configs -= [ "//build/config/compiler:default_symbols" ]
109 configs += [ "//build/config/compiler:symbols" ]
110 configs += [ "//build/config/mac:strip_all" ]
kapishnikov 2016/07/14 21:37:37 We should remove this line since we should not str
Robert Sesek 2016/07/14 21:40:12 The symbols are stripped after dsymutil is run. Th
kapishnikov 2016/07/14 21:45:50 Does the target generate the dsym bundle? If yes,
Robert Sesek 2016/07/14 21:49:34 If the gn arg |enable_dsyms=true|, then all linked
mef 2016/07/15 13:50:28 This works with gn arg enable_dsyms=true and enabl
78 } 111 }
79 112
80 bundle_data("cronet_test_bundle_data") { 113 bundle_data("cronet_test_bundle_data") {
81 testonly = true 114 testonly = true
82 sources = [ 115 sources = [
83 "//net/data/ssl/certificates/quic_test.example.com.crt", 116 "//net/data/ssl/certificates/quic_test.example.com.crt",
84 "//net/data/ssl/certificates/quic_test.example.com.key", 117 "//net/data/ssl/certificates/quic_test.example.com.key",
85 "//net/data/ssl/certificates/quic_test.example.com.key.pkcs8", 118 "//net/data/ssl/certificates/quic_test.example.com.key.pkcs8",
86 "//net/data/ssl/certificates/quic_test.example.com.key.sct", 119 "//net/data/ssl/certificates/quic_test.example.com.key.sct",
87 ] 120 ]
(...skipping 55 matching lines...) Expand 10 before | Expand all | Expand 10 after
143 rebase_path(_license_path, root_build_dir), 176 rebase_path(_license_path, root_build_dir),
144 "--gn", 177 "--gn",
145 ] 178 ]
146 } 179 }
147 180
148 copy("cronet_package_copy") { 181 copy("cronet_package_copy") {
149 sources = [ 182 sources = [
150 "$root_out_dir/Cronet.framework", 183 "$root_out_dir/Cronet.framework",
151 "//AUTHORS", 184 "//AUTHORS",
152 "//chrome/VERSION", 185 "//chrome/VERSION",
153 "//components/cronet/ios/Cronet.h",
154 "//components/cronet/ios/cronet_c_for_grpc.h",
155 ] 186 ]
156 outputs = [ 187 outputs = [
157 "$_package_dir/{{source_file_part}}", 188 "$_package_dir/{{source_file_part}}",
158 ] 189 ]
159 190
160 deps = [ 191 deps = [
161 ":cronet_framework", 192 ":cronet_framework",
162 ] 193 ]
163 } 194 }
164 195
165 group("cronet_package") { 196 group("cronet_package") {
166 deps = [ 197 deps = [
167 ":cronet_package_copy", 198 ":cronet_package_copy",
168 ":generate_license", 199 ":generate_license",
169 ] 200 ]
170 } 201 }
OLDNEW
« no previous file with comments | « no previous file | components/cronet/tools/package_ios.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698