Re: [Freeipa-devel] [PATCH 0076] Fix incompatible DNS permission

2014-06-25 Thread Petr Viktorin
On 06/25/2014 05:03 PM, Martin Basti wrote: On Wed, 2014-06-25 at 15:54 +0200, Petr Viktorin wrote: On 06/20/2014 03:28 PM, Martin Basti wrote: Patch attached. Ticket:https://fedorahosted.org/freeipa/ticket/4383 This works, just two comments: To check if an entry exists, instead of calling

Re: [Freeipa-devel] [PATCH 0076] Fix incompatible DNS permission

2014-06-25 Thread Martin Basti
On Wed, 2014-06-25 at 15:54 +0200, Petr Viktorin wrote: > On 06/20/2014 03:28 PM, Martin Basti wrote: > > Patch attached. > > > > Ticket:https://fedorahosted.org/freeipa/ticket/4383 > > This works, just two comments: > > To check if an entry exists, instead of calling > api.Command['permissi

Re: [Freeipa-devel] [PATCH 0076] Fix incompatible DNS permission

2014-06-25 Thread Petr Viktorin
On 06/20/2014 03:28 PM, Martin Basti wrote: Patch attached. Ticket:https://fedorahosted.org/freeipa/ticket/4383 This works, just two comments: To check if an entry exists, instead of calling api.Command['permission_show'](permission_name_rel) you should call the more light-weight api.

[Freeipa-devel] [PATCH 0076] Fix incompatible DNS permission

2014-06-20 Thread Martin Basti
Patch attached. Ticket:https://fedorahosted.org/freeipa/ticket/4383 -- Martin^2 Basti >From a01f6f623e7cf9261fa0029f271f8a310812f895 Mon Sep 17 00:00:00 2001 From: Martin Basti Date: Fri, 20 Jun 2014 13:52:12 +0200 Subject: [PATCH] Fix incompatible DNS permission dns(forward)zone-add/remove-per