On 15/04/16 15:42, Martin Basti wrote:



On 15.04.2016 15:03, Petr Vobornik wrote:
On 04/15/2016 01:25 PM, Martin Basti wrote:

On 15.04.2016 13:18, David Kupka wrote:
On 14/04/16 17:47, Martin Basti wrote:
https://fedorahosted.org/freeipa/ticket/5774

Patch attached.


Hi,
works for me, ACK.

FYI: While trying this I've hit related issue
(https://fedorahosted.org/freeipa/ticket/5822) but I believe the
impact is minimal.
Pushed to master: 93871bf017e1bc5f2c176aa3419278d49fcc003b

API.txt and version was not updated:

-option: Int('ipamaxusernamelength', attribute=True, autofill=False,
cli_name='maxusername', minvalue=1, multivalue=False, required=False)
+option: Int('ipamaxusernamelength', attribute=True, autofill=False,
cli_name='maxusername', maxvalue=255, minvalue=1, multivalue=False,
required=False)

Sorry.

Patch attached.

Sorry, I haven't noticed this. I remember that previously this type of errors aborted build. Why was this changed?

Anyway, the patch fixes it, ACK.


--
David Kupka

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

Reply via email to