Bug 240297
| Summary: | dns/opendnssec{2}: Fails to configure after ldns 1.7.1 update (configure: error: Can't find ldns library) | ||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | Ports & Packages | Reporter: | Roman Serbski <mefystofel> | ||||||||||
| Component: | Individual Port(s) | Assignee: | Danilo G. Baio <dbaio> | ||||||||||
| Status: | Closed FIXED | ||||||||||||
| Severity: | Affects Only Me | CC: | dbaio, fox, jaap, swills | ||||||||||
| Priority: | --- | Keywords: | regression | ||||||||||
| Version: | Latest | Flags: | koobs:
maintainer-feedback+
|
||||||||||
| Hardware: | Any | ||||||||||||
| OS: | Any | ||||||||||||
| Attachments: |
|
||||||||||||
|
Description
Roman Serbski
2019-09-03 09:56:00 UTC
Thank you for the report Roman Could you please include the following information: - pkg version -v output (as an attachment) - full build log (as an attachment) - WRKSRC/config.log (as an attachment) Thank you Kubilay. As requested: $ pkg version -v adacurses-20180127 = up-to-date with port binutils-2.32_1,1 = up-to-date with port bison-3.4.1,1 = up-to-date with port ccache-3.7.1 = up-to-date with port dialog4ports-0.1.6 = up-to-date with port gcc6-aux-20180516_1,1 = up-to-date with port gettext-runtime-0.20.1 = up-to-date with port gettext-tools-0.20.1_1 = up-to-date with port gmake-4.2.1_3 = up-to-date with port gmp-6.1.2_1 = up-to-date with port help2man-1.47.11 = up-to-date with port indexinfo-0.3.1 = up-to-date with port ini_file_manager-03_3 = up-to-date with port libiconv-1.14_11 = up-to-date with port libtextstyle-0.20.1 = up-to-date with port m4-1.4.18_1,1 = up-to-date with port mpc-1.1.0_2 = up-to-date with port mpfr-4.0.2 = up-to-date with port ncurses-6.1.20190525 = up-to-date with port p5-Locale-gettext-1.07 = up-to-date with port p5-Locale-libintl-1.31 = up-to-date with port p5-Text-Unidecode-1.30 = up-to-date with port p5-Unicode-EastAsianWidth-1.40 = up-to-date with port perl5-5.30.0 = up-to-date with port pkg-1.11.1 = up-to-date with port pkgconf-1.6.3,1 = up-to-date with port rsync-3.1.3_1 = up-to-date with port synth-2.08 = up-to-date with port texinfo-6.6_2,1 = up-to-date with port Created attachment 207138 [details]
dns___opendnssec.log
Created attachment 207139 [details]
dns___opendnssec2.log
Thanks for reporting the bug and I can confirm and reproduce it. Apparently ldns-config changed with the upgrade and now produces the wrong information.
Correct information:
/usr/local/bin/ldns-config --cflags --version
-I/usr/local/include
1.7.0
Wrong information:
/usr/local/bin/ldns-config --cflags --version
-I/include
1.7.1
It is a shell script so a very quick work around is to edit it by hand.
Meanwhile I'll chase root cause of the problem and will provide a decent patch later.
Created attachment 207165 [details] Patch to add proper prefix path to ldns-config script. (In reply to Jaap Akkerhuis from comment #5) After fiddling around a bit, adding the following additional lines in packaging/ldns-config.in helped it spew out the correct paths. --- packaging/ldns-config.in.orig 2019-09-03 19:39:55 UTC +++ packaging/ldns-config.in @@ -1,5 +1,7 @@ #!/bin/sh +prefix="@prefix@" +exec_prefix="@exec_prefix@" VERSION="@PACKAGE_VERSION@" LDFLAGS="@LDFLAGS@ @LIBSSL_LDFLAGS@ @PYTHON_LDFLAGS@" LIBS="@LIBS@ @LIBSSL_LIBS@" After the patching and re-installing dns/ldns, ldns-config shows # /usr/local/bin/ldns-config --cflags --version -I/usr/local/include 1.7.1 This also helped with building dns/opendnssec package successfully. Let me know if this is the correct way to apply the fix. (In reply to Santhosh Raju from comment #6) Yes, this is the root cause of the problem. Before I saw your reaction, I actually just created PR #240321, which contains a patch to fix this properly. (In reply to Jaap Akkerhuis from comment #7) Alright, thank you for the fix. *** Bug 240321 has been marked as a duplicate of this bug. *** Created attachment 207180 [details]
Patch to fix problem
Patch to dns/ldns to fix the problem
Comment on attachment 207180 [details]
Patch to fix problem
Patch author is MAINTAINER, set maintainer-approval attachment flag (to +) to denote maintainer approval so that it can be distinguished from non-maintainer patches
Maintainer acknowledged/responded, set maintainer-feedback flag (to +) A commit references this bug: Author: dbaio Date: Thu Sep 5 23:54:46 UTC 2019 New revision: 511275 URL: https://svnweb.freebsd.org/changeset/ports/511275 Log: dns/ldns: Fix ldns-config After the last update ldns-config is producing wrong information and at least dns/opendnssec and dns/opendnssec2 are failing to build. PR: 240297 Submitted by: Jaap Akkerhuis <jaap@NLnetLabs.nl> (maintainer) Reported by: Roman Serbski <mefystofel@gmail.com> Changes: head/dns/ldns/Makefile head/dns/ldns/files/ head/dns/ldns/files/patch-packaging_ldns-config.in Committed, thank you all. |
