Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [git commit] util-linux: disable systemd support
@ 2015-10-22 20:42 Peter Korsgaard
  2015-10-22 20:48 ` Waldemar Brodkorb
  0 siblings, 1 reply; 3+ messages in thread
From: Peter Korsgaard @ 2015-10-22 20:42 UTC (permalink / raw)
  To: buildroot

commit: http://git.buildroot.net/buildroot/commit/?id=762b64daa2fd1424053754ed55d0f9704ccc2405
branch: http://git.buildroot.net/buildroot/commit/?id=refs/heads/master

Fixes (works around) #8421

Util-linux has optional systemd support, but we cannot enable it as systemd
depends on util-linux, so that would create a circular dependency.

The systemd unit directory location detection also fails as pkg-config
0.9.12+ prefixes all directory variables with the sysroot, and the configure
script expects to see the target location, so disable that as well for good
measure.

The systemd support doesn't seem to add much, so it isn't a big loss.

Signed-off-by: Peter Korsgaard <peter@korsgaard.com>
---
 package/util-linux/util-linux.mk |    6 ++++++
 1 files changed, 6 insertions(+), 0 deletions(-)

diff --git a/package/util-linux/util-linux.mk b/package/util-linux/util-linux.mk
index 27d22cb..39ba544 100644
--- a/package/util-linux/util-linux.mk
+++ b/package/util-linux/util-linux.mk
@@ -26,6 +26,12 @@ UTIL_LINUX_CONF_OPTS += \
 	--disable-makeinstall-chown \
 	--without-python
 
+# system depends on util-linux so we enable systemd support
+# (which needs systemd to be installed)
+UTIL_LINUX_CONF_OPTS += \
+	--without-systed \
+	--with-systemdsystemunitdir=no
+
 # We don't want the host-busybox dependency to be added automatically
 HOST_UTIL_LINUX_DEPENDENCIES = host-pkgconf
 

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* [Buildroot] [git commit] util-linux: disable systemd support
  2015-10-22 20:42 [Buildroot] [git commit] util-linux: disable systemd support Peter Korsgaard
@ 2015-10-22 20:48 ` Waldemar Brodkorb
  2015-10-22 20:55   ` Peter Korsgaard
  0 siblings, 1 reply; 3+ messages in thread
From: Waldemar Brodkorb @ 2015-10-22 20:48 UTC (permalink / raw)
  To: buildroot

Hi Peter,
Peter Korsgaard wrote,

> commit: http://git.buildroot.net/buildroot/commit/?id=762b64daa2fd1424053754ed55d0f9704ccc2405
> branch: http://git.buildroot.net/buildroot/commit/?id=refs/heads/master
> 
> Fixes (works around) #8421
> 
> Util-linux has optional systemd support, but we cannot enable it as systemd
> depends on util-linux, so that would create a circular dependency.
> 
> The systemd unit directory location detection also fails as pkg-config
> 0.9.12+ prefixes all directory variables with the sysroot, and the configure
> script expects to see the target location, so disable that as well for good
> measure.
> 
> The systemd support doesn't seem to add much, so it isn't a big loss.
> 
> Signed-off-by: Peter Korsgaard <peter@korsgaard.com>
> ---
>  package/util-linux/util-linux.mk |    6 ++++++
>  1 files changed, 6 insertions(+), 0 deletions(-)
> 
> diff --git a/package/util-linux/util-linux.mk b/package/util-linux/util-linux.mk
> index 27d22cb..39ba544 100644
> --- a/package/util-linux/util-linux.mk
> +++ b/package/util-linux/util-linux.mk
> @@ -26,6 +26,12 @@ UTIL_LINUX_CONF_OPTS += \
>  	--disable-makeinstall-chown \
>  	--without-python
>  
> +# system depends on util-linux so we enable systemd support
> +# (which needs systemd to be installed)
> +UTIL_LINUX_CONF_OPTS += \
> +	--without-systed \

I think this must be a typo:
$ ./configure --help|grep systemd
                          systemd)
  --with-systemd          build with support for systemd


best regards
 Waldemar

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [Buildroot] [git commit] util-linux: disable systemd support
  2015-10-22 20:48 ` Waldemar Brodkorb
@ 2015-10-22 20:55   ` Peter Korsgaard
  0 siblings, 0 replies; 3+ messages in thread
From: Peter Korsgaard @ 2015-10-22 20:55 UTC (permalink / raw)
  To: buildroot

>>>>> "Waldemar" == Waldemar Brodkorb <wbx@openadk.org> writes:

Hi,

>> +# system depends on util-linux so we enable systemd support
 >> +# (which needs systemd to be installed)
 >> +UTIL_LINUX_CONF_OPTS += \
 >> +	--without-systed \

 > I think this must be a typo:
 > $ ./configure --help|grep systemd
 >                           systemd)
 >   --with-systemd          build with support for systemd

Argh, indeed. I moved the lines further down and reworded the comment
just before committing, and apparently fat fingered it :/

Fixed in git, thanks!

-- 
Venlig hilsen,
Peter Korsgaard 

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2015-10-22 20:55 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-10-22 20:42 [Buildroot] [git commit] util-linux: disable systemd support Peter Korsgaard
2015-10-22 20:48 ` Waldemar Brodkorb
2015-10-22 20:55   ` Peter Korsgaard

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox