On Sun, Mar 23, 2003 at 06:15:54PM +1100, Andrew Bartlett wrote: > On Sun, 2003-03-23 at 18:03, Volker Lendecke wrote: > > -----BEGIN PGP SIGNED MESSAGE----- > > Hash: SHA1 > > > > > Sorry to nitpick - but is this parameter name already is use in the SUSE > > > patch? Or could we change it without breaking existing users? > > > > Hmm. I'm not happy with that name either. But SuSE chose it. > > I love our commitment to compatibility ;-) > > > > 'ldap delete only samba attributes' might be a bit long-winded, but I'm > > > worried with that many contractions it starts to get unclear. > > > > tng uses 'ldap delete dn' which is opposite logic but a lot shorter. I > > think I like that better. What do you think about aliasing them and > > saying 'deprecated'? > > You would need to put a 'handle...' helper in loadparm.c to handle the > inverse, but I think it's worth the mess. It's is much clearer about > what it does. Go for it.
Just rename 'ldap del only sam attr' if you don't like it. It's not worth the mess. We use this in one of our products and I know these guys well. So change it, inform me about the change and we know what to do. -- Lars M�ller SuSE Linux AG, Deutschherrnstra�e 15-19, 90429 N�rnberg, Germany
pgp00000.pgp
Description: PGP signature
