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