On 12/10/2013 03:23 PM, Jan Cholasta wrote:
On 10.12.2013 15:18, Petr Viktorin wrote:
On 12/10/2013 02:15 PM, Jan Cholasta wrote:
On 10.12.2013 12:18, Petr Viktorin wrote:
On 12/05/2013 02:45 PM, Jan Cholasta wrote:

the attached patch fixes


Patch looks good, ACK.

I've added a small regression test for this, does it look OK?

Thanks, it looks OK except I don't see "dn" in result and I would rename
"isodate_re" to "generalizedtime_re".

Your patch adds "dn".

Oh, right.

user_status without --raw will report time in ISO 8601
(%Y-%m-%dT%H:%M:%SZ). GeneralizedTime would be "%Y%m%d%H%M%SZ".

Also right.

Sorry for the fuss then, ACK.

Thanks, pushed both to master: b6563984154e577cdf430f8f74f15f912ac0ee12


Freeipa-devel mailing list

Reply via email to