Bug 498920 – net-p2p/transmission USE=-systemd automagically links with libsystemd-daemon.so

• Product: Gentoo Linux
• Status: RESOLVED FIXED

[Description om3i 2014-01-22 13:47:10 UTC]

$ readelf --dynamic /usr/bin/transmission-daemon |grep \\[
0x0000000000000001 (NEEDED) Shared library: [libminiupnpc.so.9]
0x0000000000000001 (NEEDED) Shared library: [libnatpmp.so.1]
0x0000000000000001 (NEEDED) Shared library: [librt.so.1]
0x0000000000000001 (NEEDED) Shared library: [libevent-2.0.so.5]
0x0000000000000001 (NEEDED) Shared library: [libcurl.so.4]
0x0000000000000001 (NEEDED) Shared library: [libcrypto.so.1.0.0]
0x0000000000000001 (NEEDED) Shared library: [libsystemd-daemon.so.0] <<< here
0x0000000000000001 (NEEDED) Shared library: [libz.so.1]
0x0000000000000001 (NEEDED) Shared library: [libpthread.so.0]
0x0000000000000001 (NEEDED) Shared library: [libc.so.6]

$ emerge --pretend transmission

These are the packages that would be merged, in order:

Calculating dependencies... done!
[ebuild   R    ] net-p2p/transmission-2.82-r3  USE="qt4 (-ayatana) -gtk -lightweight -systemd -xfs" 0 kB

Total: 1 package (1 reinstall), Size of downloads: 0 kB

Reproducible: Always

[Comment 1 Samuli Suominen (RETIRED) 2014-01-22 18:43:38 UTC]

Provide complete build.log, and the config.log from econf (./configure) step in /var/tmp/portage/net-p2p/transmission... (use your tab completion, it's in the work/ subdirectory)

I suspect this is a bug in upstream configure.ac, in the $withval handling, propably some quoting problem or problem with []

[Comment 2 om3i 2014-01-22 21:53:36 UTC]

Created attachment 368460 [details]
build log

[Comment 3 om3i 2014-01-22 21:53:53 UTC]

Created attachment 368462 [details]
config log

[Comment 4 Samuli Suominen (RETIRED) 2014-02-07 11:44:05 UTC]

Created attachment 369776 [details, diff]
Fix missing comma that causes automagic libsystemd-daemon linking

this is a case of missing ',' comma char in configure.ac code

proof of case:

.orig is the original unmodified 2.82 and the another is the patched one:

$ objdump -p transmission-2.82/daemon/transmission-daemon |grep systemd
$ objdump -p transmission-2.82.orig/daemon/transmission-daemon |grep systemd
  NEEDED               libsystemd-daemon.so.0
  required from libsystemd-daemon.so.0:

$ grep -i systemd transmission-2.82*/config.log
transmission-2.82/config.log:  $ ./configure --without-systemd-daemon
transmission-2.82/config.log:ac_cv_env_SYSTEMD_DAEMON_CFLAGS_set=
transmission-2.82/config.log:ac_cv_env_SYSTEMD_DAEMON_CFLAGS_value=
transmission-2.82/config.log:ac_cv_env_SYSTEMD_DAEMON_LIBS_set=
transmission-2.82/config.log:ac_cv_env_SYSTEMD_DAEMON_LIBS_value=
transmission-2.82/config.log:SYSTEMD_DAEMON_CFLAGS=''
transmission-2.82/config.log:SYSTEMD_DAEMON_LIBS=''
transmission-2.82.orig/config.log:  $ ./configure --without-systemd-daemon
transmission-2.82.orig/config.log:configure:17979: checking for SYSTEMD_DAEMON
transmission-2.82.orig/config.log:configure:17987: $PKG_CONFIG --exists --print-errors "libsystemd-daemon"
transmission-2.82.orig/config.log:configure:18005: $PKG_CONFIG --exists --print-errors "libsystemd-daemon"
transmission-2.82.orig/config.log:| #define USE_SYSTEMD_DAEMON 1
transmission-2.82.orig/config.log:ac_cv_env_SYSTEMD_DAEMON_CFLAGS_set=
transmission-2.82.orig/config.log:ac_cv_env_SYSTEMD_DAEMON_CFLAGS_value=
transmission-2.82.orig/config.log:ac_cv_env_SYSTEMD_DAEMON_LIBS_set=
transmission-2.82.orig/config.log:ac_cv_env_SYSTEMD_DAEMON_LIBS_value=
transmission-2.82.orig/config.log:pkg_cv_SYSTEMD_DAEMON_CFLAGS=
transmission-2.82.orig/config.log:pkg_cv_SYSTEMD_DAEMON_LIBS='-lsystemd-daemon '
transmission-2.82.orig/config.log:DEFS='-DPACKAGE_NAME=\"transmission\" -DPACKAGE_TARNAME=\"transmission\" -DPACKAGE_VERSION=\"2.82\" -DPACKAGE_STRING=\"transmission\ 2.82\" -DPACKAGE_BUGREPORT=\"http://trac.transmissionbt.com/newticket\" -DPACKAGE_URL=\"\" -DPACKAGE=\"transmission\" -DVERSION=\"2.82\" -DSTDC_HEADERS=1 -DHAVE_SYS_TYPES_H=1 -DHAVE_SYS_STAT_H=1 -DHAVE_STDLIB_H=1 -DHAVE_STRING_H=1 -DHAVE_MEMORY_H=1 -DHAVE_STRINGS_H=1 -DHAVE_INTTYPES_H=1 -DHAVE_STDINT_H=1 -DHAVE_UNISTD_H=1 -DHAVE_DLFCN_H=1 -DLT_OBJDIR=\".libs/\" -DSTDC_HEADERS=1 -DTIME_WITH_SYS_TIME=1 -DHAVE_STDBOOL_H=1 -DHAVE_ICONV_OPEN=1 -DHAVE_PREAD=1 -DHAVE_PWRITE=1 -DHAVE_DAEMON=1 -DHAVE_DIRNAME=1 -DHAVE_BASENAME=1 -DHAVE_STRCASECMP=1 -DHAVE_LOCALTIME_R=1 -DHAVE_FALLOCATE64=1 -DHAVE_POSIX_FALLOCATE=1 -DHAVE_MEMMEM=1 -DHAVE_STRSEP=1 -DHAVE_STRTOLD=1 -DHAVE_SYSLOG=1 -DHAVE_VALLOC=1 -DHAVE_GETPAGESIZE=1 -DHAVE_POSIX_MEMALIGN=1 -DHAVE_STATVFS=1 -DHAVE_MKDTEMP=1 -DHAVE_PTHREAD=1 -DHAVE__TMP_DUMMY1_ZLIB_H=1 -DHAVE_ZLIB=1 -DHAVE_LSEEK64=1 -DHAVE_GETMNTENT=1 -DHAVE_DECL_POSIX_FADVISE=1 -DHAVE_POSIX_FADVISE=1 -DWITH_INOTIFY=1 -DHAVE_SYS_STATVFS_H=1 -DUSE_SYSTEMD_DAEMON=1 -DWITH_UTP=1 -DHAVE_MINIUPNP_16=1 -DSYSTEM_MINIUPNP=1 -DHAVE_LIBINTL_H=1 -DGETTEXT_PACKAGE=\"transmission-gtk\" -DHAVE_LOCALE_H=1 -DHAVE_LC_MESSAGES=1 -DHAVE_BIND_TEXTDOMAIN_CODESET=1 -DHAVE_GETTEXT=1 -DHAVE_DCGETTEXT=1 -DENABLE_NLS=1'
transmission-2.82.orig/config.log:SYSTEMD_DAEMON_CFLAGS=''
transmission-2.82.orig/config.log:SYSTEMD_DAEMON_LIBS='-lsystemd-daemon '
transmission-2.82.orig/config.log:#define USE_SYSTEMD_DAEMON 1

[Comment 5 Samuli Suominen (RETIRED) 2014-02-07 11:44:42 UTC]

the patch needs eautoreconf

[Comment 6 Samuli Suominen (RETIRED) 2014-02-07 11:46:04 UTC]

and the 'proof of case' was after ./configure --without-systemd-daemon and make in each directory

[Comment 7 Samuli Suominen (RETIRED) 2014-02-07 11:56:58 UTC]

Created attachment 369780 [details, diff]
patch to add missing comma

[Comment 8 Samuli Suominen (RETIRED) 2014-02-07 18:42:11 UTC]

reported to upstream w/ a patch

http://trac.transmissionbt.com/ticket/5601

[Comment 9 Samuli Suominen (RETIRED) 2014-05-30 16:03:26 UTC]

ok, 2.83 in portage which contains the patch i submitted to upstream :)