Bug 200336
| Summary: | [PATCH] net/luasocket, don't force to build with Lua 5.1 | ||||||
|---|---|---|---|---|---|---|---|
| Product: | Ports & Packages | Reporter: | Olivier Duchateau <olivierd> | ||||
| Component: | Individual Port(s) | Assignee: | David Thiel <lx> | ||||
| Status: | Closed FIXED | ||||||
| Severity: | Affects Only Me | CC: | bhtooefr, lantw44, pkubaj | ||||
| Priority: | --- | Keywords: | patch | ||||
| Version: | Latest | Flags: | bugzilla:
maintainer-feedback?
(lx) |
||||
| Hardware: | Any | ||||||
| OS: | Any | ||||||
| Bug Depends on: | |||||||
| Bug Blocks: | 200337 | ||||||
| Attachments: |
|
||||||
Without this fix, I can't upgrade audio/lollypop to 0.9.20 A commit references this bug: Author: lx Date: Mon Feb 13 20:45:04 UTC 2017 New revision: 434053 URL: https://svnweb.freebsd.org/changeset/ports/434053 Log: Remove fixed dependency on Lua 5.1. PR: 200336 Submitted by: Olivier Duchateau Changes: head/net/luasocket/Makefile Committed, thanks! (In reply to David Thiel from comment #3) This breaks net-im/prosody, which depends on Lua 5.1. If you can't revert this commit, could you repocopy this port so that it depends on Lua 5.1 and make net-im/prosody depend on it? It's my understanding that Prosody now supports 5.2, so I'll instead try to rip out the 5.1 dependency from the Prosody port. If that doesn't work, then yeah, I'll take one of those two approaches. Thanks! It looks like I was mistaken — trunk supports 5.2, but 0.9.12 does not. Will look into repocopy or revert. A commit references this bug: Author: lx Date: Tue Feb 14 22:23:06 UTC 2017 New revision: 434143 URL: https://svnweb.freebsd.org/changeset/ports/434143 Log: Copy luasocket to a 5.1-specific port PR: 200336 Changes: head/net/Makefile head/net/luasocket-51/ head/net/luasocket-51/Makefile A commit references this bug: Author: lx Date: Wed Feb 15 00:43:31 UTC 2017 New revision: 434148 URL: https://svnweb.freebsd.org/changeset/ports/434148 Log: Use new Lua 5.1-specific ports. PR: 200336 Changes: head/net-im/prosody/Makefile FWIW, this appears to introduce a conflict between net/luasocket-51 and net/luasocket, at least on my system, in the documentation. At least net-im/prosody appears to be the only thing needing luasocket, so I deleted net/luasocket, but there could be cases where both ports need to be installed... |
Created attachment 156955 [details] Patch, which avoid to build for specific Lua version net/luasocket works fine with Lua 5.1 and 5.2.