Re: [PATCH v2] build: Add support for systemd service configuration

[Marcel Holtmann <marcel@holtmann.org>]

Hi Lucas,

> >> >> -DISTCHECK_CONFIGURE_FLAGS = --disable-udevrules
> >> >> +DISTCHECK_CONFIGURE_FLAGS = --disable-udevrules --with-systemdunitdir=
> >> >
> >> > can we please do it the same way I did it for oFono and ConnMan. I have
> >> > no idea what you are doing here. If --with-systemdunitdir is not given,
> >> > then it should be disabled and it should be that by default.
> >>
> >> This is indeed the same code as in oFono and ConnMann. If
> >> `--with-systemdunitdir' is not given it defaults to checking
> >> `pkg-config --variable systemdsystemunitdir systemd'. So in case we
> >> call `make distcheck' and systemd is installed, this will fail, as it
> >> will in oFono and ConnMann. The difference is that there we have
> >> `DISTCHECK_CONFIGURE_FLAGS = --disable-datafiles'. I may do the same
> >> and let `DISTCHECK_CONFIGURE_FLAGS = --disable-configfiles'. What do
> >> you think? Then I think I could remove the `--disable-udevrules' as
> >> well.
> >
> > lets do the same as we have done in ConnMan and oFono.
> >
> > We might just rename --disable-configfiles into --disable-datafiles to
> > bring all three projects in sync here.
> >
> > The --disable-udevrules has to stay since otherwise we would install
> > into a system directory. However we can put that under the DATAFILES
> > conditional.
> >
> > Care to send a new set of patches?
> 
> Ok.
> 
> Bastien, are you going to do it or I can do it myself?

just send me an updated patch. I wanna get this merged upstream. I was
under the impression this got already fixed, but I was wrong.

Regards

Marcel