Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

surge-xt 1.0.1 doesn't build from AUR #6307

Closed
abique opened this issue Jun 23, 2022 · 5 comments
Closed

surge-xt 1.0.1 doesn't build from AUR #6307

abique opened this issue Jun 23, 2022 · 5 comments
Labels
Bug Report Item submitted using the Bug Report template

Comments

@abique
Copy link
Collaborator

abique commented Jun 23, 2022

Here the error when building from AUR:

[ 67%] Building CXX object src/common/CMakeFiles/surge-common.dir/dsp/filters/NonlinearFeedback.cpp.o
cd /home/abique/.cache/yay/surge-xt/src/surge-xt-1.0.1/build/src/common && /bin/c++ -DGHC_FILESYSTEM_FWD -DGHC_RAISE_UNICODE_ERRORS -DGHC_WIN_WSTRING_STRING_TYPE -DJUCE_ALLOW_STATIC_NULL_VARIABLES=0 -DJUCE_ALSA=1 -DJUCE_CATCH_UNHANDLED_EXCEPTIONS=0 -DJUCE_COREGRAPHICS_DRAW_ASYNC=1 -DJUCE_DIRECTSOUND=1 -DJUCE_DISPLAY_SPLASH_SCREEN=0 -DJUCE_GLOBAL_MODULE_SETTINGS_INCLUDED=1 -DJUCE_JACK=1 -DJUCE_MODAL_LOOPS_PERMITTED=0 -DJUCE_MODULE_AVAILABLE_juce_audio_basics=1 -DJUCE_MODULE_AVAILABLE_juce_audio_formats=1 -DJUCE_MODULE_AVAILABLE_juce_core=1 -DJUCE_MODULE_AVAILABLE_juce_dsp=1 -DJUCE_REPORT_APP_USAGE=0 -DJUCE_STANDALONE_APPLICATION=0 -DJUCE_STRICT_REFCOUNTEDPOINTER=1 -DJUCE_USE_CAMERA=disabled -DJUCE_USE_CURL=0 -DJUCE_VST3_CAN_REPLACE_VST2=0 -DJUCE_WASAPI=1 -DJUCE_WEB_BROWSER=0 -DLINUX=1 -DNDEBUG=1 -DTIXML_USE_STL -D_NDEBUG=1 -I/home/abique/.cache/yay/surge-xt/src/surge-xt-1.0.1/src/common -I/home/abique/.cache/yay/surge-xt/src/surge-xt-1.0.1/src/common/dsp -I/home/abique/.cache/yay/surge-xt/src/surge-xt-1.0.1/src/common/dsp/effects -I/home/abique/.cache/yay/surge-xt/src/surge-xt-1.0.1/src/common/dsp/filters -I/home/abique/.cache/yay/surge-xt/src/surge-xt-1.0.1/src/common/dsp/modulators -I/home/abique/.cache/yay/surge-xt/src/surge-xt-1.0.1/src/common/dsp/oscillators -I/home/abique/.cache/yay/surge-xt/src/surge-xt-1.0.1/src/common/dsp/utilities -I/home/abique/.cache/yay/surge-xt/src/surge-xt-1.0.1/src/common/dsp/vembertech -I/home/abique/.cache/yay/surge-xt/src/surge-xt-1.0.1/libs/simde -I/home/abique/.cache/yay/surge-xt/src/surge-xt-1.0.1/libs/fmt/include -I/home/abique/.cache/yay/surge-xt/src/surge-xt-1.0.1/libs/LuaJitLib/LuaJIT/src -I/home/abique/.cache/yay/surge-xt/src/surge-xt-1.0.1/libs/libsamplerate/include -I/home/abique/.cache/yay/surge-xt/src/surge-xt-1.0.1/libs/airwindows/include -I/home/abique/.cache/yay/surge-xt/src/surge-xt-1.0.1/libs/eurorack/eurorack -I/home/abique/.cache/yay/surge-xt/src/surge-xt-1.0.1/libs/eurorack/special_include -I/home/abique/.cache/yay/surge-xt/src/surge-xt-1.0.1/build/libs/filesystem/include -I/home/abique/.cache/yay/surge-xt/src/surge-xt-1.0.1/libs/oddsound-mts/MTS-ESP/Client -I/home/abique/.cache/yay/surge-xt/src/surge-xt-1.0.1/libs/sqlite-3.23.3/. -I/home/abique/.cache/yay/surge-xt/src/surge-xt-1.0.1/libs/strnatcmp -I/home/abique/.cache/yay/surge-xt/src/surge-xt-1.0.1/libs/tinyxml/include -I/home/abique/.cache/yay/surge-xt/src/surge-xt-1.0.1/build/src/common/juce_binarydata_surge-common-binary/JuceLibraryCode -I/home/abique/.cache/yay/surge-xt/src/surge-xt-1.0.1/libs/tuning-library/include -I/home/abique/.cache/yay/surge-xt/src/surge-xt-1.0.1/build/src/lua/include -I/home/abique/.cache/yay/surge-xt/src/surge-xt-1.0.1/src/platform/include -I/home/abique/.cache/yay/surge-xt/src/surge-xt-1.0.1/libs/PEGTL/include -I/home/abique/.cache/yay/surge-xt/src/surge-xt-1.0.1/libs/JUCE/modules -march=native -mtune=generic -O2 -pipe -fno-plt -fexceptions         -Wp,-D_FORTIFY_SOURCE=2 -Wformat -Werror=format-security         -fstack-clash-protection -fcf-protection -Wp,-D_GLIBCXX_ASSERTIONS -O3 -DNDEBUG -fPIC -Wno-multichar -Werror -fvisibility=hidden -fvisibility-inlines-hidden -Wformat-truncation=0 -Wno-free-nonheap-object -Wno-return-local-addr -std=c++17 -MD -MT src/common/CMakeFiles/surge-common.dir/dsp/filters/NonlinearFeedback.cpp.o -MF CMakeFiles/surge-common.dir/dsp/filters/NonlinearFeedback.cpp.o.d -o CMakeFiles/surge-common.dir/dsp/filters/NonlinearFeedback.cpp.o -c /home/abique/.cache/yay/surge-xt/src/surge-xt-1.0.1/src/common/dsp/filters/NonlinearFeedback.cpp
/home/abique/.cache/yay/surge-xt/src/surge-xt-1.0.1/src/common/dsp/effects/chowdsp/spring_reverb/SpringReverbProc.cpp: In member function ‘void chowdsp::SpringReverbProc::setParams(const Params&, int)’:
/home/abique/.cache/yay/surge-xt/src/surge-xt-1.0.1/src/common/dsp/effects/chowdsp/spring_reverb/SpringReverbProc.cpp:85:75: error: cannot convert ‘__m128’ to ‘juce::dsp::SIMDRegister<float>’
   85 |         apf.setParams(msToSamples(0.35f + 3.0f * params.size), _mm_load_ps(apfGVec));
      |                                                                ~~~~~~~~~~~^~~~~~~~~
      |                                                                           |
      |                                                                           __m128
In file included from /home/abique/.cache/yay/surge-xt/src/surge-xt-1.0.1/src/common/dsp/effects/chowdsp/spring_reverb/SpringReverbProc.h:24,
                 from /home/abique/.cache/yay/surge-xt/src/surge-xt-1.0.1/src/common/dsp/effects/chowdsp/spring_reverb/SpringReverbProc.cpp:2:
/home/abique/.cache/yay/surge-xt/src/surge-xt-1.0.1/src/common/dsp/effects/chowdsp/spring_reverb/SchroederAllpass.h:30:39: note:   initializing argument 2 of ‘void chowdsp::SchroederAllpass<T, order>::setParams(float, T) [with T = juce::dsp::SIMDRegister<float>; int order = 2]’
   30 |     void setParams(float delaySamp, T feedback)
      |                                     ~~^~~~~~~~

Here is the package: https://aur.archlinux.org/packages/surge-xt

@abique abique added the Bug Report Item submitted using the Bug Report template label Jun 23, 2022
@baconpaul
Copy link
Collaborator

Looks like AUR is adding an avx compile flag and hitting this.

#5993

why would a distro change compile flags? It shouldn’t!

@abique
Copy link
Collaborator Author

abique commented Jun 23, 2022

Those are the flags in /etc/makepkg.conf:

CFLAGS="-march=native -mtune=generic -O2 -pipe -fno-plt -fexceptions \
        -Wp,-D_FORTIFY_SOURCE=2 -Wformat -Werror=format-security \
        -fstack-clash-protection -fcf-protection"
CXXFLAGS="$CFLAGS -Wp,-D_GLIBCXX_ASSERTIONS"
LDFLAGS="-Wl,-O1,--sort-common,--as-needed,-z,relro,-z,now"

I'm not sure if I tweaked that, but it contains a -march=native, maybe this is the problem?

@baconpaul
Copy link
Collaborator

Yeah surge won’t compile with thst

@abique
Copy link
Collaborator Author

abique commented Jun 23, 2022

So then that was a user error!
Sorry for that.

@abique abique closed this as completed Jun 23, 2022
@baconpaul
Copy link
Collaborator

No problem! We should fix it of course. One day. But there's this project called CLAP which has been keeping me busy. Maybe you've heard of it. It's a cool new.... no wait! :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Report Item submitted using the Bug Report template
Projects
None yet
Development

No branches or pull requests

2 participants