On 09/06/2013 02:58 PM, Ana Krivokapic wrote:
This patch fixes the regression introduced by the original fix for ticket #3867.
Thank, ACK, pushed to:
Tiny nitpicks below.
+ msg = ("\nWARNING: Failed to connect to Directory Server to find "
+ "information about replication agreements. Uninstallation "
+ "will continue dispite the possible existing replication "
^^^^^^ Typo, fixed before pushing
+ if not (options.unattended or user_input("Are you sure you "
+ "want to continue "
+ "with the uninstall "
This looks funny :)
On a more rational note, unnecessarily breaking up strings makes them
harder to grep.
In the future you can use the `msg` trick you used for the WARNING above.
Freeipa-devel mailing list