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

Unified Diff: remoting/remoting.gyp

Issue 10075002: Source major & minor versions from remoting, build & patch versions from Chrome. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase. Created 8 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « remoting/VERSION ('k') | remoting/webapp/build-webapp.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/remoting.gyp
diff --git a/remoting/remoting.gyp b/remoting/remoting.gyp
index 6221af0c1a2701ca36bf168c4bd5b56d4f3f3efa..50b3af2784ac600da714fc0fef255d50a1465e01 100644
--- a/remoting/remoting.gyp
+++ b/remoting/remoting.gyp
@@ -1,3 +1,4 @@
+
# Copyright (c) 2012 The Chromium Authors. All rights reserved.
# Use of this source code is governed by a BSD-style license that can be
# found in the LICENSE file.
@@ -13,16 +14,14 @@
'host_plugin_mime_type': 'application/vnd.chromium.remoting-host',
'host_plugin_description': 'Allow another user to access your computer securely over the Internet.',
- # Borrow the scripts for generating version information for remoting
- # binaries from Chrome.
- 'variables': {
- 'version_py_path': '../chrome/tools/build/version.py',
- 'version_path': '../remoting/VERSION',
- },
- 'version_py_path': '<(version_py_path)',
- 'version_path': '<(version_path)',
+ # The version is composed from major & minor versions specific to remoting
+ # and build & patch versions inherited from Chrome.
+ 'version_py_path': '../chrome/tools/build/version.py',
+ 'version_path': '../remoting/VERSION',
+ 'chrome_version_path': '../chrome/VERSION',
'version_full':
- '<!(python <(version_py_path) -f <(version_path) -t "@MAJOR@.@MINOR@.@BUILD@.@PATCH@")',
+ '<!(python <(version_py_path) -f <(version_path) -t "@MAJOR@.@MINOR@").'
+ '<!(python <(version_py_path) -f <(chrome_version_path) -t "@BUILD@.@PATCH@")',
'conditions': [
['OS=="mac"', {
@@ -317,9 +316,10 @@
# Generates the version information resources for the Windows binaries.
# The .RC files are generated from the "version.rc.version" template and
# placed in the "<(SHARED_INTERMEDIATE_DIR)/remoting" folder.
- # The substiture strings are taken from:
- # - remoting/VERSION - the current version of Chromoting.
+ # The substition strings are taken from:
alexeypa (please no reviews) 2012/04/14 04:08:56 typo: substition
Wez 2012/04/14 22:23:12 Done. Mis-spelled a spelling correction. Ouch!
# - build/util/LASTCHANGE - the last source code revision.
+ # - chrome/VERSION - the build & patch versions.
+ # - remoting/VERSION - the major & minor versions.
# - xxx_branding - UI/localizable strings.
# - xxx.ver - per-binary non-localizable strings such as the binary
# name.
@@ -335,6 +335,7 @@
'version.rc.version',
'<(DEPTH)/build/util/LASTCHANGE',
'<(version_path)',
+ '<(chrome_version_path)',
],
'direct_dependent_settings': {
'include_dirs': [
@@ -369,6 +370,7 @@
'inputs': [
'<(template_input_path)',
'<(version_path)',
+ '<(chrome_version_path)',
'<(branding_path)',
'<(lastchange_path)',
],
@@ -379,6 +381,7 @@
'python',
'<(version_py_path)',
'-f', '<(RULE_INPUT_PATH)',
+ '-f', '<(chrome_version_path)',
'-f', '<(version_path)',
'-f', '<(branding_path)',
'-f', '<(lastchange_path)',
@@ -621,6 +624,8 @@
'sources': [
'webapp/build-webapp.py',
'webapp/verify-webapp.py',
+ '<(version_path)',
+ '<(chrome_version_path)',
'<@(remoting_webapp_files)',
'<@(remoting_webapp_locale_files)',
],
@@ -671,6 +676,8 @@
'inputs': [
'webapp/build-webapp.py',
'<(_plugin_path)',
+ '<(version_path)',
+ '<(chrome_version_path)',
'<@(remoting_webapp_files)',
'<@(remoting_webapp_locale_files)',
],
@@ -681,6 +688,7 @@
'action': [
'python', 'webapp/build-webapp.py',
'<(buildtype)',
+ '<(version_full)',
'<(host_plugin_mime_type)',
'<(_output_dir)',
'<(_zip_path)',
« no previous file with comments | « remoting/VERSION ('k') | remoting/webapp/build-webapp.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698