Don't understand why handling of globals fails while compiling under Linux

[octetta]

Building AMY on Fedora Linux 37 yields the following errors:

gcc  src/amy-example.o  src/algorithms.o  src/amy.o  src/envelope.o  src/filters.o  src/oscillators.o  src/pcm.o  src/partials.o  src/libsoundio-audio.o  src/amy-example-esp32.o -Wall -lpthread -lsoundio -lm  -o amy-example
/usr/bin/ld: src/libsoundio-audio.o:/home/stewartj/pr/amy/src/libsoundio-audio.h:9: multiple definition of `amy_channel'; src/amy-example.o:/home/stewartj/pr/amy/src/libsoundio-audio.h:9: first defined here
/usr/bin/ld: src/libsoundio-audio.o:/home/stewartj/pr/amy/src/libsoundio-audio.h:10: multiple definition of `amy_device_id'; src/amy-example.o:/home/stewartj/pr/amy/src/libsoundio-audio.h:10: first defined here
/usr/bin/ld: src/libsoundio-audio.o:/home/stewartj/pr/amy/src/libsoundio-audio.h:11: multiple definition of `amy_running'; src/amy-example.o:/home/stewartj/pr/amy/src/libsoundio-audio.h:11: first defined here
collect2: error: ld returned 1 exit status
make: *** [Makefile:38: amy-example] Error 1

I can change these three variables to extern in libsoundio-audio.h and the code compiles, but I'm worried this isn't the original intention and might lead to other problems as I work on getting the running on my Linux system.

[bwhitman]

Thanks for this. I ran into this problem myself just last night on a different gcc than we were using to develop AMY/alles.

You did the right fix; I will move all the global-ish decls out of the headers.

[bwhitman]

Just committed the fix that works for me so far. Thank you!