Script 'mail_helper' called by obssrc Hello community, here is the log from the commit of package scummvm for openSUSE:Factory checked in at 2021-04-17 00:02:19 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/openSUSE:Factory/scummvm (Old) and /work/SRC/openSUSE:Factory/.scummvm.new.12324 (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "scummvm" Sat Apr 17 00:02:19 2021 rev:23 rq:886113 version:2.2.0 Changes: -------- --- /work/SRC/openSUSE:Factory/scummvm/scummvm.changes 2020-11-05 21:56:39.251950226 +0100 +++ /work/SRC/openSUSE:Factory/.scummvm.new.12324/scummvm.changes 2021-04-17 00:02:39.361678417 +0200 @@ -1,0 +2,11 @@ +Fri Apr 16 14:49:13 UTC 2021 - Antonio Larrosa <alarr...@suse.com> + +- Add patches from upstream to fix build with fluidsynth 2.2 + * 0001-FLUIDSYNTH-Make-FluidSynth-logging-less-noisy-by-default.patch + * 0002-FLUIDSYNTH-Swapped-debug-levels-for-FLUID_WARN-and-FLUID_INFO.patch + * 0003-FLUIDSYNTH-Fix-build.patch + * 0004-AUDIO-Fix-compilation-with-Fluidsynth2.patch + * 0005-FLUIDSYNTH-Simplify-FluidSynth-version-check.patch + * 0006-FLUIDSYNTH-Fix-compilation-with-Fluidsynth-2.2.patch + +------------------------------------------------------------------- New: ---- 0001-FLUIDSYNTH-Make-FluidSynth-logging-less-noisy-by-default.patch 0002-FLUIDSYNTH-Swapped-debug-levels-for-FLUID_WARN-and-FLUID_INFO.patch 0003-FLUIDSYNTH-Fix-build.patch 0004-AUDIO-Fix-compilation-with-Fluidsynth2.patch 0005-FLUIDSYNTH-Simplify-FluidSynth-version-check.patch 0006-FLUIDSYNTH-Fix-compilation-with-Fluidsynth-2.2.patch ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ scummvm.spec ++++++ --- /var/tmp/diff_new_pack.clS8Fz/_old 2021-04-17 00:02:40.001679429 +0200 +++ /var/tmp/diff_new_pack.clS8Fz/_new 2021-04-17 00:02:40.001679429 +0200 @@ -1,7 +1,7 @@ # # spec file for package scummvm # -# Copyright (c) 2020 SUSE LLC +# Copyright (c) 2021 SUSE LLC # # All modifications and additions to the file contributed by third parties # remain the property of their copyright owners, unless otherwise agreed @@ -27,6 +27,12 @@ Group: Amusements/Games/Other URL: https://www.scummvm.org/ Source: https://www.scummvm.org/frs/scummvm/%{version}/scummvm-%{version}.tar.xz +Patch0: 0001-FLUIDSYNTH-Make-FluidSynth-logging-less-noisy-by-default.patch +Patch1: 0002-FLUIDSYNTH-Swapped-debug-levels-for-FLUID_WARN-and-FLUID_INFO.patch +Patch2: 0003-FLUIDSYNTH-Fix-build.patch +Patch3: 0004-AUDIO-Fix-compilation-with-Fluidsynth2.patch +Patch4: 0005-FLUIDSYNTH-Simplify-FluidSynth-version-check.patch +Patch5: 0006-FLUIDSYNTH-Fix-compilation-with-Fluidsynth-2.2.patch BuildRequires: desktop-file-utils BuildRequires: discord-rpc-devel BuildRequires: gcc-c++ @@ -80,6 +86,12 @@ %prep %setup -q +%patch0 -p1 +%patch1 -p1 +%patch2 -p1 +%patch3 -p1 +%patch4 -p1 +%patch5 -p1 # build the endianness test without optimization otherwise gcc is too smart # and optimize everything away, making the test fail ++++++ 0001-FLUIDSYNTH-Make-FluidSynth-logging-less-noisy-by-default.patch ++++++ >From 08b54f489b663793168764986c1d1a834c33add3 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Torbj=C3=B6rn=20Andersson?= <eriktorbj...@users.sourceforge.net> Date: Wed, 13 Jan 2021 09:44:44 +0100 Subject: [PATCH] FLUIDSYNTH: Make FluidSynth logging less noisy by default --- audio/softsynth/fluidsynth.cpp | 29 +++++++++++++++++++++++++++++ 1 file changed, 29 insertions(+) diff --git a/audio/softsynth/fluidsynth.cpp b/audio/softsynth/fluidsynth.cpp index e0d7c4e3df33..fe2b0689c283 100644 --- a/audio/softsynth/fluidsynth.cpp +++ b/audio/softsynth/fluidsynth.cpp @@ -46,6 +46,29 @@ #include "backends/platform/ios7/ios7_common.h" #endif +static void logHandler(int level, const char *message, void *data) { + switch (level) { + case FLUID_PANIC: + error("FluidSynth: %s", message); + break; + case FLUID_ERR: + warning("FluidSynth: %s", message); + break; + case FLUID_WARN: + debug(2, "FluidSynth: %s", message); + break; + case FLUID_INFO: + debug(1, "FluidSynth: %s", message); + break; + case FLUID_DBG: + debug(3, "FluidSynth: %s", message); + break; + default: + fluid_default_log_function(level, message, data); + break; + } +} + class MidiDriver_FluidSynth : public MidiDriver_Emulated { private: MidiChannel_MPU401 _midiChannels[16]; @@ -166,6 +189,12 @@ int MidiDriver_FluidSynth::open() { if (_isOpen) return MERR_ALREADY_OPEN; + fluid_set_log_function(FLUID_PANIC, logHandler, NULL); + fluid_set_log_function(FLUID_ERR, logHandler, NULL); + fluid_set_log_function(FLUID_WARN, logHandler, NULL); + fluid_set_log_function(FLUID_INFO, logHandler, NULL); + fluid_set_log_function(FLUID_DBG, logHandler, NULL); + #if defined(FLUIDSYNTH_VERSION_MAJOR) && FLUIDSYNTH_VERSION_MAJOR > 1 // When provided with in-memory SoundFont data, only use the configured // SoundFont instead if it's explicitly configured on the current game. ++++++ 0002-FLUIDSYNTH-Swapped-debug-levels-for-FLUID_WARN-and-FLUID_INFO.patch ++++++ >From 21d65cedf231cb03cd2e63863bd6d9d6bc70ed1d Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Torbj=C3=B6rn=20Andersson?= <eriktorbj...@users.sourceforge.net> Date: Mon, 8 Mar 2021 20:19:22 +0100 Subject: [PATCH] FLUIDSYNTH: Swapped debug levels for FLUID_WARN and FLUID_INFO If FLUID_INFO is "verbose informational messages", perhaps they should be better hidden from the average user than warnings. Particularly since warnings are the only kind I've actually seen in the wild. (Change suggested by criezy.) --- audio/softsynth/fluidsynth.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/audio/softsynth/fluidsynth.cpp b/audio/softsynth/fluidsynth.cpp index fe2b0689c283..e41a7515b205 100644 --- a/audio/softsynth/fluidsynth.cpp +++ b/audio/softsynth/fluidsynth.cpp @@ -55,10 +55,10 @@ static void logHandler(int level, const char *message, void *data) { warning("FluidSynth: %s", message); break; case FLUID_WARN: - debug(2, "FluidSynth: %s", message); + debug(1, "FluidSynth: %s", message); break; case FLUID_INFO: - debug(1, "FluidSynth: %s", message); + debug(2, "FluidSynth: %s", message); break; case FLUID_DBG: debug(3, "FluidSynth: %s", message); ++++++ 0003-FLUIDSYNTH-Fix-build.patch ++++++ >From a3bc5d64b8c4041326c8a214c47f9a206fb8b693 Mon Sep 17 00:00:00 2001 From: sluicebox <22204938+sluice...@users.noreply.github.com> Date: Mon, 8 Mar 2021 21:00:59 -0800 Subject: [PATCH] FLUIDSYNTH: Fix build --- audio/softsynth/fluidsynth.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/audio/softsynth/fluidsynth.cpp b/audio/softsynth/fluidsynth.cpp index e41a7515b205..215547c6bad4 100644 --- a/audio/softsynth/fluidsynth.cpp +++ b/audio/softsynth/fluidsynth.cpp @@ -46,7 +46,7 @@ #include "backends/platform/ios7/ios7_common.h" #endif -static void logHandler(int level, const char *message, void *data) { +static void logHandler(int level, char *message, void *data) { switch (level) { case FLUID_PANIC: error("FluidSynth: %s", message); ++++++ 0004-AUDIO-Fix-compilation-with-Fluidsynth2.patch ++++++ >From 6ef406ac20a68f53e66bb98a0c9842dc9553da07 Mon Sep 17 00:00:00 2001 From: Thierry Crozat <cri...@scummvm.org> Date: Tue, 9 Mar 2021 11:32:57 +0000 Subject: [PATCH] AUDIO: Fix compilation with Fluidsynth2 --- audio/softsynth/fluidsynth.cpp | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/audio/softsynth/fluidsynth.cpp b/audio/softsynth/fluidsynth.cpp index 215547c6bad4..3b9f55346437 100644 --- a/audio/softsynth/fluidsynth.cpp +++ b/audio/softsynth/fluidsynth.cpp @@ -46,7 +46,12 @@ #include "backends/platform/ios7/ios7_common.h" #endif -static void logHandler(int level, char *message, void *data) { +#if defined(FLUIDSYNTH_VERSION_MAJOR) && FLUIDSYNTH_VERSION_MAJOR > 1 +static void logHandler(int level, const char *message, void *data) +#else +static void logHandler(int level, char *message, void *data) +#endif +{ switch (level) { case FLUID_PANIC: error("FluidSynth: %s", message); ++++++ 0005-FLUIDSYNTH-Simplify-FluidSynth-version-check.patch ++++++ >From 631b13b5f4990b667e19d9fbdc467f0c873b6c15 Mon Sep 17 00:00:00 2001 From: Le Philousophe <lephilouso...@users.noreply.github.com> Date: Mon, 5 Apr 2021 12:22:51 +0200 Subject: [PATCH] FLUIDSYNTH: Simplify FluidSynth version check --- audio/softsynth/fluidsynth.cpp | 18 +++++++++++++----- 1 file changed, 13 insertions(+), 5 deletions(-) diff --git a/audio/softsynth/fluidsynth.cpp b/audio/softsynth/fluidsynth.cpp index 3b9f55346437..e770fb68543b 100644 --- a/audio/softsynth/fluidsynth.cpp +++ b/audio/softsynth/fluidsynth.cpp @@ -46,7 +46,15 @@ #include "backends/platform/ios7/ios7_common.h" #endif -#if defined(FLUIDSYNTH_VERSION_MAJOR) && FLUIDSYNTH_VERSION_MAJOR > 1 +// We assume here Fluidsynth minor will never be above 255 and +// that micro versions won't break API compatibility +#if defined(FLUIDSYNTH_VERSION_MAJOR) && defined(FLUIDSYNTH_VERSION_MINOR) +#define FS_API_VERSION ((FLUIDSYNTH_VERSION_MAJOR << 8) | FLUIDSYNTH_VERSION_MINOR) +#else +#define FS_API_VERSION 0 +#endif + +#if FS_API_VERSION >= 0x0200 static void logHandler(int level, const char *message, void *data) #else static void logHandler(int level, char *message, void *data) @@ -103,7 +111,7 @@ class MidiDriver_FluidSynth : public MidiDriver_Emulated { void setEngineSoundFont(Common::SeekableReadStream *soundFontData) override; bool acceptsSoundFontData() override { -#if defined(FLUIDSYNTH_VERSION_MAJOR) && FLUIDSYNTH_VERSION_MAJOR > 1 +#if FS_API_VERSION >= 0x0200 return true; #else return false; @@ -162,7 +170,7 @@ void MidiDriver_FluidSynth::setStr(const char *name, const char *val) { // Soundfont memory loader callback functions. -#if defined(FLUIDSYNTH_VERSION_MAJOR) && FLUIDSYNTH_VERSION_MAJOR > 1 +#if FS_API_VERSION >= 0x0200 static void *SoundFontMemLoader_open(const char *filename) { void *p; if (filename[0] != '&') { @@ -200,7 +208,7 @@ int MidiDriver_FluidSynth::open() { fluid_set_log_function(FLUID_INFO, logHandler, NULL); fluid_set_log_function(FLUID_DBG, logHandler, NULL); -#if defined(FLUIDSYNTH_VERSION_MAJOR) && FLUIDSYNTH_VERSION_MAJOR > 1 +#if FS_API_VERSION >= 0x0200 // When provided with in-memory SoundFont data, only use the configured // SoundFont instead if it's explicitly configured on the current game. bool isUsingInMemorySoundFontData = _engineSoundFontData && !ConfMan.getActiveDomain()->contains("soundfont"); @@ -280,7 +288,7 @@ int MidiDriver_FluidSynth::open() { const char *soundfont = !isUsingInMemorySoundFontData ? ConfMan.get("soundfont").c_str() : Common::String::format("&%p", (void *)_engineSoundFontData).c_str(); -#if defined(FLUIDSYNTH_VERSION_MAJOR) && FLUIDSYNTH_VERSION_MAJOR > 1 +#if FS_API_VERSION >= 0x0200 if (isUsingInMemorySoundFontData) { fluid_sfloader_t *soundFontMemoryLoader = new_fluid_defsfloader(_settings); fluid_sfloader_set_callbacks(soundFontMemoryLoader, ++++++ 0006-FLUIDSYNTH-Fix-compilation-with-Fluidsynth-2.2.patch ++++++ >From d0d38b0199dae9a1dbfecc53db54dd0a070694b4 Mon Sep 17 00:00:00 2001 From: Le Philousophe <lephilouso...@users.noreply.github.com> Date: Mon, 5 Apr 2021 12:23:26 +0200 Subject: [PATCH] FLUIDSYNTH: Fix compilation with Fluidsynth 2.2 We replace deprecated functions as well --- audio/softsynth/fluidsynth.cpp | 43 ++++++++++++++++++++++++++++++++++ 1 file changed, 43 insertions(+) diff --git a/audio/softsynth/fluidsynth.cpp b/audio/softsynth/fluidsynth.cpp index e770fb68543b..f01dd8c7362b 100644 --- a/audio/softsynth/fluidsynth.cpp +++ b/audio/softsynth/fluidsynth.cpp @@ -180,11 +180,19 @@ static void *SoundFontMemLoader_open(const char *filename) { return p; } +#if FS_API_VERSION >= 0x0202 +static int SoundFontMemLoader_read(void *buf, fluid_long_long_t count, void *handle) { +#else static int SoundFontMemLoader_read(void *buf, int count, void *handle) { +#endif return ((Common::SeekableReadStream *) handle)->read(buf, count) == (uint32)count ? FLUID_OK : FLUID_FAILED; } +#if FS_API_VERSION >= 0x0202 +static int SoundFontMemLoader_seek(void *handle, fluid_long_long_t offset, int origin) { +#else static int SoundFontMemLoader_seek(void *handle, long offset, int origin) { +#endif return ((Common::SeekableReadStream *) handle)->seek(offset, origin) ? FLUID_OK : FLUID_FAILED; } @@ -193,7 +201,11 @@ static int SoundFontMemLoader_close(void *handle) { return FLUID_OK; } +#if FS_API_VERSION >= 0x0202 +static fluid_long_long_t SoundFontMemLoader_tell(void *handle) { +#else static long SoundFontMemLoader_tell(void *handle) { +#endif return ((Common::SeekableReadStream *) handle)->pos(); } #endif @@ -237,7 +249,11 @@ int MidiDriver_FluidSynth::open() { _synth = new_fluid_synth(_settings); if (ConfMan.getBool("fluidsynth_chorus_activate")) { +#if FS_API_VERSION >= 0x0202 + fluid_synth_chorus_on(_synth, -1, 1); +#else fluid_synth_set_chorus_on(_synth, 1); +#endif int chorusNr = ConfMan.getInt("fluidsynth_chorus_nr"); double chorusLevel = (double)ConfMan.getInt("fluidsynth_chorus_level") / 100.0; @@ -252,22 +268,49 @@ int MidiDriver_FluidSynth::open() { chorusType = FLUID_CHORUS_MOD_TRIANGLE; } +#if FS_API_VERSION >= 0x0202 + fluid_synth_set_chorus_group_nr(_synth, -1, chorusNr); + fluid_synth_set_chorus_group_level(_synth, -1, chorusLevel); + fluid_synth_set_chorus_group_speed(_synth, -1, chorusSpeed); + fluid_synth_set_chorus_group_depth(_synth, -1, chorusDepthMs); + fluid_synth_set_chorus_group_type(_synth, -1, chorusType); +#else fluid_synth_set_chorus(_synth, chorusNr, chorusLevel, chorusSpeed, chorusDepthMs, chorusType); +#endif } else { +#if FS_API_VERSION >= 0x0202 + fluid_synth_chorus_on(_synth, -1, 0); +#else fluid_synth_set_chorus_on(_synth, 0); +#endif } if (ConfMan.getBool("fluidsynth_reverb_activate")) { +#if FS_API_VERSION >= 0x0202 + fluid_synth_reverb_on(_synth, -1, 1); +#else fluid_synth_set_reverb_on(_synth, 1); +#endif double reverbRoomSize = (double)ConfMan.getInt("fluidsynth_reverb_roomsize") / 100.0; double reverbDamping = (double)ConfMan.getInt("fluidsynth_reverb_damping") / 100.0; int reverbWidth = ConfMan.getInt("fluidsynth_reverb_width"); double reverbLevel = (double)ConfMan.getInt("fluidsynth_reverb_level") / 100.0; +#if FS_API_VERSION >= 0x0202 + fluid_synth_set_reverb_group_roomsize(_synth, -1, reverbRoomSize); + fluid_synth_set_reverb_group_damp(_synth, -1, reverbDamping); + fluid_synth_set_reverb_group_width(_synth, -1, reverbWidth); + fluid_synth_set_reverb_group_level(_synth, -1, reverbLevel); +#else fluid_synth_set_reverb(_synth, reverbRoomSize, reverbDamping, reverbWidth, reverbLevel); +#endif } else { +#if FS_API_VERSION >= 0x0202 + fluid_synth_reverb_on(_synth, -1, 0); +#else fluid_synth_set_reverb_on(_synth, 0); +#endif } Common::String interpolation = ConfMan.get("fluidsynth_misc_interpolation");