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

Unified Diff: build/android/gyp/copy_v17_resources.py

Issue 14476011: [Android] Auto-generate API 14 resources from the existing API 17 resources. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fixed one missing dependencies_res_input_dirs renaming 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 side-by-side diff with in-line comments
Download patch
Index: build/android/gyp/copy_v17_resources.py
diff --git a/build/android/gyp/copy_v17_resources.py b/build/android/gyp/copy_v17_resources.py
new file mode 100755
index 0000000000000000000000000000000000000000..381ee23c92c70609b963d2452efe2828b5bb4bff
--- /dev/null
+++ b/build/android/gyp/copy_v17_resources.py
@@ -0,0 +1,87 @@
+#!/usr/bin/env python
+#
+# Copyright 2013 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.
+
+"""Copy resource files and add -v17 to the sub directory names.
cjhopman 2013/05/03 21:23:04 Nit: Copy xml resource files? See comment below.
Kibeom Kim (inactive) 2013/05/03 22:09:57 Done.
+
+This is coupled with generate_v14_resources.py. Please refer to
+generate_v14_resources.py's comment for why we are doing this.
+Or crbug.com/235118 .
cjhopman 2013/05/03 21:23:04 Nit: add http:// (many linkifiers require it).
Kibeom Kim (inactive) 2013/05/03 22:09:57 Done.
+"""
+
+import optparse
+import os
+import shutil
+import sys
+
+from util import build_utils
+
+
+def CopyResourcesInDir(input_dir, output_dir):
cjhopman 2013/05/03 21:23:04 Nit: Maybe rename to CopyXmlResourcesInDir? Or is
Kibeom Kim (inactive) 2013/05/03 22:09:57 Done.
+ """Copy all XML resources from input_dir to output_dir."""
+ for input_file in build_utils.FindInDirectory(input_dir, '*.xml'):
+ output_path = os.path.join(output_dir,
+ os.path.relpath(input_file, input_dir))
+ build_utils.MakeDirectory(os.path.dirname(output_path))
+ shutil.copy2(input_file, output_path)
+
+
+def ParseArgs():
+ """Parses command line options.
+
+ Returns:
+ An options object as from optparse.OptionsParser.parse_args()
+ """
+ parser = optparse.OptionParser()
+ parser.add_option('--res-dir',
+ help='directory containing resources to be copied')
+ parser.add_option('--res-v17-dir',
+ help='output directory to which resources will be copied.')
+ parser.add_option('--stamp', help='File to touch on success')
+
+ options, args = parser.parse_args()
+
+ if args:
+ parser.error('No positional arguments should be given.')
+
+ # Check that required options have been provided.
+ required_options = ('res_dir', 'res_v17_dir')
+ build_utils.CheckOptions(options, parser, required=required_options)
+ return options
+
+
+def main(argv):
+ options = ParseArgs()
+
+ build_utils.DeleteDirectory(options.res_v17_dir)
+ build_utils.MakeDirectory(options.res_v17_dir)
+
+ for name in os.listdir(options.res_dir):
+ if not os.path.isdir(os.path.join(options.res_dir, name)):
+ continue
+
+ dir_pieces = name.split('-')
+ resource_type = dir_pieces[0]
+ qualifiers = dir_pieces[1:]
+
+ # We only copy resources under layout*/ and xml*/.
+ if resource_type not in ('layout', 'xml'):
+ continue
+
+ # Skip RTL resources because they are not supported by API 14.
+ if 'ldrtl' in qualifiers:
+ continue
+
+ # Copy all the resource files.
+ input_path = os.path.join(options.res_dir, name)
+ output_path = os.path.join(options.res_v17_dir, name + '-v17')
+ CopyResourcesInDir(input_path, output_path)
+
+ if options.stamp:
+ build_utils.Touch(options.stamp)
+
+if __name__ == '__main__':
+ sys.exit(main(sys.argv))
+
« no previous file with comments | « no previous file | build/android/gyp/generate_v14_resources.py » ('j') | build/android/gyp/generate_v14_resources.py » ('J')

Powered by Google App Engine
This is Rietveld 408576698