Ticket #48234

Meson: Lua detection regression compared to autotools

Open Date: 2023-06-13 15:29 Last Update: 2023-06-26 04:13

Reporter:
Owner:
Type:
Status:
Closed
Component:
Priority:
5 - Medium
Severity:
5 - Medium
Resolution:
Fixed
File:
3

Details

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.

Ticket History (3/8 Histories)

2023-06-13 15:29 Updated by: cazfi
  • New Ticket "Meson: Lua detection regression compared to autotools" created
2023-06-14 13:39 Updated by: cazfi
2023-06-24 04:08 Updated by: cazfi
Comment

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.

2023-06-24 04:35 Updated by: cazfi
  • Owner Update from (None) to cazfi
  • Resolution Update from None to Accepted
Comment

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.

2023-06-26 04:13 Updated by: cazfi
  • Status Update from Open to Closed
  • Resolution Update from Accepted to Fixed

Edit

You are not logged in. I you are not logged in, your comment will be treated as an anonymous post. » Login