Skip to content

upnp: allow 1.14.30#2448

Merged
MaxKellermann merged 1 commit intoMusicPlayerDaemon:v0.24.xfrom
PPN-SD:allow_1.14.30
Mar 18, 2026
Merged

upnp: allow 1.14.30#2448
MaxKellermann merged 1 commit intoMusicPlayerDaemon:v0.24.xfrom
PPN-SD:allow_1.14.30

Conversation

@PPN-SD
Copy link
Copy Markdown
Contributor

@PPN-SD PPN-SD commented Mar 17, 2026

ABIbreak in 1.14.26 has been reverted w/ 1.14.30, reapplied in 1.18.X
align with upstream versions and make easier packaging.

@MaxKellermann
Copy link
Copy Markdown
Member

There is still no reply from upstream to my bug report: pupnp/pupnp#528

I don't understand your version_compare call. According to the Meson documentation, it allows only a single parameter: https://mesonbuild.com/Syntax.html#version_compare - but you pass three. What does that mean?

@PPN-SD PPN-SD force-pushed the allow_1.14.30 branch 2 times, most recently from 7d169b6 to 3d2ae5f Compare March 17, 2026 19:32
@PPN-SD
Copy link
Copy Markdown
Contributor Author

PPN-SD commented Mar 17, 2026

I don't understand your version_compare call. According to the Meson documentation, it allows only a single parameter:
https://mesonbuild.com/Syntax.html#version_compare

Two are allowed since meson-1.8. But maybe you won't up this for that.

but you pass three. What does that mean?

Two bad pushes, sry.

@MaxKellermann
Copy link
Copy Markdown
Member

The stable branch (v0.24.x) must remain compatible with Meson 1.0.
The unstable branch (master) must remain compatible with Debian stable, so we may update to 1.7 but not newer.

Your PR targets master - are you aware that this will not be part of the next bug-fix release then?

@MaxKellermann
Copy link
Copy Markdown
Member

The unstable branch (master) must remain compatible with Debian stable, so we may update to 1.7 but not newer.

And it should be compatible with GitHub Actions runner images - as you see in the build failure, it has only Meson 1.3.2. That is not the most important thing (because GitHub often lags behind more than even Debian), it's most important to be compatible with Debian Stable - Debian is a good benchmark for "builds everywhere (unless people are reaaaaaaally on old OS)".

@PPN-SD
Copy link
Copy Markdown
Contributor Author

PPN-SD commented Mar 17, 2026

The stable branch (v0.24.x) must remain compatible with Meson 1.0. The unstable branch (master) must remain compatible with Debian stable, so we may update to 1.7 but not newer.

OK.

Your PR targets master - are you aware that this will not be part of the next bug-fix release then?

The most important thing is in my opinion to let the distributions apply version 1.14.30 and not directly block version >= 1.14.26. It doesn’t matter what happens to this PR, to be honest.

@MaxKellermann
Copy link
Copy Markdown
Member

It doesn’t matter what happens to this PR, to be honest.

Does this mean you don't want to fix this PR?

@PPN-SD PPN-SD force-pushed the allow_1.14.30 branch 2 times, most recently from 889ded3 to 171d112 Compare March 18, 2026 05:51
@PPN-SD
Copy link
Copy Markdown
Contributor Author

PPN-SD commented Mar 18, 2026

It should be fine now.

ABIbreak in 1.14.26 has been reverted w/ 1.14.30, reapplied in 1.18.X
align with upstream versions and make easier packaging.

Signed-off-by: Nicolas PARLANT <nicolas.parlant@parhuet.fr>
@PPN-SD PPN-SD changed the base branch from master to v0.24.x March 18, 2026 06:05
@MaxKellermann MaxKellermann merged commit fda4057 into MusicPlayerDaemon:v0.24.x Mar 18, 2026
@PPN-SD PPN-SD deleted the allow_1.14.30 branch March 18, 2026 09:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants