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

Bug 759526

Summary: net-misc/chrony-4.0 missing IUSE="gnutls" and DEPEND="gnutls? ( net-libs/gnutls:= )"
Product: Gentoo Linux Reporter: Matt Whitlock <gentoo>
Component: Current packagesAssignee: Sam James <sam>
Status: RESOLVED FIXED    
Severity: normal CC: base-system, crabbedhaloablution
Priority: Normal Keywords: PATCH, PullRequest
Version: unspecified   
Hardware: All   
OS: Linux   
See Also: https://github.com/gentoo/gentoo/pull/18658
Whiteboard:
Package list:
Runtime testing required: ---
Attachments: chrony-4.0.ebuild.patch

Description Matt Whitlock 2020-12-11 21:43:31 UTC
Created attachment 678046 [details, diff]
chrony-4.0.ebuild.patch

net-misc/chrony-4.0 defaults to automatically linking with GnuTLS if it is available. On systems that have net-libs/gnutls installed, this results in a dynamic link from /usr/sbin/chronyd to libgnutls.so.* but no RDEPEND record of that dependency.

# scanelf -n /usr/sbin/chronyd
 TYPE   NEEDED FILE
ET_DYN libm.so.6,libnettle.so.8,libgnutls.so.30,libcap.so.2,libseccomp.so.2,libpthread.so.0,libc.so.6 /usr/sbin/chronyd

# fgrep -c gnutls /var/db/pkg/net-misc/chrony-4.0/RDEPEND
0

A suggested fix is:

 * add "gnutls" to IUSE
 * add "gnutls? ( net-libs/gnutls:= )" to DEPEND and RDEPEND
 * add $(usex gnutls '' --without-gnutls) to myconf

The attached patch applies these modifications.
Comment 1 crabbed halo ablution 2020-12-14 19:45:08 UTC
The solution is probably going to be a bit more complicated.
gnutls is only used for nts
nts also needs nettle support.

I'll cook up something correct-looking and link it.
Comment 2 Larry the Git Cow gentoo-dev 2021-02-08 14:54:09 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=e69294ba7087691b826ad8ed73dac52c9a3527aa

commit e69294ba7087691b826ad8ed73dac52c9a3527aa
Author:     Peter Alfredsen <crabbedhaloablution@icloud.com>
AuthorDate: 2020-12-15 11:46:03 +0000
Commit:     Sam James <sam@gentoo.org>
CommitDate: 2021-02-08 14:53:59 +0000

    net-misc/chrony: enable NTS support
    
    Enabled NTS support in ebuild. Needs net-libs/gnutls and dev-libs/nettle.
    NTS is the only thing net-libs/gnutls is used for.
    
    Reported-By: Matt Whitlock <gentoo@mattwhitlock.name>
    Closes: https://bugs.gentoo.org/759526
    Signed-off-by: Peter Alfredsen <crabbedhaloablution@icloud.com>
    Signed-off-by: Sam James <sam@gentoo.org>

 net-misc/chrony/chrony-9999.ebuild | 6 +++++-
 net-misc/chrony/metadata.xml       | 3 ++-
 2 files changed, 7 insertions(+), 2 deletions(-)