Skip to content

Conversation

@sudara
Copy link
Collaborator

@sudara sudara commented Dec 4, 2024

Closes #137

This improves support for pluginval as a CMake sub-project in a JUCE project.

  • Moves JUCE from submodule to CPM dependency on 8.0.11
  • Only adds JUCE when top level
  • Only sets deployment target when top level
  • Only generate docs when top level
  • Removes PLUGINVAL_FETCH_JUCE option (no longer needed, superseded by not being top level)
  • Adds documentation
  • Adds some conditionals around 8.0.11 deprecations to allow earlier versions of JUCE.

Caveats:

Technically, this PR would make maintenance more complicated — it's now BYOJ, Bring Your Own JUCE — which means pluginval will be compiled against more than one version of JUCE. We could add a caveat in the README that there's no guarantees on other JUCE versions than what pluginval depends on, specify that it's JUCE 8 only, etc...

@sudara sudara force-pushed the use-as-dependency branch 2 times, most recently from 89740ac to 1b833be Compare December 4, 2024 17:30
@sudara sudara changed the title Use as dependency Pluginval as dependency Dec 4, 2024
@sudara sudara marked this pull request as ready for review December 5, 2024 21:11
@sudara sudara marked this pull request as draft December 5, 2024 21:13
@sudara sudara force-pushed the use-as-dependency branch from dcbe3cf to db9523e Compare July 11, 2025 22:04
@drowaudio
Copy link
Contributor

Am I supposed to be reviewing or merging this?
I like the feature set but it seems to have not changed much in the last year? Is it stable enough to go in?

You might notice I switched to CPM for a few other dependancies on develop. Can this branch be pulled in line with that?

@sudara
Copy link
Collaborator Author

sudara commented Dec 1, 2025

@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.

@drowaudio
Copy link
Contributor

Ah ok. Poke me if/when you get to that.

I might switch to juce via CPM at some point soon anyway...

@sudara sudara force-pushed the use-as-dependency branch from dc9e598 to b596c78 Compare January 16, 2026 20:57
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.

Make "pluginval as a dependency" a 1st class citizen

3 participants