From f337df463d62514c2729ca8c3e1137170e44c95d Mon Sep 17 00:00:00 2001 From: Avery Pennarun Date: Mon, 22 Nov 2010 22:51:54 -0800 Subject: [PATCH] state.stamp() can't imply state.built(). ...because we deliberately stamp non-generated files as well, and that doesn't need to imply that we rebuilt them just now. In fact, we know for a fact that we *didn't* rebuild them just now, but we still need to record the timestamp for later. --- builder.py | 1 + state.py | 1 - 2 files changed, 1 insertion(+), 1 deletion(-) diff --git a/builder.py b/builder.py index e21478a..bf2e373 100644 --- a/builder.py +++ b/builder.py @@ -139,6 +139,7 @@ class BuildJob: os.rename(tmpname, t) else: unlink(tmpname) + state.built(t) state.stamp(t) else: unlink(tmpname) diff --git a/state.py b/state.py index 6810bab..fce958d 100644 --- a/state.py +++ b/state.py @@ -71,7 +71,6 @@ def _stampname(t): def stamp(t): - built(t) mark(t) stampfile = _stampname(t) newstampfile = _sname('stamp' + str(os.getpid()), t)