Bug 212016 - dns/powerdns, dns/powerdns-recursor: Fix build against LibreSSL 2.4+
Summary: dns/powerdns, dns/powerdns-recursor: Fix build against LibreSSL 2.4+
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Many People
Assignee: Bernard Spil
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-08-20 21:30 UTC by Ralf van der Enden
Modified: 2016-09-11 18:44 UTC (History)
1 user (show)

See Also:
koobs: merge-quarterly-


Attachments
Fix build against LibreSSL 2.4+ (3.77 KB, patch)
2016-08-20 21:30 UTC, Ralf van der Enden
tremere: maintainer-approval+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Ralf van der Enden 2016-08-20 21:30:08 UTC
Created attachment 173898 [details]
Fix build against LibreSSL 2.4+

Fix build against LibreSSL 2.4+

This patch fixes the build failure caused by the OPENSSL_MAJOR_VERSION check for builds against security/libressl 2.4.2+ since ports r420102 (Remove OPENSSL_VERSION_NUMBER patch)
Comment 1 Kubilay Kocak freebsd_committer freebsd_triage 2016-08-21 05:04:49 UTC
Thank you Ralf, can you please confirm this change passes QA (portlint, poudriere)
Comment 2 Ralf van der Enden 2016-08-21 06:51:43 UTC
Poudriere buildlogs can be found here:
https://pkg.cainites.net/build.html?mastername=freebsd_10-3x64-system&build=2016-08-21_08h46m26s
Comment 3 Kubilay Kocak freebsd_committer freebsd_triage 2016-08-21 09:47:29 UTC
Thanks Ralf, if logs are requested please include them as attachments as remote references/URL's tend to go stale

Are the quarterly versions of the ports also affected (does this need merging?)
Comment 4 Ralf van der Enden 2016-08-21 12:46:09 UTC
The 3.x branch doesn't rely on OpenSSL, so I don't believe merging is necessary.

I'm not in the vicinity of a PC right now, but I'll upload the poudriere buildlogs ASAP
Comment 5 Kubilay Kocak freebsd_committer freebsd_triage 2016-08-21 16:45:15 UTC
(In reply to Ralf van der Enden from comment #4)

Thanks Ralf :)

QA logs as attachments are not necessary, just a confirmation similar to the following would be great:

portlint: OK (looks fine.)
porttest: OK (poudriere: <versions>, <archs>, <options> tested)
Comment 6 commit-hook freebsd_committer freebsd_triage 2016-08-22 10:59:09 UTC
A commit references this bug:

Author: brnrd
Date: Mon Aug 22 10:58:07 UTC 2016
New revision: 420599
URL: https://svnweb.freebsd.org/changeset/ports/420599

Log:
  dns/powerdns: Fix build with LibreSSL

    - Add upstream patch 115f658ee2000a4cdcc13e999da50b3634c6a907
    - Patch dns/powerdns-recursor as well

  PR:		212016
  Submitted by:	Ralf van der Enden <tremere@cainites.net> (maintainer)
  Reported by:	Ralf van der Enden <tremere@cainites.net> (maintainer)
  MFH:		2016Q3

Changes:
  head/dns/powerdns/files/patch-libressl
  head/dns/powerdns-recursor/files/patch-libressl
Comment 7 Kubilay Kocak freebsd_committer freebsd_triage 2016-09-11 11:38:29 UTC
Assign to committer that resolved. Yours to update if necessary and close Bernard.