diff mbox series

package/cryptsetup: fix tmpfiles.d setup

Message ID 20210614104210.2100899-1-john@metanate.com
State Accepted
Headers show
Series package/cryptsetup: fix tmpfiles.d setup | expand

Commit Message

John Keeping June 14, 2021, 10:42 a.m. UTC
The configure script uses pkg-config to detect the location of
tmpfiles.d but imposes an unspecified ordering dependency with systemd.
Instead of relying on systemd being built before cryptsetup, set the
directory path explcitly, and ensure it is not set when systemd-tmpfiles
is disabled.

Signed-off-by: John Keeping <john@metanate.com>
---
 package/cryptsetup/cryptsetup.mk | 6 ++++++
 1 file changed, 6 insertions(+)

Comments

Thomas Petazzoni July 20, 2021, 9:49 p.m. UTC | #1
On Mon, 14 Jun 2021 11:42:10 +0100
John Keeping <john@metanate.com> wrote:

> The configure script uses pkg-config to detect the location of
> tmpfiles.d but imposes an unspecified ordering dependency with systemd.
> Instead of relying on systemd being built before cryptsetup, set the
> directory path explcitly, and ensure it is not set when systemd-tmpfiles
> is disabled.
> 
> Signed-off-by: John Keeping <john@metanate.com>
> ---
>  package/cryptsetup/cryptsetup.mk | 6 ++++++
>  1 file changed, 6 insertions(+)

Applied to master, thanks.

Thomas
Peter Korsgaard Aug. 4, 2021, 3:17 p.m. UTC | #2
>>>>> "John" == John Keeping <john@metanate.com> writes:

 > The configure script uses pkg-config to detect the location of
 > tmpfiles.d but imposes an unspecified ordering dependency with systemd.
 > Instead of relying on systemd being built before cryptsetup, set the
 > directory path explcitly, and ensure it is not set when systemd-tmpfiles
 > is disabled.

 > Signed-off-by: John Keeping <john@metanate.com>

Committed to 2021.02.x and 2021.05.x, thanks.
diff mbox series

Patch

diff --git a/package/cryptsetup/cryptsetup.mk b/package/cryptsetup/cryptsetup.mk
index cd5883d0b4..eee84a7958 100644
--- a/package/cryptsetup/cryptsetup.mk
+++ b/package/cryptsetup/cryptsetup.mk
@@ -33,6 +33,12 @@  else
 CRYPTSETUP_CONF_OPTS += --with-crypto_backend=kernel
 endif
 
+ifeq ($(BR2_PACKAGE_SYSTEMD_TMPFILES),y)
+CRYPTSETUP_CONF_OPTS += --with-tmpfilesdir=/usr/lib/tmpfiles.d
+else
+CRYPTSETUP_CONF_OPTS += --with-tmpfilesdir=no
+endif
+
 HOST_CRYPTSETUP_DEPENDENCIES = \
 	host-pkgconf \
 	host-lvm2 \