Bug 225149
| Summary: | net/haproxy: Fix openssl linking | ||||||
|---|---|---|---|---|---|---|---|
| Product: | Ports & Packages | Reporter: | Bernard Spil <brnrd> | ||||
| Component: | Individual Port(s) | Assignee: | Dmitry Sivachenko <demon> | ||||
| Status: | Closed FIXED | ||||||
| Severity: | Affects Only Me | CC: | brnrd, pkubaj | ||||
| Priority: | --- | Keywords: | needs-qa, patch | ||||
| Version: | Latest | Flags: | bugzilla:
maintainer-feedback?
(demon) |
||||
| Hardware: | Any | ||||||
| OS: | Any | ||||||
| Attachments: |
|
||||||
The patch looks fine for me, but please also make the same changes to net/haproxy-devel port. Thanks! A commit references this bug: Author: brnrd Date: Mon Jan 15 21:10:53 UTC 2018 New revision: 459115 URL: https://svnweb.freebsd.org/changeset/ports/459115 Log: net/haproxy-devel: Fix linking SSL libs - Pass lib and include paths to make - Optionsify PR: 225149 Approved by: demon (maintainer) Changes: head/net/haproxy-devel/Makefile The build error on -devel on 10.x is pre-existing. A commit references this bug: Author: brnrd Date: Mon Jan 15 21:12:58 UTC 2018 New revision: 459116 URL: https://svnweb.freebsd.org/changeset/ports/459116 Log: net/haproxy: Fix linking SSL libs - Pass lib and include paths to make - Optionsify PR: 225149 Approved by: demon (maintainer) Changes: head/net/haproxy/Makefile |
Created attachment 189694 [details] svn diff for net/haproxy I was notified of an issue when using haproxy with BoringSSL. It was linking base libssl and libcrypto even though BoringSSL was set as ssl provider. Whilst in the Makefile I migrated to the new Options framework. The LIB_DEPENDS for static PCRE was set to the dynamic .so lib file as the static .a files in the pcre port are not optional (i.e. will always be available when devel/pcre is installed). The port now forces the SSL lib and include dir so it will link correctly. Proposed commit msg ``` net/haproxy: Fix SSL linking - Pass lib and include paths to make - Optionsify PR: Submitted by: brnrd Approved by: ... (maintainer) ```