Bug 198780 - [PATCH] devel/dcmtk: Make EGD conditional/Fix build with LibreSSL
Summary: [PATCH] devel/dcmtk: Make EGD conditional/Fix build with LibreSSL
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: Kurt Jaeger
Keywords: easy, feature, patch, patch-ready
Depends on:
Reported: 2015-03-21 19:45 UTC by Bernard Spil
Modified: 2015-03-29 20:25 UTC (History)
1 user (show)

See Also:

svn diff for devel/dcmtk (964 bytes, patch)
2015-03-21 19:45 UTC, Bernard Spil
no flags Details | Diff
Poudriere build log for devel/dcmtk (72.88 KB, application/x-gzip)
2015-03-21 19:46 UTC, Bernard Spil
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Bernard Spil freebsd_committer 2015-03-21 19:45:44 UTC
Created attachment 154637 [details]
svn diff for devel/dcmtk

devel/ace RAND_egd with OpenSSL version > 0.9.5 which makes building fail with LibreSSL which has removed EGD. FreeBSD does not require EGD at all, /dev/random has been available since FreeBSD 4.2.

Adds some missing include/opensslv.h as well as build failed due to OPENSSL_VERSION defined
Comment 1 Bernard Spil freebsd_committer 2015-03-21 19:46:23 UTC
Created attachment 154638 [details]
Poudriere build log for devel/dcmtk

larger than 1MB so compressed
Comment 2 Bernard Spil freebsd_committer 2015-03-21 19:49:18 UTC
Forgot to edit comment... This is for devel/dcmtk not devel/ace. No need for opensslv.h additions
Comment 3 Kurt Jaeger freebsd_committer 2015-03-29 20:09:53 UTC
Comment 4 commit-hook freebsd_committer 2015-03-29 20:23:35 UTC
A commit references this bug:

Author: pi
Date: Sun Mar 29 20:22:43 UTC 2015
New revision: 382626
URL: https://svnweb.freebsd.org/changeset/ports/382626

  devel/dcmtk: add EGD conditional to fix build with LibreSSL

  PR:		198780
  Submitted by:	Bernard Spil <spil.oss@gmail.com>

Comment 5 Kurt Jaeger freebsd_committer 2015-03-29 20:25:53 UTC
Build tests were OK with 10.1a, 9.3a, 8.4i.

So: Committed, thanks!