[PROPOSAL] Remove BOOLREV (enable spoolss, writable, writeable) parameters

[Jelmer Vernooij]

On Sat, Feb 15, 2014 at 03:26:20PM +1300, Andrew Bartlett wrote:
> On Fri, 2014-02-14 at 21:28 +0100, Michael Adam wrote:
> > On 2014-02-14 at 18:10 +0100, Andreas Schneider wrote:
> > > On Tuesday 14 January 2014 17:30:46 Andrew Bartlett wrote:
> > > > While not required by any work being done right now, looking forward to
> > > > a consistent loadparm in the future, I think the job would be easier if
> > > > we didn't have BOOLREV synonyms.
> > > > 
> > > > Could we deprecate these parameters for 4.2, and remove them for 4.3?
> > > > 
> > > > It would be safe (in a security sense) for upgrades to 4.3, because if
> > > > they were still in use, such a share would just become read only, or at
> > > > worst spoolss would be enabled.  In the long term, I think it would be
> > > > easier to explain our syntax to our users without this 'feature'.
> > > > 
> > > > What do folks think?
> > > 
> > > You mean:
> > > 
> > > wurst = true
> > > wurst = True
> > > wurst = yes
> > > wurst = Yes
> > 
> > This is not about the different ways we offer to
> > specify a boolean value.
> > 
> > What Andrew is referring to here is the concept
> > of having REVERSE boolean synonyms like
> > 
> > writeable  <-> read only
> > 
> > 
> > Regarding Andrew's question, I am not certain.
> > The inverse synonyms are quite convenient at times,
> > and people are used to it, so what more precisely would
> > we gain by removing them?
> 
> We don't gain much really, mostly a little code simplicity.  They are
> pretty weird, but since I posted that I've found much more that is even
> weirder that we need to keep.
> 
> In this case, what I put was more a question than a proposal and as long
> as someone feels they are convenient I'll let it rest there.

It would also make reading different smb.conf files less confusing,
and you can more easily spot duplicate entries.

Cheers,

Jelmer
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20140215/b6b0a4f5/attachment.pgp>