From d488d2f9523f50e8a324f83f93b2f77cb4a8671b Mon Sep 17 00:00:00 2001 From: Andrew Hamilton Date: Sat, 30 Jan 2016 07:50:04 +0000 Subject: [PATCH] Fixed 'All results up-to-date" not showing when starting with all up-to-date. --- BUGS | 4 ++-- vigil | 8 ++++---- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/BUGS b/BUGS index fca001f..1b79c4b 100644 --- a/BUGS +++ b/BUGS @@ -1,7 +1,5 @@ Current - Within the sandbox sudo is not working for tools. -- If vigil is started with eveything up-to-date the up-to-date message never - appears. Current (tool related) @@ -212,6 +210,8 @@ Fixed e.g. vigil.py pylint, BUGS metadata, BUGS _pygments - Tracebacks occur if you pause during early startup. - Scrolling in the help screen doesn't work with the arrow keys. +- If vigil is started with eveything up-to-date the up-to-date message never + appears. Won't fix diff --git a/vigil b/vigil index d4f693d..16d41ee 100755 --- a/vigil +++ b/vigil @@ -889,15 +889,15 @@ class Runner: self.result = summary.get_closest_placeholder() except StopIteration: self.result = None + if summary.result_total == summary.completed_total: + log.log_message("All results are up to date.") + if self.is_being_tested: + os.kill(os.getpid(), signal.SIGINT) break with contextlib.suppress(ValueError): # Process was terminated self.result.run(log, appearance_changed_event, self.worker, self) summary.completed_total += 1 - if summary.result_total == summary.completed_total: - log.log_message("All results are up to date.") - if self.is_being_tested: - os.kill(os.getpid(), signal.SIGINT) jobs_added_event.clear() def pause(self):