Bug#877512: slapd: enabled systemd integration (untested patch)

[Quanah Gibson-Mount]

--On Thursday, June 29, 2023 12:27 PM +0200 Andreas Henriksson 
<andreas at fatal.se> wrote:

>> Feel free to file a bug upstream if you think the current configure.ac
>> code needs adjustment.
> [...]
>
> It's my impression that configure.ac is missing a call to:
>
> PKG_PROG_PKG_CONFIG(0.29)
>
> Thus the PKG_CONFIG variable will be unset, and thus the PKG_CHECK_*
> macros will just skip over and do nothing.
>
>
> FWIW you do have this:
> m4_ifndef([PKG_PREREQ],
>         [m4_fatal([must install pkg-config 0.29 or later before running
> 	autoconf/autogen])])
> ... but that only seems to check that pkg.m4 is new enough, not that
> the actual pkg-config binary/utility exists.
>
> Adding `PKG_PROG_PKG_CONFIG(0.29)` directly after the m4_ifndef and
> rebuilding gave me the expected systemdsystemunitdir=/lib/systemd/system
> (as systemd.pc says on debian) rather than the hardcoded fallbacks.

Please file a bug at https://bugs.openldap.org :)

Regards,
Quanah