[Freeipa-devel] [PATCH] 901 remove has_upg() and has_managed_entries()

[Rob Crittenden]

Martin Kosek wrote:
> On Tue, 2011-11-08 at 14:49 -0500, Rob Crittenden wrote:
>> These functions are leftovers from when the managed entries plugin was
>> being developed and not widely available. They are no longer needed.
>>
>> rob
>
> NACK.
>
> has_upg() does not check if the managed entry plugin present in dirsrv
> but rather if the UPG rule is enabled. If the UPG rule is not enabled,
> user gidNumber is set to ipausers' GID.
>
> has_upg() behavior was lately being fixed in:
> https://fedorahosted.org/freeipa/ticket/1964
>
> Martin
>

This is one case I should have listened to the little voice in my head.

Revised patch.

rob
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-rcrit-901-2-managed.patch
Type: text/x-patch
Size: 3269 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20111110/2b0e52d6/attachment.bin>