Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 501646

Summary: sys-apps/systemd force it to only need intltool for live builds
Product: Gentoo Linux Reporter: thomasg <thomas>
Component: [OLD] Core systemAssignee: Gentoo systemd Team <systemd>
Status: RESOLVED OBSOLETE    
Severity: enhancement    
Priority: Normal    
Version: unspecified   
Hardware: All   
OS: Linux   
See Also: https://bugs.gentoo.org/show_bug.cgi?id=443028
Whiteboard:
Package list:
Runtime testing required: ---
Attachments: systemd_intltool_dependency.patch
systemd_live_dependencies.patch

Description thomasg 2014-02-18 00:24:57 UTC
sys-apps/systemd has a dependency on intltool, which is only needed for live/git builds, not for release tarballs.
intltool is used in the autotools eclass when eautoreconf is called, which only happens for the live ebuild anyway.
Fixing this will reduce unnecessary dependencies.

Reproducible: Always
Comment 1 thomasg 2014-02-18 00:26:28 UTC
Created attachment 370682 [details, diff]
systemd_intltool_dependency.patch

This patch fixes the aforementioned problem.
It has been created against the sys-apps/systemd-9999.ebuild, but should apply (and be applied) to the other systemd ebuilds as well.
Comment 2 thomasg 2014-02-18 00:45:15 UTC
Created attachment 370684 [details, diff]
systemd_live_dependencies.patch

Pardon the spam, I just noticed, that this issue applies to multiple dependencies.
I enhanced the patch to include them as well.

Additionally, the gtk-docs dependency appears also to be a live dependency; however, this would require more intrusive changes to the ebuild (because gtk-doctize is called manually), which I'm not ready to propose right now.
Comment 3 Samuli Suominen (RETIRED) gentoo-dev 2014-02-18 08:19:45 UTC
Are you sure? sys-fs/udev has elaborate hack to avoid intltool/gettext dependency:

# Remove requirements for gettext and intltool wrt bug #443028
if ! has_version dev-util/intltool && ! [[ ${PV} = 9999* ]]; then
  sed -i \
      -e '/INTLTOOL_APPLIED_VERSION=/s:=.*:=0.40.0:' \
      -e '/XML::Parser perl module is required for intltool/s|^|:|' \
      configure || die
  eval export INTLTOOL_{EXTRACT,MERGE,UPDATE}=/bin/true
  eval export {MSG{FMT,MERGE},XGETTEXT}=/bin/true
fi

So how is this solved by just moving deps in systemd when it's the same tarball?!
Comment 4 thomasg 2014-02-18 11:54:39 UTC
Hi Samuli,

you're right, I looked into it and intltool is indeed a hard builttime dependency right now.
I talked to upstream, this is very likely a buildsystem bug, but I don't know yet how hard it is to fix it (intltool should be used exclusively at make dist).
At the very least it will take until the next systemd release anyway.

So we can't really build without intltool right know, I'll look into it to get it fixed upstream, until that, we probably should close this bug for know and I can reopen it when things move forward.
Implementing a workaround like in udev could probably be done, but isn't a solution I'd prefer.

Sorry for the inconvenience.
Comment 5 Pacho Ramos gentoo-dev 2015-09-07 18:11:01 UTC
This is obsolete now as upstream don't do tarballs anymore