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

make it build on M2 macOS 13 (probably M1 and 10.13) #93

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

atsushieno
Copy link

I had been trying to build ADLplug/OPNplug on my M2 mac and managed to get it working, so I would share it here.

There was a drawback in the change set though; midi_keyboard_ex.h/.cc does not compile with the latest clang++ from Xcode toolchains (it fails to resolve the class name at JUCE_DECLARE_NON_COPYABLE_WITH_LEAK_DETECTOR). Looks like it is required only to reduce juce_audio_utils, so I only removed the hack around (ex namespace) and simply added the module to the build as I don't see the reason why it had better be cut. If there is any way to preserve it to still compile fine, then I would bring it back, I do not stick to it.

@Wohlstand
Copy link

Thanks!
I took some look, and seems fine.

@Wohlstand
Copy link

Wohlstand commented Oct 12, 2023

However, I can't merge, @jpcima didn't gave me a permission to this repo, and now I can't contact him: he seems disappeared for a while... (After September 2022, he disappeared)

@Wohlstand
Copy link

Wohlstand commented Oct 12, 2023

I guess, I'll fork this repo to apply/receive fixes until the wonder happens and @jpcima comes back.

@atsushieno
Copy link
Author

I have my own fork with a handful of improvements and migrations to JUCE7 there. If it's looking easier to maintain I can give you write access and do whatever that does not cost (much). https://github.com/atsushieno/ADLplug-AE/

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

Successfully merging this pull request may close these issues.

None yet

2 participants