Created attachment 199711 [details] unified diff to update net-im/prosody to 0.11.1 The attached patch should update the prosody port to 0.11.1. This changes prosody's lua version to lua52, removes an unnecessary dependency and reworks the patches. The patch to util-src/time.c is due to https://issues.prosody.im/1251 and should be resolved in future releaes. I'm not sure about the pkg-plist file, so any feedback is greatly appreciated. The package builds fine and is currently running on my own server w/o any issues as of yet.
Hello! Has this been tested with LuaJIT at all? That's one of the things that's been holding us back.
(In reply to David Thiel from comment #1) Hi David! I've never used luajit and apparently completely skipped testing it. There's even a mistake in the luajit-dependency in the makefile. It builds once that is fixed, I'll at least give it a brief spin w/ luajit and post the fixed patch one that's done.
(In reply to David Thiel from comment #1) Alright, after looking at this for some more time I'm uncertain on how to proceed from here: - using prosody w/ luajit (kinda obviously) requires the lua-5.1 versions of various libraries to be installed. - it seems difficult to use lua-5.2 by default and switch the deps to lua-5.1 if the luajit-option is selected. Judging from the way lua-5.1 ports are currently handled, it might be for the best to - remove luajit from the lua-5.2 version of prosody, switch to USES=lua instead of lua:52 explicitly - add a prosody-51 and/or prosody-luajit port which pins USES to lua:51 Dropping luajit from the "main" port seems almost unavoidable as the prosody 0.11 series will be the last one to support lua 5.1. (see [1] ). Meanwhile, I've received feedback that the non-luajit part works fine, even on freebsd-12.0, so at least that's ok. Feedback would be welcome, Sascha [1] See the part about "Lua 5.2" on https://blog.prosody.im/prosody-0-11-0-released/
Thanks for looking into it. To avoid the churn of adding and removing a 5.1-specific prosody port, is there any compelling reason to not just leave the current port at 5.1, and when the next version is released, cut out luajit and convert solely to 5.2? AFAIK there's not a specific downside to 5.1 other than being a bit older.
A commit references this bug: Author: lx Date: Tue Jan 29 23:15:56 UTC 2019 New revision: 491602 URL: https://svnweb.freebsd.org/changeset/ports/491602 Log: Update to 0.11.2. PR: 233692 Submitted by: Sascha Biberhofer Changes: head/net-im/prosody/Makefile head/net-im/prosody/distinfo head/net-im/prosody/files/patch-GNUmakefile head/net-im/prosody/files/patch-Makefile head/net-im/prosody/files/patch-configure head/net-im/prosody/files/patch-util-src_pposix.c head/net-im/prosody/files/patch-util-src_time.c head/net-im/prosody/pkg-plist
Committed with modifications: - Update to 0.11.2 - Keep lua51 - Remove rc script from plist