-
-
Notifications
You must be signed in to change notification settings - Fork 68
Pluginval as dependency #141
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
base: develop
Are you sure you want to change the base?
Conversation
89740ac to
1b833be
Compare
dcbe3cf to
db9523e
Compare
|
Am I supposed to be reviewing or merging this? You might notice I switched to CPM for a few other dependancies on develop. Can this branch be pulled in line with that? |
|
@drowaudio I've been using this branch for a year as a dependency of my synth but never got the example plugins running in CI. I'll need to get that going and resolve recent conflicts before it's mergable. |
|
Ah ok. Poke me if/when you get to that. I might switch to juce via CPM at some point soon anyway... |
Opening as a draft for feedback and iteration.
Addresses #137
This improves support for pluginval as a CMake sub-project in a JUCE project.
PLUGINVAL_FETCH_JUCEoption (no longer needed, superseded by not being top level)Open Questions:
Remaining TODO:
Note to Dave and other maintainers:
With CPM, it's always a good idea to have the environment variable CPM_SOURCE_CACHE set — this will let you do offline builds vs. hitting the network on reconfigure. We can also add this optimization in CI too.
Add the following to .bashrc or .zshrc:
The submodule is gone, so you can optionally clean up your .git/config folder, you can run: