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

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

Issue 2418433003: Don't base android lint version off of android sdk version. (Closed)
Patch Set: 24->23 Created 4 years, 2 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 # This file contains common system config stuff for the Android build. 5 # This file contains common system config stuff for the Android build.
6 6
7 if (is_android) { 7 if (is_android) {
8 assert(rebase_path("//", root_build_dir) == "../../", 8 assert(rebase_path("//", root_build_dir) == "../../",
9 "Android output directory must be nested 2 levels within src/ (" + 9 "Android output directory must be nested 2 levels within src/ (" +
10 "e.g.: out-gn/Debug). http://crbug.com/412935") 10 "e.g.: out-gn/Debug). http://crbug.com/412935")
(...skipping 22 matching lines...) Expand all
33 assert(defined(default_android_ndk_major_version)) 33 assert(defined(default_android_ndk_major_version))
34 } 34 }
35 35
36 if (!defined(default_android_sdk_root)) { 36 if (!defined(default_android_sdk_root)) {
37 default_android_sdk_root = "//third_party/android_tools/sdk" 37 default_android_sdk_root = "//third_party/android_tools/sdk"
38 default_android_sdk_version = "23" 38 default_android_sdk_version = "23"
39 default_android_sdk_build_tools_version = "23.0.1" 39 default_android_sdk_build_tools_version = "23.0.1"
40 } 40 }
41 41
42 if (!defined(default_lint_android_sdk_root)) { 42 if (!defined(default_lint_android_sdk_root)) {
43 default_lint_android_sdk_root = default_android_sdk_root 43 # Purposefully repeated so that downstream can change
44 default_lint_android_sdk_version = default_android_sdk_version 44 # default_android_sdk_root without changing lint version.
45 default_lint_android_sdk_root = "//third_party/android_tools/sdk"
46 default_lint_android_sdk_version = "23"
45 } 47 }
46 48
47 if (!defined(default_android_keystore_path)) { 49 if (!defined(default_android_keystore_path)) {
48 default_android_keystore_path = 50 default_android_keystore_path =
49 "//build/android/ant/chromium-debug.keystore" 51 "//build/android/ant/chromium-debug.keystore"
50 default_android_keystore_name = "chromiumdebugkey" 52 default_android_keystore_name = "chromiumdebugkey"
51 default_android_keystore_password = "chromium" 53 default_android_keystore_password = "chromium"
52 } 54 }
53 55
54 if (!defined(google_play_services_library)) { 56 if (!defined(google_play_services_library)) {
(...skipping 273 matching lines...) Expand 10 before | Expand all | Expand 10 after
328 "//build/toolchain/android:${android_secondary_abi_cpu}" 330 "//build/toolchain/android:${android_secondary_abi_cpu}"
329 } 331 }
330 } 332 }
331 } 333 }
332 334
333 declare_args() { 335 declare_args() {
334 # Enables used resource whitelist generation. Set for official builds only 336 # Enables used resource whitelist generation. Set for official builds only
335 # as a large amount of build output is generated. 337 # as a large amount of build output is generated.
336 enable_resource_whitelist_generation = is_android && is_official_build 338 enable_resource_whitelist_generation = is_android && is_official_build
337 } 339 }
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