Re: [Freeipa-devel] [PATCH] 211 Fix internal error in the user-status command

2013-12-10 Thread Petr Viktorin
On 12/05/2013 02:45 PM, Jan Cholasta wrote: Hi, the attached patch fixes https://fedorahosted.org/freeipa/ticket/4066. Honza Patch looks good, ACK. I've added a small regression test for this, does it look OK? -- Petr³ From bb333401aa551f84a16192d2d42de20bd5633f84 Mon Sep 17 00:00:00 2001

Re: [Freeipa-devel] [PATCH] 211 Fix internal error in the user-status command

2013-12-10 Thread Jan Cholasta
On 10.12.2013 12:18, Petr Viktorin wrote: On 12/05/2013 02:45 PM, Jan Cholasta wrote: Hi, the attached patch fixes https://fedorahosted.org/freeipa/ticket/4066. Honza 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

Re: [Freeipa-devel] [PATCH] 211 Fix internal error in the user-status command

2013-12-10 Thread Petr Viktorin
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: Hi, the attached patch fixes https://fedorahosted.org/freeipa/ticket/4066. Honza Patch looks good, ACK. I've added a small regression test for this, does it

Re: [Freeipa-devel] [PATCH] 211 Fix internal error in the user-status command

2013-12-10 Thread Jan Cholasta
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: Hi, the attached patch fixes https://fedorahosted.org/freeipa/ticket/4066. Honza Patch looks good, ACK. I've added a

Re: [Freeipa-devel] [PATCH] 211 Fix internal error in the user-status command

2013-12-10 Thread Petr Viktorin
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: Hi, the attached patch fixes https://fedorahosted.org/freeipa/ticket/4066.

[Freeipa-devel] [PATCH] 211 Fix internal error in the user-status command

2013-12-05 Thread Jan Cholasta
Hi, the attached patch fixes https://fedorahosted.org/freeipa/ticket/4066. Honza -- Jan Cholasta From 751ba817f04ba189b2000e0d0f02611ac62243cb Mon Sep 17 00:00:00 2001 From: Jan Cholasta jchol...@redhat.com Date: Thu, 5 Dec 2013 14:43:19 +0100 Subject: [PATCH] Fix internal error in the