mtytel / helm

Helm - a free polyphonic synth with lots of modulation
http://tytel.org/helm
GNU General Public License v3.0
2.36k stars 200 forks source link

Build fails with glibc 2.26 on 32-bit Intel #190

Open andreasbaumann opened 6 years ago

andreasbaumann commented 6 years ago

Experienced on Archlinux32 (32-bit Intel):

../../../JUCE/modules/juce_audio_formats/codecs/flac/libFLAC/cpu.c:250:3: error: ‘__sigemptyset’ was not declared in this scope
   __sigemptyset(&sigill_sse.sa_mask);
   ^~~~~~~~~~~~~
../../../JUCE/modules/juce_audio_formats/codecs/flac/libFLAC/cpu.c:250:3: note: suggested alternative: ‘sigemptyset’
   __sigemptyset(&sigill_sse.sa_mask);
   ^~~~~~~~~~~~~
   sigemptyset

__sigemptyset is an internal GNU function and should not be use outside glibc itself.

I recall it has been deprecated from the public API in glibc 2.26.

      #ifdef __ANDROID__
        sigemptyset (&sigill_sse.sa_mask);
      #else
                __sigemptyset(&sigill_sse.sa_mask);
      #endif

should be:

sigemptyset (&sigill_sse.sa_mask);

IMHO :-)

mtytel commented 6 years ago

Ah, thanks for the report. Since this in the JUCE library, probably would be best to send it upstream to the JUCE library. https://github.com/WeAreROLI/JUCE/

andreasbaumann commented 6 years ago

Ah. Sorry. Didn't realize this is in JUCE. :-) Will report upstream..

andreasbaumann commented 6 years ago

https://github.com/WeAreROLI/JUCE/issues/374