Dne 10.7.2015 v 10:59 Jan Cholasta napsal(a):
Dne 10.7.2015 v 10:43 Martin Basti napsal(a):
On 10/07/15 07:29, Jan Cholasta wrote:

Dne 9.7.2015 v 17:21 Martin Basti napsal(a):

Patch attached.

NACK, you should remove the --rename option from certprofile-mod. You
can do it by removing "rdn_is_primary_key = True" from certprofile.


Updated patch attached.

What I meant was remove --rename *and* do the check from your previous

Anyway, I didn't realize we already released IPA with certprofile and
removing --rename would be a backward incompatible change, so I think
it's better to just keep it.

So ACK on the original patch.

Pushed to master: 67b2b3408579814f7ff307cfd20bc4250edbea15

Jan Cholasta

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

Reply via email to