diff options
| author | dol-sen <brian.dolbec@gmail.com> | 2011-05-17 22:12:50 -0700 |
|---|---|---|
| committer | dol-sen <brian.dolbec@gmail.com> | 2011-05-17 22:12:50 -0700 |
| commit | 50b971a3724c2b5637d1ce3fc6587d6e3325f2ca (patch) | |
| tree | f9db26d31dfd5125cb484266f0d67f0af662a6a0 | |
| parent | 6dbd30b42a513437e4219446f34997610b351353 (diff) | |
| download | layman-50b971a3724c2b5637d1ce3fc6587d6e3325f2ca.tar.gz layman-50b971a3724c2b5637d1ce3fc6587d6e3325f2ca.tar.bz2 layman-50b971a3724c2b5637d1ce3fc6587d6e3325f2ca.zip | |
fix a false -d success for non-existent overlay. fix a couple error messages.
fix a long line
| -rw-r--r-- | layman/api.py | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/layman/api.py b/layman/api.py index 7e4ad0e..30f850c 100644 --- a/layman/api.py +++ b/layman/api.py @@ -111,13 +111,15 @@ class LaymanAPI(object): results = [] for ovl in repos: if not self.is_installed(ovl): - results.append(True) + self._error("Repository '"+ovl+"': was not installed") + results.append(False) continue try: - self._get_installed_db().delete(self._get_installed_db().select(ovl)) + self._get_installed_db().delete( + self._get_installed_db().select(ovl)) results.append(True) except Exception as e: - self._error(ERROR_INTERNAL_ERROR, + self._error( "Failed to disable repository '"+ovl+"':\n"+str(e)) results.append(False) self.get_installed(dbreload=True) @@ -148,8 +150,7 @@ class LaymanAPI(object): self._get_installed_db().add(self._get_remote_db().select(ovl), quiet=True) results.append(True) except Exception as e: - self._error(ERROR_INTERNAL_ERROR, - "Failed to enable repository '"+ovl+"' : "+str(e)) + self._error("Failed to enable repository '"+ovl+"' : "+str(e)) results.append(False) self.get_installed(dbreload=True) if False in results: |