On Thu, 04 Oct 2012, Sumit Bose wrote:
Hi,

this patch tries to avoid the ldapmodiy error messages during
ipa-adtrust-install by checking if the related object already exists.
Fixes https://fedorahosted.org/freeipa/ticket/3012 .
In general -- ACK for the approach. However, I wonder if you could create
a helper method that would accept:
  - ldif file name,
  - cn component
  - name of the plugin for the "already configured" message

Then every __add_* method would call simply the helper with appropriate
arguments.

--
/ Alexander Bokovoy

_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

Reply via email to