Bug 233777 - security/softhsm2: fix build with non-base openssl
Summary: security/softhsm2: fix build with non-base openssl
Status: New
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: freebsd-ports-bugs mailing list
URL:
Keywords: patch
Depends on:
Blocks:
 
Reported: 2018-12-04 15:40 UTC by Sergey Akhmatov
Modified: 2018-12-11 19:08 UTC (History)
1 user (show)

See Also:
jaap: maintainer-feedback+


Attachments
softhsm2 fix with non-base openssl (1.69 KB, patch)
2018-12-04 15:40 UTC, Sergey Akhmatov
no flags Details | Diff
softhsm2 fix with non-base openssl (1.67 KB, patch)
2018-12-04 16:36 UTC, Sergey Akhmatov
sergey: maintainer-approval? (jaap)
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Sergey Akhmatov 2018-12-04 15:40:25 UTC
Created attachment 199815 [details]
softhsm2 fix with non-base openssl

The port fails to build with non-base openssl, e.g. with DEFAULT_VERSIONS= ssl=openssl111 on 11.2 or DEFAULT_VERSIONS= ssl=openssl on 12.0

The attached patch fixes thing for me:
- Add USES=ssl and OPENSSLBASE path to configure options.
- Add patch from upstream to fix build with libressl: https://github.com/opendnssec/SoftHSMv2/commit/308b0b2760d6cb218003768747346d31764f1cfe


QA: poudriere testport OK on 11.2-RELEASE and 12.0-RC3 with DEFAULT_VERSIONS= ssl= (base, openssl, openssl111, libressl, libressl-devel)
Comment 1 Sergey Akhmatov 2018-12-04 16:36:51 UTC
Created attachment 199821 [details]
softhsm2 fix with non-base openssl

Updated patch, missed "USES= ssl" which makes "CRYP_OPEN_USES= ssl" useless, remove it.
Comment 2 jaap 2018-12-11 19:08:25 UTC
(In reply to Sergey Akhmatov from comment #1)

The patch seems to work fine, tested on multiple releases (using poudriere). Approved.

Small issue, the committer might want to raise the PORTREVISON.

    jaap