Update bsd.defaults.mk for lua to 5.3 and questions about lua.mk

[russ.haley_gmail.com]

Summary

Hi, I saw that the default version of lua is still 5.2 so thought I'd attempt to push that up. I'd also like to ask some questions about the lua.mk file and possibly "clean it up"?

Test Plan

Looking for input.

[russ.haley_gmail.com]

Mk/Uses/lua.mk

80 ↗

(On Diff #37222)

So I don't really know if using ${PORTNAME} is valid or not, but wanted to look at standardizing it with the lua ports themselves (at least 5.3 and anything forward). Just looking for input really. How do I test this without modifying my actual ports dir (which I use as a reference)?

[antoine]

There was an exp-run a few months ago and the results were bad.
Maybe lua ports could be flavored...

Using ${PORTNAME} in Uses/lua.mk is really wrong.

[russ.haley_gmail.com]

In D13691#286171, @antoine wrote:

There was an exp-run a few months ago and the results were bad.
Maybe lua ports could be flavored...

Sorry, could you expand on exp-run and what you mean by flavored?

Using ${PORTNAME} in Uses/lua.mk is really wrong.

I thought it might be but didn't know.

[mat]

Before the change in default version can go in, an exp-run needs to be requested. See 19.8.5. How can an experimental test build of the ports tree (exp-run) be requested?

[antoine]

The last exp-run was: https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=220281

[russ.haley_gmail.com]

In D13691#286204, @antoine wrote:

The last exp-run was: https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=220281

Much appreciated. I'll start going through these failures. Given that some of these may be "unfixable"/unused/unmaintained, what is the expectation for "pass level" for an exp-run?

Is there any way to cross check how many of these failures build using Lua 5.2? Is is possible/acceptable to pin items to 5.2 that fail and let others use the default?

lzlib for instance is:

• unmaintained by the creator (hisham from luarocks)
• has a replacement packaged (lua-zlib) https://luarocks.org/modules/brimworks/lua-zlib
• Is available through luarocks should someone want it.

Can I ((propose a patch to) pin it to 5.2 (if it builds with 5.2) and allow the dependents to build?

[russ.haley_gmail.com]

Removed changes to lua.mk. Review is tracking questions on exp-run failures. Alternatively we can close this and I can use bugzilla.

[russ.haley_gmail.com]

Hi, I've looked over the failures and I think I'd rather just close this review and comment on the original bugzilla ticket. Not sure how to close a review?

[russ.haley_gmail.com]

Would really like it if we could all work towards getting Lua 5.4 as the default version on FreeBSD!