diff options
author | Brian Dolbec <brian.dolbec@gmail.com> | 2011-02-13 16:47:14 -0800 |
---|---|---|
committer | Brian Dolbec <brian.dolbec@gmail.com> | 2011-02-13 16:47:14 -0800 |
commit | ac8a52948f107265718dabe97e144a7cc4a70091 (patch) | |
tree | 6a8c3e2f647596f7d97bf968c09739571ed97378 | |
parent | simplify debug.py to subclass the simplified Message class (diff) | |
download | layman-ac8a52948f107265718dabe97e144a7cc4a70091.tar.gz layman-ac8a52948f107265718dabe97e144a7cc4a70091.tar.bz2 layman-ac8a52948f107265718dabe97e144a7cc4a70091.zip |
fix a long line, clean an unused import, fix the sync return value better
-rw-r--r-- | layman/api.py | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/layman/api.py b/layman/api.py index f3d68b6..63682f6 100644 --- a/layman/api.py +++ b/layman/api.py @@ -17,7 +17,6 @@ from sys import stderr import os from layman.config import BareConfig -#from layman.action import Sync from layman.dbbase import UnknownOverlayException, UnknownOverlayMessage from layman.db import DB, RemoteDB @@ -320,7 +319,8 @@ class LaymanAPI(object): candidates = ' %s' % tuple(available_srcs)[0] else: plural = 's' - candidates = '\n'.join((' %d. %s' % (ovl + 1, v)) for ovl, v in enumerate(available_srcs)) + candidates = '\n'.join((' %d. %s' % (ovl + 1, v)) \ + for ovl, v in enumerate(available_srcs)) warnings.append((ovl, 'The source of the overlay "%(repo_name)s" seems to have changed.\n' @@ -363,11 +363,10 @@ class LaymanAPI(object): self.output.error('\nErrors:\n------\n') for ovl, result in fatals: self.output.error(result + '\n') - return False self.sync_results = (success, warnings, fatals) - return True + return fatals != [] def fetch_remote_list(self): |