Re: New NIS+ patch for CVS 16.10.01

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 2001.10.17 07:23 Andrew Morgan wrote:
> http://sourceforge.net/tracker/index.php?func=detail&aid=414661&group_id=6663&atid=106663
> 
> It pains me to do this, but I am going to have to reject this patch. Its
> too big a change for me to feel comfortable with, especially as I don't
> have access to a testing environment.
> 
> Please read my comments in the above bug report and consider making
> things a little easier for me if you want to pursue this.
> 
> Thanks
> 
> Andrew
> 

Hi,

sorry for my late response. I had a look on you remarks. Basically i totaly
agree with your 
proposals. pam_unix is too import for the folk so it should work properly.
Its a good idea to split the patch in 2 parts. 
But I think we should discuss some parts of the rearangement of the
existing code. 
I thinks its a good idea to have one function filling out the user_t
structure 

This function have to be splittet in two parts: 
   * one getting the informations with nss
   * one using the old implemnted style

Additionally 









[Index of Archives]     [Fedora Users]     [Kernel]     [Red Hat Install]     [Linux for the blind]     [Gimp]

  Powered by Linux