On Thu, 04 Oct 2012, Sumit Bose wrote:

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

/ Alexander Bokovoy

Freeipa-devel mailing list

Reply via email to