Bug 253807

Summary: devel/lua-cqueues: fix SSL, update to newer version with 5.4 support
Product: Ports & Packages Reporter: Andrew "RhodiumToad" Gierth <andrew>
Component: Individual Port(s)Assignee: Neel Chauhan <nc>
Status: Closed FIXED    
Severity: Affects Only Me CC: freebsd, nc
Priority: --- Keywords: buildisok
Version: LatestFlags: bugzilla: maintainer-feedback? (freebsd)
Hardware: Any   
OS: Any   
Bug Depends on:    
Bug Blocks: 254155    
Attachments:
Description Flags
Update to 20200726 and fix ssl
andrew: maintainer-approval?
lua-cqueues 20200726 freebsd: maintainer-approval-

Description Andrew "RhodiumToad" Gierth 2021-02-24 03:39:52 UTC
Created attachment 222781 [details]
Update to 20200726 and fix ssl

Existing port fails to build with ssl=openssl (rather than ssl=base) in DEFAULT_VERSIONS, due to missing -I option. (Fails at runtime with a missing symbol, due to using base's include files with port's libssl.so)

While here, update to 20200726 which adds support for Lua 5.4.
Comment 1 Automation User 2021-02-24 15:55:16 UTC
Build and package info is available at https://gitlab.com/swills/freebsd-ports/pipelines/261187884
Comment 2 Leo Vandewoestijne 2021-02-25 11:27:35 UTC
Created attachment 222821 [details]
lua-cqueues 20200726

Spot-on! Nice, thanks.

I've improved your patch by addressing another issue at the same time
(the fixed /usr/local)

Tested OK (portlint and poudriere).
Comment 3 Leo Vandewoestijne 2021-02-25 11:35:09 UTC
For clarity: patch 222781 (-) was used to create patch 222821 (+).
Comment 4 Andrew "RhodiumToad" Gierth 2021-02-25 11:41:34 UTC
No, that change to /usr/local is wrong; the value of lua_OLD_CMD (used by shebangfix) is what is being searched for on the #! line of the examples in the distfile, in order to replace it, so it has to match those files regardless of ${LOCALBASE}.
Comment 5 Leo Vandewoestijne 2021-02-25 11:47:54 UTC
(In reply to andrew from comment #4)

Oh dang - of course!
(too focused on pleasing portlint)
Comment 6 Leo Vandewoestijne 2021-02-25 11:50:08 UTC
For clarity (...): patch 222781 (+) is correct, and patch 222821 (-) is not.
(and I can't set it to +)
Comment 7 Neel Chauhan freebsd_committer freebsd_triage 2021-03-16 18:09:25 UTC
Committed!
Comment 8 commit-hook freebsd_committer freebsd_triage 2021-03-16 18:10:02 UTC
A commit references this bug:

Author: nc
Date: Tue Mar 16 18:09:19 UTC 2021
New revision: 568595
URL: https://svnweb.freebsd.org/changeset/ports/568595

Log:
  devel/lua-cqueues: Update to 20200726 and fix SSL

  PR:		253807
  Submitted by:	andrew@tao11.riddles.org.uk, freebsd@dns.company
  Reported by:	andrew@tao11.riddles.org.uk
  Approved by:	freebsd@dns.company (maintainer)

Changes:
  head/devel/lua-cqueues/Makefile
  head/devel/lua-cqueues/distinfo