[Chromoting] Pass list of locale files into build-webapp in a single file.

remoting/tools/build/remoting_localize.py

 #!/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.
 
 """
 localize.py -- Generates an output file from the given template replacing
 variables and localizing strings.
 
 The script uses Jinja2 template processing library (src/third_party/jinja2).
@@ skipping 702 matching lines @@
     target = GypTemplate(options.locale_output)
     for lang in languages:
       context['languages'] = [ lang ]
       context['language'] = lang
       context['pak_suffix'] = GetDataPackageSuffix(lang)
       context['json_suffix'] = GetJsonSuffix(lang)
       message_map.SelectLanguage(lang)
 
       template_file_name = target.safe_substitute(context)
       outputs.append(template_file_name)
-      if not options.print_only:
+      if not options.print_only and not options.print_to_filelist:
         WriteIfChanged(template_file_name, template.render(context),
                        options.encoding)
   else:
     outputs.append(options.output)
     if not options.print_only:
       WriteIfChanged(options.output, template.render(context), options.encoding)
 
   if options.print_only:
     # Quote each element so filename spaces don't mess up gyp's attempt to parse
     # it into a list.
     return " ".join(['"%s"' % x for x in outputs])
 
+  if options.print_to_filelist:
+    # Strip off the quotes from each filename when writing into a listfile.
+    content = u'\n'.join([x.strip('"') for x in outputs])
+    WriteIfChanged(options.print_to_filelist, content, options.encoding)
+
   return
 
 
 def DoMain(argv):
   usage = "Usage: localize [options] locales"
   parser = OptionParser(usage=usage)
   parser.add_option(
       '-d', '--define', dest='define', action='append', type='string',
       help='define a variable (NAME=VALUE).')
   parser.add_option(
       '--encoding', dest='encoding', type='string', default='utf-8',
       help="set the encoding of <output>. 'utf-8' is the default.")
   parser.add_option(
       '--jinja2', dest='jinja2', type='string',
       help="specifies path to the jinja2 library.")
   parser.add_option(
       '--locale_dir', dest='locale_dir', type='string',
       help="set path to localized message files.")
   parser.add_option(
       '--locale_output', dest='locale_output',  type='string',
       help='specify the per-locale output file name.')
   parser.add_option(
       '-o', '--output', dest='output', type='string',
       help="specify the output file name.")
   parser.add_option(
       '--print_only', dest='print_only', action='store_true',
       default=False, help='print the output file names only.')
   parser.add_option(
+      '--print_to_filelist', dest='print_to_filelist', type='string',

[Jamie, 2015/02/05 19:26:20]

print_to_filelist sounds like it should be a boolean. Maybe locale_list_filename
would be a better name for the variable (the command-line can stay as it is if
you prefer).

[garykac, 2015/02/06 02:13:35]

Done.

+      help='print the output file names into a file.')
+  parser.add_option(
       '-t', '--template', dest='template', type='string',
       help="specify the template file name.")
   parser.add_option(
       '--variables', dest='variables', action='append', type='string',
       help='read variables (NAME=VALUE) from file.')
 
   options, locales = parser.parse_args(argv)
   if not locales:
     parser.error('At least one locale must be specified')
   if bool(options.output) == bool(options.locale_output):
     parser.error(
         'Either --output or --locale_output must be specified but not both')
-  if not options.template and not options.print_only:
-    parser.error('The template name is required unless --print_only is used')
+  if (not options.template and
+      not options.print_only and not options.print_to_filelist):
+    parser.error('The template name is required unless either --print_only '
+                 'or --print_to_filelist is used')
 
   return Localize(options.template, locales, options)
 
 if __name__ == '__main__':
   sys.exit(DoMain(sys.argv[1:]))
-