Le lundi 27 juin 2011 à 14:24 +0200, Lennart Poettering a écrit :
> On Fri, 24.06.11 15:08, Frederic Crozat (fcro...@suse.com) wrote:
> 
> Heya,
> 
> Sounds good! A few comments:
>  
> > %service_add()
> > if [ "$1" -eq 1 ] ; then 
> > # Initial installation 
> > /bin/systemctl daemon-reload >/dev/null 2>&1 || :
> > fi
> > 
> > %service_add_enabled()
> > if [ "$1" -eq 1 ] ; then 
> > # Initial installation
> > /bin/systemctl enable %{1}.service >/dev/null 2>&1 || :
> > fi
> 
> I'd strongly advise against this. Units might install more than one
> service, and other units instead of services (example: sshd might just
> enable a socket by default, but not a service). Hence we should not
> appaned .service but leave it to the package to specifiy the full name
> and we also need to make sure packagers can specify multiple unit files
> at once.

This question was also spotted on opensuse-packaging and we thought it
should be discussed here :)

So, usage would become :

%post 
%service_add ssh.socket

%post
%service_add_enabled foobar.service 

?

> Bill, what's your take on this? For some reason we solve very little
> with macros like this on Fedora. Not sure why. Should I push for
> including macros like this in rpm, or in the systemd packages?

If we could get some skeleton directly in "upstream" rpm, that would be
awesome ;)

-- 
Frederic Crozat <fcro...@suse.com>
SUSE

_______________________________________________
systemd-devel mailing list
systemd-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/systemd-devel

Reply via email to