Meson: Lua detection regression compared to autotools
Reply To cazfi
The patch they have provided bumps mimum meson requirement.
We already require 0.57, which is higher than 0.56 of Debian Bullseye. At the same time new 0.60 dependency would still be lower than 0.61 of Ubuntu Jammy. Further, checking meson 0.60 Release Notes, there are many features that we would want when making meson the primary build system for all kind of tasks.
So, I'd say we bump the dependency in main & S3_2. Even thought all of meson build is flagged experimental in S3_1, I'd rather follow the rule that we don't increase such minimum requirements on a series that has already released at least betas.
Attached main & S3_2 patches are the original patch with relevant documentation changes added (Both touching doc/INSTALL.meson, S3_2 one touching also README.packaging 3.1 -> 3.2 changes list). The patch retains its original author information, with just mention of my additions in the commit message.
S3_1 one I wrote from scratch, so I'm the author, commit message just mentioning bug reporting information.
Kangie has opened PR https://github.com/freeciv/freeciv/pull/48 , thus reporting an autotools -> meson regression that lua is not searched from alternative locations.
The patch they have provided bumps mimum meson requirement. This is going to need some actions. At best case just a documentation updated (doc/INSTALL.meson on all branches where the minimum version changes, README.packaging in the oldest branch where the minimum version changes). At worst case we need to implement that boilerplate, that minimum meson version bump tried to avoid, instead. As for S3_1, there's also an option to just not to implement the fix, as meson based build is experimental feature on that branch anyway.