diff options
author | Martin Basti <mbasti@redhat.com> | 2016-11-01 13:19:00 +0100 |
---|---|---|
committer | Jan Cholasta <jcholast@redhat.com> | 2016-11-11 12:13:56 +0100 |
commit | 847b6eddab00973740413b4c46f86940cb73d25a (patch) | |
tree | 384c0f1bd3443436cc7eb713fe3832c99229c4ed /ipaclient | |
parent | 5249eb817efbb5708d097173a8d5f1e322fb201e (diff) | |
download | freeipa-847b6eddab00973740413b4c46f86940cb73d25a.tar.gz freeipa-847b6eddab00973740413b4c46f86940cb73d25a.tar.xz freeipa-847b6eddab00973740413b4c46f86940cb73d25a.zip |
client: use correct code for failed uninstall
Hardcoded value 1 means CLIENT_INSTALL_ERROR, but this part belongs to
uninstallation so it should be CLIENT_UNINSTALL_ERROR
https://fedorahosted.org/freeipa/ticket/6392
Reviewed-By: Stanislav Laznicka <slaznick@redhat.com>
Diffstat (limited to 'ipaclient')
-rw-r--r-- | ipaclient/install/client.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/ipaclient/install/client.py b/ipaclient/install/client.py index d1f845ff6..3334396d7 100644 --- a/ipaclient/install/client.py +++ b/ipaclient/install/client.py @@ -3242,7 +3242,7 @@ def uninstall(options): root_logger.warning( "Please remove file '%s' manually.", preferences_fname) - rv = 0 + rv = SUCCESS if fstore.has_files(): root_logger.error('Some files have not been restored, see %s' % @@ -3254,7 +3254,7 @@ def uninstall(options): 'restored, see /var/lib/ipa/sysrestore/sysrestore.state', module) has_state = True - rv = 1 + rv = CLIENT_UNINSTALL_ERROR if has_state: root_logger.warning( |