[PATCH] add native systemd init file rhbz#617316

[Jiri Moskovcak]

This patch adds the abrtd.service file to abrt package and ensures it's
installed when systemd is available. The systemd is automatically
detected by autotools, so it shouldn't affect systems without systemd.
Please note that to test it and build on rawhide/F14 you have to change
the line in spe file:
%define with_systemd 0
to
%define with_systemd 1

Please review,
Jirka

[Karel Klic]

I did a test build for F-14 and it contains the systemd config file. 
F-12 build still works as expected. The code changes look reasonable.

Karel

[Nikola Pajkovsky]

On 08/06/2010 06:56 PM, Jiri Moskovcak wrote:
> This patch adds the abrtd.service file to abrt package and ensures it's
> installed when systemd is available. The systemd is automatically
> detected by autotools, so it shouldn't affect systems without systemd.
> Please note that to test it and build on rawhide/F14 you have to change
> the line in spe file:
> %define with_systemd 0
> to
> %define with_systemd 1
>
> Please review,
> Jirka

No, it can be done automatically.

--with-systemdsystemunitdir looooooong. I'm proposing --with-systemd

[Jiri Moskovcak]

On 08/07/2010 10:00 AM, Nikola Pajkovsky wrote:
> On 08/06/2010 06:56 PM, Jiri Moskovcak wrote:
> > This patch adds the abrtd.service file to abrt package and ensures it's
> > installed when systemd is available. The systemd is automatically
> > detected by autotools, so it shouldn't affect systems without systemd.
> > Please note that to test it and build on rawhide/F14 you have to change
> > the line in spe file:
> > %define with_systemd 0
> > to
> > %define with_systemd 1
> >
> > Please review,
> > Jirka
>
> No, it can be done automatically.

yes, Karel already pointed that out, so it's the logic in the spec file
is now:

%if 0%{?fedora} >= 14
    %define with_systemd 1
%else
    %define with_systemd 0
%endif
> --with-systemdsystemunitdir looooooong. I'm proposing --with-systemd

this is actually from the guide
https://fedoraproject.org/wiki/User:Johannbg/QA/Systemd/Daemon#Installing_Sy...
so I think we should follow the guidelines, so we don't differ from
other projects and the option name is the same.

J.