forks/apenwarr-redo
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

state.File.is_checked() was being too paranoid.

Browse source 
It wasn't allowing us to short circuit a dependency if that dependency had
been built previously, but that was already being checked (more correctly)
in dirty_deps().
This commit is contained in:
Avery Pennarun 2010-12-09 03:43:08 -08:00
commit e1a0fc9c12
1 changed files with 1 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

4
state.py
View file

@ -204,9 +204,7 @@ class File(object):
return self.changed_runid and self.changed_runid >= vars.RUNID return self.changed_runid and self.changed_runid >= vars.RUNID
def is_checked(self): def is_checked(self):
return (self.checked_runid and self.checked_runid >= vars.RUNID return self.checked_runid and self.checked_runid >= vars.RUNID
and not (self.changed_runid
and self.changed_runid >= self.checked_runid))
def deps(self): def deps(self):
q = ('select Deps.mode, Deps.source, ' q = ('select Deps.mode, Deps.source, '