Bug 236617

Summary: irc/bitchx: Ensure paths to correct OpenSSL are used in build
Product: Ports & Packages Reporter: caf
Component: Individual Port(s)Assignee: Steve Wills <swills>
Status: Closed FIXED    
Severity: Affects Some People CC: freebsd
Priority: --- Keywords: needs-qa
Version: LatestFlags: freebsd: maintainer-feedback+
koobs: merge-quarterly?
Hardware: Any   
OS: Any   
Attachments:
Description Flags
Tested patch for port freebsd: maintainer-approval+

Description caf 2019-03-18 12:55:59 UTC
Created attachment 202955 [details]
Tested patch for port

When the default OpenSSL version is set to something other than base, the correct include path is not being used (so the package is compiled against the headers from the base version of OpenSSL, but linked against the ports version).

This is clearly bogus, and causes build failures on FreeBSD 11.2 when DEFAULT_VERSIONS is set to ssl=openssl111.

This bumps the PORTREVISION because even if it apparently compiles successfully with the wrong headers there could be some subtle breakage.
Comment 1 BitchX FreeBSD 2019-03-18 13:35:01 UTC
This supplies the OpenSSL base where the configure script expects it.
Comment 2 commit-hook freebsd_committer freebsd_triage 2019-03-19 15:05:36 UTC
A commit references this bug:

Author: swills
Date: Tue Mar 19 15:04:33 UTC 2019
New revision: 496264
URL: https://svnweb.freebsd.org/changeset/ports/496264

Log:
  irc/bitchx: Use correct OpenSSL include path

  PR:		236617
  Submitted by:	caf@bitchx.org
  Approved by:	BitchX FreeBSD <freebsd@bitchx.org> (maintainer)

Changes:
  head/irc/bitchx/Makefile
Comment 3 Steve Wills freebsd_committer freebsd_triage 2019-03-19 15:05:54 UTC
Committed, thanks!