Revert "tools,gyp: don't force build actions with multiple outputs"

This reverts commit 5d8373a. Fixes: #23255 PR-URL: #23257 Reviewed-By: Daniel Bevenius <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Refael Ackermann <[email protected]> Reviewed-By: Anna Henningsen <[email protected]>
nodejs · Oct 5, 2018 · a656268 · a656268
1 parent a6b55c7
commit a656268
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/tools/gyp/pylib/gyp/generator/make.py b/tools/gyp/pylib/gyp/generator/make.py
@@ -1758,10 +1758,8 @@ def WriteMakeRule(self, outputs, inputs, actions=None, comment=None,
       self.WriteLn('%s: %s' % (' '.join(outputs), intermediate))
       self.WriteLn('\t%s' % '@:')
       self.WriteLn('%s: %s' % ('.INTERMEDIATE', intermediate))
-      # Don't add `force_append` (FORCE_DO_CMD) to the intermediate sentinal.
-      # Adding it makes the action run alway, even when there are no changes.
-      # (refack): AFAICT because `*.intermediate` files don't have build rules.
-      self.WriteLn('%s: %s' % (intermediate, ' '.join(inputs)))
+      self.WriteLn('%s: %s%s' %
+                   (intermediate, ' '.join(inputs), force_append))
       actions.insert(0, '$(call do_cmd,touch)')
 
     if actions: