Merge lp://staging/~javier.collado/utah/bug1071020 into lp://staging/utah
Proposed by
Javier Collado
| Status: | Merged |
|---|---|
| Approved by: | Javier Collado |
| Approved revision: | 828 |
| Merged at revision: | 827 |
| Proposed branch: | lp://staging/~javier.collado/utah/bug1071020 |
| Merge into: | lp://staging/utah |
| Diff against target: |
33 lines (+12/-0) 2 files modified
debian/changelog (+2/-0) utah/client/common.py (+10/-0) |
| To merge this branch: | bzr merge lp://staging/~javier.collado/utah/bug1071020 |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Max Brustkern (community) | Approve | ||
|
Review via email:
|
|||
Description of the change
This branch creates a fake command result when the run_as user is not found in
the password database with an error that is friendlier than the error that the
user would get if the command were really executed.
I don't like much creating fake results to convey error messages, but I haven't
found any better way to do this. Maybe we could create a different result
dictionary for those commands that haven't been executed, but I don't really
thing that is worth the effort for now. Please let me now your thoughts on this.
To post a comment you must log in.
Revision history for this message
| Andy Doan (doanac) wrote | # |
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
+1