Page MenuHomeFreeBSD

lang/lua53 - Correct variable order in Makefile.
ClosedPublic

Authored by russ.haley_gmail.com on Jul 24 2020, 9:49 PM.
Tags
None
Referenced Files
F81924320: D25797.diff
Tue, Apr 23, 6:24 AM
Unknown Object (File)
Sat, Apr 20, 6:05 PM
Unknown Object (File)
Fri, Apr 19, 12:00 AM
Unknown Object (File)
Tue, Apr 16, 5:52 AM
Unknown Object (File)
Thu, Apr 11, 6:20 AM
Unknown Object (File)
Sun, Mar 31, 2:37 AM
Unknown Object (File)
Feb 17 2024, 3:12 AM
Unknown Object (File)
Feb 17 2024, 3:11 AM

Details

Summary

The patch moves the options variables below the build variables to conform to the handbook.

Test Plan

Tested on FreeBSD 12.1 STABLE. No material change to the functioning of the port.

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

5.3.6 is imminent, if there's any point at all in doing this cosmetic change then it could at least be folded into that.

Also PORTREVISION should not be changed if there are no functional changes, since you'd just be forcing rebuilds/updates of every dependent port for no reason.

5.3.6 is imminent, if there's any point at all in doing this cosmetic change then it could at least be folded into that.

Also PORTREVISION should not be changed if there are no functional changes, since you'd just be forcing rebuilds/updates of every dependent port for no reason.

I posted this now while I had the time because I'm prone to forget. We can let it sit until 5.3.6 is released. If I have time I'll update it with the RC1.

Here's one that updates to 5.3.6 and has more extensive changes to variable order that satisfies portclippy, barring the use of a few variables it doesn't know about.

I'm going to commit this around noon my time (so ~3 hours from the time of writing, CDT/UTC-5) -- I'm going to go ahead and incorporate the pkg-plist change and PKGNAMESUFFIX=${LUA_VER_STR} change from the lua54 port. I'm also considering renaming patch-src__Makefile to patch-src_Makefile in advance so the provenance of the patch remains intact (for now) when lua53 gets repocopied over to lua54.

This revision was not accepted when it landed; it landed in state Needs Review.Oct 15 2020, 5:28 PM
Closed by commit rP552431: lang/lua53: update to 5.3.6 (authored by kevans). · Explain Why
This revision was automatically updated to reflect the committed changes.