On Tue, 2013-01-08 at 08:53 -0500, Stephen Gallagher wrote:
On Tue 08 Jan 2013 04:01:31 AM EST, Jakub Hrozek wrote:
> On Mon, Jan 07, 2013 at 09:21:13PM -0500, Simo Sorce wrote:
>> They seem unused, they just have a test each.
>> Ok to kill them ?
>
> If they are unused, then kill them. I haven't looked into the git
> history, but I suspect that they've been used at one point and then the
> code that used them was removed.
I'm pretty sure they're dead code. They were originally in there from
the original implementation where we believed a netgroup was fully
representable within a single data source (e.g. LDAP) When we
discovered that we had to handle memberNisNetgroups that might be
coming from a non-SSSD source, we rewrote the design completely. I
suspect those functions just got overlooked.
Ok I'll kill them all.
Patch soon.
Simo.
--
Simo Sorce * Red Hat, Inc * New York