Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 759526 - net-misc/chrony-4.0 missing IUSE="gnutls" and DEPEND="gnutls? ( net-libs/gnutls:= )"
Summary: net-misc/chrony-4.0 missing IUSE="gnutls" and DEPEND="gnutls? ( net-libs/gnut...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Sam James
URL:
Whiteboard:
Keywords: PATCH, PullRequest
Depends on:
Blocks:
 
Reported: 2020-12-11 21:43 UTC by Matt Whitlock
Modified: 2021-02-08 14:54 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
chrony-4.0.ebuild.patch (chrony-4.0.ebuild.patch,957 bytes, patch)
2020-12-11 21:43 UTC, Matt Whitlock
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
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(-)