Audio4Linux / JDSP4Linux

An audio effect processor for PipeWire and PulseAudio clients
GNU General Public License v3.0
915 stars 35 forks source link

Build fails on GCC 13/fedora 38 #111

Closed theAeon closed 1 year ago

theAeon commented 1 year ago
n file included from ../../src/data/PresetRule.h:4,
                 from ../../src/data/PresetManager.h:4,
                 from ../../src/data/PresetManager.cpp:1:
../../src/audio/base/IOutputDevice.h:12:17: error: 'uint32_t' was not declared in this scope
   12 |     uint id = ((uint32_t)0xffffffff);
      |                 ^~~~~~~~
../../src/audio/base/IOutputDevice.h:5:1: note: 'uint32_t' is defined in header '<cstdint>'; did you forget to '#include <cstdint>'?
    4 | #include <string>
  +++ |+#include <cstdint>
    5 | 
../../src/audio/base/IOutputDevice.h:12:26: error: expected ')' before numeric constant
   12 |     uint id = ((uint32_t)0xffffffff);
      |                          ^~~~~~~~~~
../../src/audio/base/IOutputDevice.h:12:15: note: to match this '('
   12 |     uint id = ((uint32_t)0xffffffff);
      |               ^
make[1]: *** [Makefile:3598: PresetManager.o] Error 1
make[1]: Leaving directory '/builddir/build/BUILD/JDSP4Linux-2.4/build/src'
make: *** [Makefile:73: sub-src-make_first] Error 2
error: Bad exit status from /var/tmp/rpm-tmp.igo3Pq (%build)

Seems to me you need to explicitly include on GCC 13 now. See also:

https://gcc.gnu.org/gcc-13/porting_to.html https://github.com/llvm/llvm-project/issues/55711

theAeon commented 1 year ago

Dup #100

and I just told someone off for posting dupes too, my bad.