Re: [Freeipa-devel] [PATCH] Added try/except for error handling ipautil

2015-08-19 Thread Martin Basti
On 08/19/2015 01:49 PM, Abhijeet Kasurde wrote: Hi All, Please find the latest patch with review comments included. Thanks Martin for your help and review comments. Thanks, Abhijeet Kasurde On 08/19/2015 05:08 PM, Martin Basti wrote: On 08/17/2015 02:08 PM, Abhijeet Kasurde wrote: Hi

Re: [Freeipa-devel] [PATCH] Added try/except for error handling ipautil

2015-08-19 Thread Abhijeet Kasurde
Hi All, Please find the latest patch with review comments included. Thanks Martin for your help and review comments. Thanks, Abhijeet Kasurde On 08/19/2015 05:08 PM, Martin Basti wrote: On 08/17/2015 02:08 PM, Abhijeet Kasurde wrote: Hi All, I have included all review comments. Please

Re: [Freeipa-devel] [PATCH] Added try/except for error handling ipautil

2015-08-17 Thread Abhijeet Kasurde
Hi All, I have included all review comments. Please let me know your views. On 08/17/2015 04:09 PM, Martin Basti wrote: On 08/17/2015 11:11 AM, Abhijeet Kasurde wrote: Hi All, Please find the update patch with review comments, On 08/14/2015 05:19 PM, Martin Basti wrote: On 08/14/2015

Re: [Freeipa-devel] [PATCH] Added try/except for error handling ipautil

2015-08-17 Thread Abhijeet Kasurde
Hi All, Please find the update patch with review comments, On 08/14/2015 05:19 PM, Martin Basti wrote: On 08/14/2015 06:57 AM, Abhijeet Kasurde wrote: On 08/13/2015 07:08 PM, Martin Basti wrote: On 08/10/2015 01:47 PM, Abhijeet Kasurde wrote: Hi All, This patch fixes bug -

Re: [Freeipa-devel] [PATCH] Added try/except for error handling ipautil

2015-08-17 Thread Martin Basti
On 08/17/2015 11:11 AM, Abhijeet Kasurde wrote: Hi All, Please find the update patch with review comments, On 08/14/2015 05:19 PM, Martin Basti wrote: On 08/14/2015 06:57 AM, Abhijeet Kasurde wrote: On 08/13/2015 07:08 PM, Martin Basti wrote: On 08/10/2015 01:47 PM, Abhijeet Kasurde

Re: [Freeipa-devel] [PATCH] Added try/except for error handling ipautil

2015-08-14 Thread Martin Basti
On 08/14/2015 06:57 AM, Abhijeet Kasurde wrote: On 08/13/2015 07:08 PM, Martin Basti wrote: On 08/10/2015 01:47 PM, Abhijeet Kasurde wrote: Hi All, This patch fixes bug - https://fedorahosted.org/freeipa/ticket/3406 Thanks, Abhijeet Kasurde Hello, thank you for the patch 1) -

Re: [Freeipa-devel] [PATCH] Added try/except for error handling ipautil

2015-08-13 Thread Abhijeet Kasurde
On 08/13/2015 07:08 PM, Martin Basti wrote: On 08/10/2015 01:47 PM, Abhijeet Kasurde wrote: Hi All, This patch fixes bug - https://fedorahosted.org/freeipa/ticket/3406 Thanks, Abhijeet Kasurde Hello, thank you for the patch 1) -except ValueError: +except