Mark all outputs as done, not just target

Avoids duplicate work in certain situations (with -b, but possible also in others?).
texttheater · Feb 24, 2021 · 7c33480 · 7c33480
1 parent e438f76
commit 7c33480
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/produce b/produce
@@ -728,6 +728,13 @@ class Production:
                                         neutral_targets, pretend_up_to_date)
                 self.set_result(target, result)
                 debug(3, 'created result for {}: {}'.format(target, repr(result)))
+                # Step 7: mark other outputs as done
+                for output in outputs:
+                    if output == target:
+                        continue
+                    output_result = ProductionResult(True, mtime(output))
+                    self.set_result(output, output_result)
+                    debug(3, 'created result for {}: {}'.format(output, repr(output_result)))
                 return result
             except BaseException as e:
                 self.set_result(target, e)
@@ -865,7 +872,7 @@ class Production:
                 debug(3, 'started subprocess')
                 while True:
                     try:
-                        debug(3, 'waiting')
+                        debug(5, 'waiting')
                         proc.wait(0.05)
                     except subprocess.TimeoutExpired:
                         pass

diff --git a/setup.cfg b/setup.cfg
@@ -1,6 +1,6 @@
 [metadata]
 name = produce
-version = 0.4.1
+version = 0.4.2
 author = Kilian Evang
 author_email = kilian.evang@gmail.com
 url = https://github.com/texttheater/produce