CVS update: samba/source/param

Lars Mueller lmuelle at SuSE.de
Mon Mar 24 04:05:45 EST 2003


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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
Url : http://lists.samba.org/archive/samba-cvs/attachments/20030323/746d6200/attachment.bin


More information about the samba-cvs mailing list