On 10/30/2015 02:09 PM, Martin Basti wrote:

Patch attached

Hi Martin,

NACK since I'm not a big fan of having (nearly) the same function defined in multiple modules:

$ git grep -n 'def exit_on_managed_topology'
install/tools/ipa-csreplica-manage:397:def exit_on_managed_topology(what, hint="topologysegment"):
install/tools/ipa-replica-manage:1386:def exit_on_managed_topology(what):

Otherwise the patch works fine.

Martin^3 Babinsky

Manage your subscription for the Freeipa-devel mailing list:
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

Reply via email to