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

Side by Side Diff: build/protoc_java.gypi

Issue 11308030: Use _target_name instead of package_name for java*.gypi (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@apk_jar
Patch Set: Created 7 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 | « build/java_apk.gypi ('k') | chrome/chrome.gyp » ('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 (c) 2012 The Chromium Authors. All rights reserved. 1 # Copyright (c) 2012 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 is meant to be included into a target to provide a rule 5 # This file is meant to be included into a target to provide a rule
6 # to invoke protoc in a consistent manner. This is only to be included 6 # to invoke protoc in a consistent manner. This is only to be included
7 # for Java targets. When including this file, a .jar-file will be generated. 7 # for Java targets. When including this file, a .jar-file will be generated.
8 # For other targets, see protoc.gypi. 8 # For other targets, see protoc.gypi.
9 # 9 #
10 # To use this, create a gyp target with the following form: 10 # To use this, create a gyp target with the following form:
(...skipping 27 matching lines...) Expand all
38 38
39 { 39 {
40 'variables': { 40 'variables': {
41 'protoc': '<(PRODUCT_DIR)/<(EXECUTABLE_PREFIX)protoc<(EXECUTABLE_SUFFIX)', 41 'protoc': '<(PRODUCT_DIR)/<(EXECUTABLE_PREFIX)protoc<(EXECUTABLE_SUFFIX)',
42 'java_out_dir': '<(PRODUCT_DIR)/java_proto/<(_target_name)/src', 42 'java_out_dir': '<(PRODUCT_DIR)/java_proto/<(_target_name)/src',
43 'proto_in_dir%': '.', 43 'proto_in_dir%': '.',
44 'stamp_file': '<(java_out_dir).stamp', 44 'stamp_file': '<(java_out_dir).stamp',
45 'script': '<(DEPTH)/build/protoc_java.py', 45 'script': '<(DEPTH)/build/protoc_java.py',
46 46
47 # The rest of the variables here are for the java.gypi include. 47 # The rest of the variables here are for the java.gypi include.
48 'package_name': '<(_target_name)',
49 'java_in_dir': '<(DEPTH)/build/android/empty', 48 'java_in_dir': '<(DEPTH)/build/android/empty',
50 'generated_src_dirs': ['<(java_out_dir)'], 49 'generated_src_dirs': ['<(java_out_dir)'],
51 # Adding the |stamp_file| to |additional_input_paths| makes the actions in 50 # Adding the |stamp_file| to |additional_input_paths| makes the actions in
52 # the include of java.gypi depend on the genproto_java action. 51 # the include of java.gypi depend on the genproto_java action.
53 'additional_input_paths': ['<(stamp_file)'], 52 'additional_input_paths': ['<(stamp_file)'],
54 }, 53 },
55 'actions': [ 54 'actions': [
56 { 55 {
57 'action_name': 'genproto_java', 56 'action_name': 'genproto_java',
58 'inputs': [ 57 'inputs': [
(...skipping 15 matching lines...) Expand all
74 ], 73 ],
75 'message': 'Generating Java code from <(proto_in_dir)', 74 'message': 'Generating Java code from <(proto_in_dir)',
76 }, 75 },
77 ], 76 ],
78 'dependencies': [ 77 'dependencies': [
79 '<(DEPTH)/third_party/protobuf/protobuf.gyp:protoc#host', 78 '<(DEPTH)/third_party/protobuf/protobuf.gyp:protoc#host',
80 '<(DEPTH)/third_party/protobuf/protobuf.gyp:protobuf_lite_javalib', 79 '<(DEPTH)/third_party/protobuf/protobuf.gyp:protobuf_lite_javalib',
81 ], 80 ],
82 'includes': [ 'java.gypi' ], 81 'includes': [ 'java.gypi' ],
83 } 82 }
OLDNEW
« no previous file with comments | « build/java_apk.gypi ('k') | chrome/chrome.gyp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698