Retrieve generator variables and use those for prefix/suffix, rather than repeating

pylib/gyp/generator/ninja.py

 # Copyright (c) 2012 Google Inc. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
+import copy
 import gyp
 import gyp.common
 import gyp.system_test
 import gyp.xcode_emulation
 import os.path
 import re
 import subprocess
 import sys
 
 import gyp.ninja_syntax as ninja_syntax
@@ skipping 808 matching lines @@
     """Return the 'output' (full output path) to the binary in a bundle."""
     assert self.is_mac_bundle
     path = self.ExpandSpecial(generator_default_variables['PRODUCT_DIR'])
     return os.path.join(path, self.xcode_settings.GetExecutablePath())
 
   def ComputeOutputFileName(self, spec, type=None):
     """Compute the filename of the final output for the current target."""
     if not type:
       type = spec['type']
 
+    default_variables = copy.copy(generator_default_variables)
+    CalculateVariables(default_variables, {'flavor': self.flavor})
+
     # Compute filename prefix: the product prefix, or a default for
     # the product type.
-    if self.flavor == 'win':
-      DEFAULT_PREFIX = {
-        'loadable_module': '',
-        'shared_library': '',
-        'static_library': '',
-        }
-    else:
-      DEFAULT_PREFIX = {
-        'loadable_module': 'lib',
-        'shared_library': 'lib',
-        'static_library': 'lib',
-        }
+    DEFAULT_PREFIX = {
+      'loadable_module': default_variables['SHARED_LIB_PREFIX'],
+      'shared_library': default_variables['SHARED_LIB_PREFIX'],
+      'static_library': default_variables['STATIC_LIB_PREFIX'],
+      'executable': default_variables['EXECUTABLE_PREFIX'],
+      }
     prefix = spec.get('product_prefix', DEFAULT_PREFIX.get(type, ''))
 
     # Compute filename extension: the product extension, or a default
     # for the product type.
-    if self.flavor == 'win':
-      DEFAULT_EXTENSION = {
-        'static_library': 'lib',
-        'loadable_module': 'dll',
-        'shared_library': 'dll',
-        'executable': 'exe',
-        }
-    else:
-      DEFAULT_EXTENSION = {
-        'static_library': 'a',
-        'loadable_module': 'so',
-        'shared_library': 'so',
-        }
+    DEFAULT_EXTENSION = {
+        'loadable_module': default_variables['SHARED_LIB_SUFFIX'],
+        'shared_library': default_variables['SHARED_LIB_SUFFIX'],
+        'static_library': default_variables['STATIC_LIB_SUFFIX'],
+        'executable': default_variables['EXECUTABLE_SUFFIX'],
+      }
     extension = spec.get('product_extension',
                          DEFAULT_EXTENSION.get(type, ''))
-    if extension:
-      extension = '.' + extension
 
     if 'product_name' in spec:
       # If we were given an explicit name, use that.
       target = spec['product_name']
     else:
       # Otherwise, derive a name from the target name.
       target = spec['target_name']
       if prefix == 'lib':
         # Snip out an extra 'lib' from libs if appropriate.
         target = StripPrefix(target, 'lib')
@@ skipping 109 matching lines @@
     global generator_additional_non_configuration_keys
     generator_additional_non_configuration_keys = getattr(xcode_generator,
         'generator_additional_non_configuration_keys', [])
     global generator_additional_path_sections
     generator_additional_path_sections = getattr(xcode_generator,
         'generator_additional_path_sections', [])
     global generator_extra_sources_for_rules
     generator_extra_sources_for_rules = getattr(xcode_generator,
         'generator_extra_sources_for_rules', [])
   elif flavor == 'win':
-    default_variables.setdefault('OS', 'win')
-    default_variables.setdefault('EXECUTABLE_SUFFIX', '.exe')
-    default_variables.setdefault('STATIC_LIB_PREFIX', '')
-    default_variables.setdefault('STATIC_LIB_SUFFIX', '.lib')
-    default_variables.setdefault('SHARED_LIB_PREFIX', '')
-    default_variables.setdefault('SHARED_LIB_SUFFIX', '.dll')
+    default_variables['OS'] = 'win'
+    default_variables['EXECUTABLE_SUFFIX'] = '.exe'
+    default_variables['STATIC_LIB_PREFIX'] = ''
+    default_variables['STATIC_LIB_SUFFIX'] = '.lib'
+    default_variables['SHARED_LIB_PREFIX'] = ''
+    default_variables['SHARED_LIB_SUFFIX'] = '.dll'
   else:
     operating_system = flavor
     if flavor == 'android':
       operating_system = 'linux'  # Keep this legacy behavior for now.
     default_variables.setdefault('OS', operating_system)
     default_variables.setdefault('SHARED_LIB_SUFFIX', '.so')
     default_variables.setdefault('SHARED_LIB_DIR',
                                  os.path.join('$!PRODUCT_DIR', 'lib'))
     default_variables.setdefault('LIB_DIR', '')
 
@@ skipping 242 matching lines @@
 
   user_config = params.get('generator_flags', {}).get('config', None)
   if user_config:
     GenerateOutputForConfig(target_list, target_dicts, data, params,
                             user_config)
   else:
     config_names = target_dicts[target_list[0]]['configurations'].keys()
     for config_name in config_names:
       GenerateOutputForConfig(target_list, target_dicts, data, params,
                               config_name)