On 2020/04/21 18:11, Charlene Wendling wrote: > > fluidsynth can't be built in the current macppc bulk: > > > ../src/utils/fluidsynth_priv.h:32:10: fatal error: 'config_maxmsp43.h' file > > not found > > This is happening because clang defines '__POWERPC__', but not gcc: > > $ /usr/bin/gcc -dM -E - < /dev/null | grep __POWERPC__ > $ /usr/bin/cc -dM -E - < /dev/null | grep __POWERPC__ > #define __POWERPC__ 1 > > The below diff reverts to the gcc "behaviour" and allows to build > fluidsynth on macppc [0]. > > Comments/feedback are welcome, > > Charlène. > > > [0] https://bin.charlenew.xyz/macppc/packages/fluidsynth-1.1.6p5.log > > > Index: patches/patch-src_utils_fluidsynth_priv_h > =================================================================== > RCS file: patches/patch-src_utils_fluidsynth_priv_h > diff -N patches/patch-src_utils_fluidsynth_priv_h > --- /dev/null 1 Jan 1970 00:00:00 -0000 > +++ patches/patch-src_utils_fluidsynth_priv_h 21 Apr 2020 16:04:39 -0000 > @@ -0,0 +1,18 @@ > +$OpenBSD$ > + > +Don't use an header that is not provided > + > +Index: src/utils/fluidsynth_priv.h > +--- src/utils/fluidsynth_priv.h.orig > ++++ src/utils/fluidsynth_priv.h > +@@ -28,10 +28,6 @@ > + #include "config.h" > + #endif > + > +-#if defined(__POWERPC__) && !(defined(__APPLE__) && defined(__MACH__)) > +-#include "config_maxmsp43.h" > +-#endif > +- > + #if defined(WIN32) && !defined(MINGW32) > + #include "config_win32.h" > + #endif >
OK with me. BTW the offending line is gone in newer versions of fluidsynth. If anyone is interested in updating this port, mail me, I have a partial diff for 2.1.2 with the patches merged, but between the move to CMake and changes in how the audio output drivers are hooked in something isn't quite right so it needs poking at a bit.
