On 15/04/15 15:53, Martin Babinsky wrote:
On 04/14/2015 04:24 PM, Martin Basti wrote:
I just reverted this bit to original form in order to not fix
something that isn't broken. Is that ok?
On 14/04/15 16:12, Martin Basti wrote:
On 14/04/15 14:25, Martin Babinsky wrote:
This patch addresses https://fedorahosted.org/freeipa/ticket/4966
The noise during rollback/uninstall is caused mainly by unsuccessful
attempts to remove files that do not exist anymore. These errors are
now logged at debug level and do not pop-up to stdout/stderr.
Hello, thank you for the patch.
The option add_warning is quite unclear to me. It does not show
warning but error. I suggest something like, show_hint,
show_user_action, or something show_additional_..., or
except OSError as e:
log_file_removal_error(e, preferences_fname, True)
In this case file not found error should never happen.
Could you remove the 'if file_exists' part and handle just exception?
this is inconsistent with change above, choose one style please:
except OSError, e:
"Failed to remove '%s': %s", ca_file, e)
Attaching updated patch.
just one nitpick, can you move the new function into installutils, it
can be used in different scripts not just in ipaclient.
Manage your subscription for the Freeipa-devel mailing list:
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code