[Freeipa-devel] [PATCH] move LOCAL auth into a separate backend

[Sumit Bose]

Stephen Gallagher schrieb:
> 
> I'd like to see the individual back-ends stored in their own directories
> (keeping the trend set by the LDAP back-end). Can you move the
> LOCAL_auth backend into a "LOCAL" subdirectory?
> 
> Similarly, (though probably in a separate patch) I'd like to see the
> proxy moved down a level as well.
> 
> Basically I want it to be easy and less cluttered if we decide to add
> back-end-specific helper files, so everything can be kept in one place.
> 
> On to the review:
> It's only in the comments, but the header for LOCAL_auth.c reads:
> "PAM e credentials"
> I'm not sure what you meant to write there. Maybe "Native LOCAL Backend"
> would be accurate.
> 
> Other than that, I'd say it looks pretty good to me.
> 
valid points, new version attached

bye,
Sumit
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-move-LOCAL-auth-into-a-separate-backend.patch
Type: text/x-patch
Size: 34509 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20090529/660ff213/attachment.bin>