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

macOS Monterey x86_64 compilation error #61

Open
esrsound opened this issue Dec 11, 2021 · 2 comments
Open

macOS Monterey x86_64 compilation error #61

esrsound opened this issue Dec 11, 2021 · 2 comments

Comments

@esrsound
Copy link

esrsound commented Dec 11, 2021

Hello. I trying to compile v2 branch. Getting this error after make dep:

cc -DPD -DHAVE_UNISTD_H -DUSEAPI_DUMMY -I./libpd_wrapper -I./libpd_wrapper/util -I./pure-data/src -DHAVE_LIBDL -I/System/Library/Frameworks/JavaVM.framework/Headers -ffast-math -funroll-loops -fomit-frame-pointer -O3 -DLIBPD_EXTRA -DPDINSTANCE -DPDTHREADS  -DPD_LONGINTTYPE="long long" -mmacosx-version-min=10.9 -stdlib=libc++   -c -o libpd_wrapper/z_hooks.o libpd_wrapper/z_hooks.c
cc -DPD -DHAVE_UNISTD_H -DUSEAPI_DUMMY -I./libpd_wrapper -I./libpd_wrapper/util -I./pure-data/src -DHAVE_LIBDL -I/System/Library/Frameworks/JavaVM.framework/Headers -ffast-math -funroll-loops -fomit-frame-pointer -O3 -DLIBPD_EXTRA -DPDINSTANCE -DPDTHREADS  -DPD_LONGINTTYPE="long long" -mmacosx-version-min=10.9 -stdlib=libc++   -c -o libpd_wrapper/z_libpd.o libpd_wrapper/z_libpd.c
libpd_wrapper/z_libpd.c:91:3: error: implicit declaration of function 'sys_set_audio_api' is invalid in C99 [-Werror,-Wimplicit-function-declaration]
  sys_set_audio_api(API_DUMMY);
  ^
libpd_wrapper/z_libpd.c:91:3: note: did you mean 'sys_get_audio_apis'?
./pure-data/src/s_stuff.h:166:13: note: 'sys_get_audio_apis' declared here
EXTERN void sys_get_audio_apis(char *buf);
            ^
libpd_wrapper/z_libpd.c:163:29: error: too many arguments to function call, expected single argument 'as', have 12 arguments
  sys_set_audio_settings(1, indev, 1, inch,
  ~~~~~~~~~~~~~~~~~~~~~~    ^~~~~~~~~~~~~~~
./pure-data/src/s_stuff.h:153:13: note: 'sys_set_audio_settings' declared here
EXTERN void sys_set_audio_settings(t_audiosettings *as);
            ^
2 errors generated.
make[1]: *** [libpd_wrapper/z_libpd.o] Error 1
make: *** [dep/lib/libpd.a] Error 2
➜  VCV-Prototype git:(v2) ✗
@esrsound esrsound changed the title macOS Monterey x64_32 compilation error macOS Monterey x86_64 compilation error Dec 11, 2021
@mxa
Copy link
Collaborator

mxa commented Dec 12, 2021

Just learned through this issue that there is a v2 branch. Is there an official process how the compatibility to v2 is implemented, who is in charge and what are the changes? @AndrewBelt we need some guidance.

@sleexyz
Copy link

sleexyz commented Jan 12, 2022

I was able to build so far on the v2 branch (on a M1 Mac, Monterey, compiling for x86_64) with just the Faust and LuaJIT script engines. However, I get an error within Rack2 that says No engine found for ..dsp extension (and the same for .lua when I try to load lua scripts).

EDIT: Found the bug. The V2 SDK's system::getExtension returns the leading . (as it uses std::filesystem::path::extension under the hood) whereas V1's string::filenameExtension doesn't.

f693e76 doesn't account for this change, which is why the error message shows two periods.

(I wonder if the change in semantics in the SDK was intentional? Anyways, the std::filesystem::path behavior does seem to make more sense.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants