Revert r1362: MSVS 2008: Fix to ensure that actions are re-run when the command changes

pylib/gyp/generator/msvs.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 ntpath
 import os
 import posixpath
 import re
 import subprocess
@@ skipping 886 matching lines @@
   sources, excluded_sources, excluded_idl = (
       _AdjustSourcesAndConvertToFilterHierarchy(
           spec, options, project_dir, sources, excluded_sources, list_excluded))
 
   # Add in files.
   missing_sources = _VerifySourcesExist(sources, project_dir)
   p.AddFiles(sources)
 
   _AddToolFilesToMSVS(p, spec)
   _HandlePreCompiledHeaders(p, sources, spec)
-  _AddActions(actions_to_add, spec, relative_path_of_gyp_file, project.path)
+  _AddActions(actions_to_add, spec, relative_path_of_gyp_file)
   _AddCopies(actions_to_add, spec)
   _WriteMSVSUserFile(project.path, version, spec)
 
   # NOTE: this stanza must appear after all actions have been decided.
   # Don't excluded sources with actions attached, or they won't run.
   excluded_sources = _FilterActionsFromExcluded(
       excluded_sources, actions_to_add)
   _ExcludeFilesFromBeingBuilt(p, spec, excluded_sources, excluded_idl,
                               list_excluded)
   _AddAccumulatedActionsToMSVS(p, spec, actions_to_add)
@@ skipping 515 matching lines @@
                                     {'UsePrecompiledHeader': '0',
                                      'ForcedIncludeFiles': '$(NOINHERIT)'})
             p.AddFileConfig(_FixPath(source),
                             _ConfigFullName(config_name, config),
                             {}, tools=[tool])
   # Do nothing if there was no precompiled source.
   if extensions_excluded_from_precompile:
     DisableForSourceTree(sources)
 
 
-def _WriteFileIfChanged(filename, data):
-  if os.path.exists(filename):
-    fh = open(filename, 'r')
-    old_data = fh.read()
-    fh.close()
-    if old_data == data:
-      return
-  fh = open(filename, 'w')
-  fh.write(data)
-  fh.close()
-
-
-def _AddActions(actions_to_add, spec, relative_path_of_gyp_file, vcproj_file):
+def _AddActions(actions_to_add, spec, relative_path_of_gyp_file):
   # Add actions.
   actions = spec.get('actions', [])
   # Don't setup_env every time. When all the actions are run together in one
   # batch file in VS, the PATH will grow too long.
   # Membership in this set means that the cygwin environment has been set up,
   # and does not need to be set up again.
   have_setup_env = set()
   for a in actions:
     # Attach actions to the gyp file if nothing else is there.
     inputs = a.get('inputs') or [relative_path_of_gyp_file]
     attached_to = inputs[0]
     need_setup_env = attached_to not in have_setup_env
     cmd = _BuildCommandLineForRule(spec, a, has_input_path=False,
                                    do_setup_env=need_setup_env)
     have_setup_env.add(attached_to)
-    if vcproj_file:
-      # Additionally, on MSVS 2008, we record the command in a file
-      # and add this as a dependency so that when the command changes,
-      # the action gets re-run.  This is not necessary on MSVS 2010
-      # which tracks command changes for us.
-      command_file = '%s_%s.gypcmd' % (vcproj_file, a['action_name'])
-      _WriteFileIfChanged(command_file, cmd)
-      inputs = inputs + [os.path.basename(command_file)]
     # Add the action.
     _AddActionStep(actions_to_add,
                    inputs=inputs,
                    outputs=a.get('outputs', []),
                    description=a.get('message', a['action_name']),
                    command=cmd)
 
 
 def _WriteMSVSUserFile(project_path, version, spec):
   # Add run_as and test targets.
@@ skipping 1467 matching lines @@
 
 
 def _GenerateMSBuildProject(project, options, version, generator_flags):
   spec = project.spec
   configurations = spec['configurations']
   project_dir, project_file_name = os.path.split(project.path)
   msbuildproj_dir = os.path.dirname(project.path)
   if msbuildproj_dir and not os.path.exists(msbuildproj_dir):
     os.makedirs(msbuildproj_dir)
   # Prepare list of sources and excluded sources.
+  gyp_path = _NormalizedSource(project.build_file)
+  relative_path_of_gyp_file = gyp.common.RelativePath(gyp_path, project_dir)
+
   gyp_file = os.path.split(project.build_file)[1]
   sources, excluded_sources = _PrepareListOfSources(spec, gyp_file)
   # Add rules.
   actions_to_add = {}
   props_files_of_rules = set()
   targets_files_of_rules = set()
   extension_to_rule_name = {}
   list_excluded = generator_flags.get('msvs_list_excluded_files', True)
   _GenerateRulesForMSBuild(project_dir, options, spec,
                            sources, excluded_sources,
                            props_files_of_rules, targets_files_of_rules,
                            actions_to_add, extension_to_rule_name)
   sources, excluded_sources, excluded_idl = (
       _AdjustSourcesAndConvertToFilterHierarchy(spec, options,
                                                 project_dir, sources,
                                                 excluded_sources,
                                                 list_excluded))
-  _AddActions(actions_to_add, spec, project.build_file, None)
+  _AddActions(actions_to_add, spec, project.build_file)
   _AddCopies(actions_to_add, spec)
 
   # NOTE: this stanza must appear after all actions have been decided.
   # Don't excluded sources with actions attached, or they won't run.
   excluded_sources = _FilterActionsFromExcluded(
       excluded_sources, actions_to_add)
 
   exclusions = _GetExcludedFilesFromBuild(spec, excluded_sources, excluded_idl)
   actions_spec, sources_handled_by_action = _GenerateActionsForMSBuild(
       spec, actions_to_add)
@@ skipping 127 matching lines @@
   action_spec.extend(
       # TODO(jeanluc) 'Document' for all or just if as_sources?
       [['FileType', 'Document'],
        ['Command', command],
        ['Message', description],
        ['Outputs', outputs]
       ])
   if additional_inputs:
     action_spec.append(['AdditionalInputs', additional_inputs])
   actions_spec.append(action_spec)