Skip to content

Commit

Permalink
Bugfix: was marking non-updatd outputs as updated
Browse files Browse the repository at this point in the history
Need to check whether the target was actually updated before marking side
outputs as updatd.
  • Loading branch information
texttheater committed Feb 25, 2021
1 parent 7c33480 commit 0b8ca71
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 8 deletions.
15 changes: 8 additions & 7 deletions produce
Original file line number Diff line number Diff line change
Expand Up @@ -728,13 +728,14 @@ 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)))
# Step 7: mark other outputs as updated
if result.updated:
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)
Expand Down
2 changes: 1 addition & 1 deletion setup.cfg
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
[metadata]
name = produce
version = 0.4.2
version = 0.4.3
author = Kilian Evang
author_email = kilian.evang@gmail.com
url = https://github.com/texttheater/produce
Expand Down

0 comments on commit 0b8ca71

Please sign in to comment.