Meson configure pass, but compilation complain about missing SDL2

• Rapporteur: lachu
• Propriétaire: cazfi
• Type: Bogues
• État: Atteints
• Composant: Bootstrap
• Jalon: 3.1.0-beta1 (atteints)
• Priorité: 7
• Sévérité: 5 - moyen
• Résolution: Fixed
• Fichier: 2

Détails

Meson build system of Freeciv have bug, with cause compilation to fail with complain about missing SDL2 libraries, even if client was set to other than SDL2 and audio was disabled.

Ticket History (3/10 Histories)

2023-02-09 00:05 Updated by: lachu

• New Ticket "Meson build pass, but compilation complain about missing SDL2" created

2023-02-09 00:05 Updated by: lachu

• File meson.log (File ID: 11572) is attached

2023-02-09 15:05 Updated by: cazfi

• Jalon Update from (Aucun) to 3.1.0-beta1 (atteints)
• Composant Update from General to Bootstrap

2023-02-10 01:45 Updated by: lachu

• Summary Updated

Commentaire

To be more clarify - not meson build passed, but configure process. Compilation gives result as described.

2023-02-10 01:54 Updated by: cazfi

• File 0035-Meson-Fix-build-when-SDL2-is-not-present.patch (File ID: 11577) is attached

2023-02-10 01:55 Updated by: cazfi

• Propriétaire Update from (Aucun) to cazfi
• Résolution Update from Aucun to Accepted

Commentaire

Does attached patch help? I have not tested in any environment that would really lack SDL2.

2023-02-12 20:03 Updated by: lachu

Commentaire

Reply To cazfi

Does attached patch help? I have not tested in any environment that would really lack SDL2.

Sorry for do not respond in time. I see status is accepted, but on my repository (git log) there is no content, which could describe this patch. But... I got:

../meson.build:846:0: ERROR: Unknown variable "lzma_dep".

I do not known if it is related.

This is not probably related. I look inside patch and nothing suspicious was inside.

(Edited, 2023-02-12 20:04 Updated by: lachu)

2023-02-12 20:13 Updated by: cazfi

Commentaire

Reply To lachu

Reply To cazfi

Does attached patch help? I have not tested in any environment that would really lack SDL2.

Sorry for do not respond in time. I see status is accepted, but on my repository (git log) there is no content, which could describe this patch.

Since osdn lacks more accurate status, we use "accepted" as the sign that the review period is running for the patch. It's changed to "Fixed" when pushed to the repository.

../meson.build:846:0: ERROR: Unknown variable "lzma_dep".

Thanks -> #47338

2023-02-14 02:21 Updated by: lachu

Commentaire

Reply To cazfi

Reply To lachu

Reply To cazfi

Does attached patch help? I have not tested in any environment that would really lack SDL2.

Sorry for do not respond in time. I see status is accepted, but on my repository (git log) there is no content, which could describe this patch.

Since osdn lacks more accurate status, we use "accepted" as the sign that the review period is running for the patch. It's changed to "Fixed" when pushed to the repository.

../meson.build:846:0: ERROR: Unknown variable "lzma_dep".

Thanks -> #47338

Build properly under Debian BookWorm.

 $ meson -Daudio=false -Dclients=gtk3.22 -Dqtver=qt5

Information about OS:

 $ cat /etc/os-release 
 PRETTY_NAME="Debian GNU/Linux bookworm/sid"
 NAME="Debian GNU/Linux"
 ID=debian
 HOME_URL="https://www.debian.org/"
 SUPPORT_URL="https://www.debian.org/support"
 BUG_REPORT_URL="https://bugs.debian.org/"

But my version is stripped, because I am using Docker Container.

2023-02-14 05:06 Updated by: cazfi

• État Update from Ouvert to Atteints
• Résolution Update from Accepted to Fixed