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

Side by Side Diff: build/android/tests/multiple_proguards/multiple_proguards.gyp

Issue 12913017: Support multiple proguard flags from java_apk.gypi. Proguard supports multiple proguard configurati… (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 7 years, 8 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
OLDNEW
(Empty)
1 # Copyright (c) 2013 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file.
4 {
5 'variables': {
6 'chromium_code': 1,
7 'package_name': 'multiple_proguard',
8 },
9 'targets': [
10 {
11 'target_name': 'multiple_proguards_test_apk',
12 'type': 'none',
13 'variables': {
14 'app_manifest_version_name%': '<(android_app_version_name)',
15 'app_manifest_version_code%': '<(android_app_version_code)',
cjhopman 2013/03/29 14:43:46 You shouldn't need the app_manifest_version* varia
jayjang 2013/03/29 15:05:19 Done.
16 'java_in_dir': '../multiple_proguards',
cjhopman 2013/03/29 14:43:46 change this to: 'java_in_dir': 'src',
jayjang 2013/03/29 15:05:19 it couldn't be 'src'. In fact it searches manifest
17 'proguard_enabled': 'true',
18 'proguard_flags_paths': [
19 '<(DEPTH)/build/android/tests/multiple_proguards/proguard1.flags',
cjhopman 2013/03/29 14:43:46 These paths can be relative to this directory, i.e
jayjang 2013/03/29 15:05:19 Done.
20 '<(DEPTH)/build/android/tests/multiple_proguards/proguard2.flags',
21 ],
22 'R_package': 'dummy',
23 'R_package_relpath': 'dummy',
cjhopman 2013/03/29 14:43:46 I don't think you need these R_package* variables
jayjang 2013/03/29 15:05:19 Unfortunately it's required because some variables
24 'apk_name': 'MultipleProguards',
25 },
26 'dependencies': [
27 # guava has references to objects using reflection which
28 # should be ignored in proguard step.
29 '../../../../third_party/guava/guava.gyp:guava_javalib',
30 ],
31 'includes': [ '../../../../build/java_apk.gypi' ],
32 },
33 ],
34 }
OLDNEW
« no previous file with comments | « build/android/tests/multiple_proguards/AndroidManifest.xml ('k') | build/android/tests/multiple_proguards/proguard1.flags » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698