Upstreamed the SNDIO implementation patch that was only in the port for some reason.
Created attachment 199491 [details] patch
Created attachment 199492 [details] patch
Created attachment 199493 [details] patch
I can confirm that Sndio appears to still be functional after this change. audio/qsynth works fine with this update too, but please be aware that this update breaks at least audio/lmms (see bug #233462). I have not tested other consumers.
(In reply to Tobias Kortkamp from comment #4) Thank you, Tobias, for checking dependencies. ----- There is one unexpected problem which I didn't realize existed: fluid_sndio.c has a custom license. FluidSynth needs it to be changed to LGPL in order for this code to be adopted. I contacted Jacob Meuser asking for change of license, but there is no guarantee that he will answer. If there will be no answer, we will have to keep the sndio in the port. It didn't occur to me to check the license when I submitted the patch to the FluidSynth upstream.
Created attachment 199520 [details] patch * Upstream the remaining FreeBSD patches * Remove the OSS patch that doesn't seem to matter any more
Created attachment 201668 [details] fluidsynth.diff Update to FluidSynth 2.0.3. Upstream closed the sndio patch upstreaming PR. I guess we will have to keep these in the tree. I had to refactor sndio support a bit for FluidSynth 2.0.3. The previous attempt [1] for upstreaming the other patches was closed too. I've opened another PR [2] which addresses their comments. [1] https://github.com/FluidSynth/fluidsynth/pull/472 [2] https://github.com/FluidSynth/fluidsynth/pull/508
Created attachment 201669 [details] fluidsynth.diff Remove enable-ladcca which doesn't do anything anymore.
A commit references this bug: Author: riggs Date: Sat Mar 2 16:47:00 UTC 2019 New revision: 494417 URL: https://svnweb.freebsd.org/changeset/ports/494417 Log: Update to upstream version 2.0.3 While on it: - Pet portlint PR: 233448 Submitted by: yuri Reviewed by: tobik Changes: head/audio/fluidsynth/Makefile head/audio/fluidsynth/distinfo head/audio/fluidsynth/files/fluid_sndio.c head/audio/fluidsynth/files/patch-CMakeLists.txt head/audio/fluidsynth/files/patch-src_CMakeLists.txt head/audio/fluidsynth/files/patch-src_bindings_fluid__rtkit.c head/audio/fluidsynth/files/patch-src_config.cmake head/audio/fluidsynth/files/patch-src_drivers_fluid__adriver.c head/audio/fluidsynth/files/patch-src_drivers_fluid__adriver.h head/audio/fluidsynth/files/patch-src_drivers_fluid__mdriver.c head/audio/fluidsynth/files/patch-src_drivers_fluid__mdriver.h head/audio/fluidsynth/files/patch-src_drivers_fluid__oss.c head/audio/fluidsynth/pkg-plist
A commit references this bug: Author: riggs Date: Sat Mar 2 16:55:58 UTC 2019 New revision: 494418 URL: https://svnweb.freebsd.org/changeset/ports/494418 Log: Chase update of fluidsynth to 2.0.3 PR: 233448 Changes: head/audio/alure/Makefile head/audio/bristol/Makefile head/audio/calf-lv2/Makefile head/audio/carla/Makefile head/audio/csound/Makefile head/audio/denemo/Makefile head/audio/drumstick/Makefile head/audio/fluidplug-lv2/Makefile head/audio/fluidsynth-dssi/Makefile head/audio/lmms/Makefile head/audio/musicpd/Makefile head/audio/qsynth/Makefile head/audio/sdl2_mixer/Makefile head/audio/soundkonverter/Makefile head/audio/tuxguitar/Makefile head/games/doomsday/Makefile head/games/exult/Makefile head/games/gzdoom/Makefile head/games/scummvm/Makefile head/games/slade/Makefile head/games/stratagus/Makefile head/games/zdoom/Makefile head/multimedia/audacious-plugins/Makefile head/multimedia/audacious-plugins-gtk3/Makefile head/multimedia/vlc/Makefile
Thanks for the great work on this one!