[PATCH] Remove loadparm dependency from dbwrap

Volker Lendecke Volker.Lendecke at SerNet.DE
Wed Apr 18 06:05:20 UTC 2018


On Wed, Apr 18, 2018 at 08:00:09AM +0200, Swen Schillig wrote:
> Hi Volker
> On Wed, 2018-04-18 at 07:54 +0200, Volker Lendecke via samba-technical
> wrote:
> > Hi!
> > 
> > $SUBJECT says it all. If requested, we can squash a few of the
> > patches.
> > 
> > Review appreciated!
> > 
> Would it make sense to create a define for the default permission
> "0600" and remove the magic number ?

Possibly. But that's not the focus of this patchset. The focus of this
patchset is to remove the calls to lpctx_* from lib/dbwrap/. To have
the patchset exactly on this topic, I tried to avoid changing other
parameters.

If you want to also change the default permission specification, feel
free to propose a separate patchset.

Volker

-- 
SerNet GmbH, Bahnhofsallee 1b, 37081 Göttingen
phone: +49-551-370000-0, fax: +49-551-370000-9
AG Göttingen, HRB 2816, GF: Dr. Johannes Loxen
http://www.sernet.de, mailto:kontakt at sernet.de



More information about the samba-technical mailing list