diff options
| author | Tristan Van Berkom <tristan.vanberkom@codethink.co.uk> | 2017-07-12 17:32:49 +0900 |
|---|---|---|
| committer | Tristan Van Berkom <tristan.vanberkom@codethink.co.uk> | 2017-07-12 17:32:49 +0900 |
| commit | aa0ec80255bee0771a0d0fe5d9401b9f34d7036f (patch) | |
| tree | d3c257c9b51fc23395c8d8d159a9c63407ec1f07 /buildstream/_scheduler/pushqueue.py | |
| parent | 611c8515b7d8048c4aaaf33983b975b446cffe97 (diff) | |
| download | buildstream-aa0ec80255bee0771a0d0fe5d9401b9f34d7036f.tar.gz | |
_scheduler: Changed return value of Queue.done() around
Previously it returned True to record an element as 'skipped',
this seems backwards; instead return True to indicate an element
was 'processed' and False to indicate an element was 'skipped'
Diffstat (limited to 'buildstream/_scheduler/pushqueue.py')
| -rw-r--r-- | buildstream/_scheduler/pushqueue.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/buildstream/_scheduler/pushqueue.py b/buildstream/_scheduler/pushqueue.py index 1c0ab9824..a339f5604 100644 --- a/buildstream/_scheduler/pushqueue.py +++ b/buildstream/_scheduler/pushqueue.py @@ -49,4 +49,4 @@ class PushQueue(Queue): # Element._push() returns True if it uploaded an artifact, # here we want to appear skipped if the remote already had # the artifact. - return not result + return result |