[PATCH] turn packaging files into template, preprocess and install them

Andrew Bartlett abartlet at samba.org
Mon Dec 18 08:05:31 UTC 2017


On Thu, 2017-12-14 at 17:17 +0100, Aurélien Aptel via samba-technical
wrote:
> Hi,
> 
> This patch adds a bunch of systemd options to the configure script to
> allow installing the systemd files in packaging/.
> 
> The service files are turned into configure-style templates with @VAR@
> substitution and now always use proper values as a result.
> 
> The --systemd-{nmb,smb,winbind,samba}-extra options can be used to add
> vendor specific directives to the respective service file. These options
> can be used multiple times to add multiple directives.
> 
> The service files are always preprocessed but only installed if
> --systemd-install-services is given. The
> location where the service
> files are installed can be changed with the new option
> --with-systemddir.

Can we have this three parts, the --with-systemdir change, the generic
subs and the -extra stuff?  

Also, is it really work having an --systemd-install-services option?  

It is just an extra file on systems that don't use systemd, and these
option combinations just end up untested.  Software packages not using
systemd will just not package the file, which is fine, and manual
installs will just get a file somewhere under /usr/local/samba unless
they set --with-systemddir

I like the normal subs, because we already have that info, but the
'extra' stuff feels odd to me, isn't that why distributions have
patches they apply?  

Thanks,

Andrew Bartlett

-- 
Andrew Bartlett                       http://samba.org/~abartlet/
Authentication Developer, Samba Team  http://samba.org
Samba Developer, Catalyst IT          http://catalyst.net.nz/services/samba




More information about the samba-technical mailing list