surge-synthesizer / tuning-workbench-synth

A simple JUCE synth which uses our tuning-library
GNU General Public License v3.0
48 stars 8 forks source link

Not building on Arch and OpenSUSE #147

Closed waddafunk closed 1 year ago

waddafunk commented 1 year ago

Currently the synth does not build on Arch and OpenSUSE out of the box since support for gcc 12 has been added in JUCE 7.

Git submodules initiated the JUCE repo pointing to JUCE 6, by changing the repo to the latest JUCE 7 release I was able to build.

I am using Manjaro but this should be a problem for every Arch and OpenSUSE user, since the default for these systems is gcc 12 guessing from this thread.

baconpaul commented 1 year ago

The juce 6 submodule surge ships with has that diff included.

https://github.com/surge-synthesizer/JUCE/commits/3d663fad37799a8c0091a7e60da220adb469f608

so unless you have an error message or somehow didn’t use our submodules I’m not sure why you think the diff isn’t included

baconpaul commented 1 year ago

Oh this is TWS not surge duh sorry.

baconpaul commented 1 year ago

TWS will be fine with juce 7 - we should just update the submodule. Sorry for reading the repo wrong!

waddafunk commented 1 year ago

No problem, happy I could help!

baconpaul commented 1 year ago

(The reason its been so long since an update, by the way, is we put all the features from TWS into Surge XT so most users have migrated there. But a submodule update is easy and I'll do it this AM for users of the synth in old sessions and stuff)

baconpaul commented 1 year ago

OK all updated. Thanks for flagging it and sorry again for reading the wrong repo! Too many projects here :)

Enjoy