davy7125 / polyphone

A soundfont editor for quickly designing musical instruments.
https://www.polyphone.io
GNU General Public License v3.0
363 stars 49 forks source link

fix linux build - correctly include jack header (at least openSUSE pk… #84

Closed edogawa23 closed 4 years ago

edogawa23 commented 4 years ago

…g-config --cflags jack doesn't provide proper -I include path)

davy7125 commented 4 years ago

Could you please confirm that you set up correctly the $$PREFIX variable, knowing that in the .pro file there is this include: INCLUDEPATH += $$PREFIX/include/jack If the PREFIX is correct, you shouldn't need to specify jack/jack.h

If #include "jack/jack.h" is the correct way, we should probably also change INCLUDEPATH += $$PREFIX/include/jack into INCLUDEPATH += $$PREFIX/include

edogawa23 commented 4 years ago

You're right, I added the %{_prefix} rpm macro to the call to qmake in my spec file and now it built successfully. Thanks for the hint, I had peeked around but obviously not carefully enough... sorry for the noise and thanks for this great program!