Skip to content

Conversation

madebr
Copy link
Contributor

@madebr madebr commented Sep 4, 2025

This was changed in FluidSynth/fluidsynth#1546

Closes #755

@sezero
Copy link
Contributor

sezero commented Sep 4, 2025

As far as I can see this is a cheat / a bandaid n our part to workaround mainstream's breakage, and a cmake-specific one at that? What does their *.pc file look like? Becuease if it says -lfludsynth, other build systems, autotools for e..g., are broken as well, and we should wait for mainstream to fix their sh^w^w thing instead and not do this?

@madebr
Copy link
Contributor Author

madebr commented Sep 4, 2025

Looks like they require the suffix:

prefix=/mingw64
exec_prefix=${prefix}
libdir=${exec_prefix}/lib
includedir=${prefix}/include

Name: FluidSynth
Description: Software SoundFont synth
Version: 2.4.8
Requires.private: glib-2.0 gthread-2.0 sndfile portaudio-2.0 gmodule-2.0 sdl3 readline
Libs: -L${libdir} -lfluidsynth-3
Libs.private: -ldsound -lksuser -lwinmm -lole32 -lws2_32 ${prefix}/lib/libgomp.dll.a ${prefix}/lib/libmingwthrd.a
Cflags: -I${includedir}

@madebr madebr marked this pull request as draft September 4, 2025 00:46
@sezero
Copy link
Contributor

sezero commented Sep 4, 2025

Looks like they require the suffix:

Geez.. Where the hell did they get that idea...

@sezero
Copy link
Contributor

sezero commented Sep 4, 2025

Mainstrean issue: FluidSynth/fluidsynth#1645

Looks like they require the suffix:

Geez.. Where the hell did they get that idea...

Looks like it happened in FluidSynth/fluidsynth#1548 as a followup
to FluidSynth/fluidsynth#1546

@madebr madebr mentioned this pull request Sep 4, 2025
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.

MingW broken?
2 participants