Skip to content

Bump clap-helpers to resolve CMP0177 CMake warning#170

Merged
baconpaul merged 2 commits intofree-audio:mainfrom
sudara:cmake_warning
Jan 16, 2026
Merged

Bump clap-helpers to resolve CMP0177 CMake warning#170
baconpaul merged 2 commits intofree-audio:mainfrom
sudara:cmake_warning

Conversation

@sudara
Copy link
Copy Markdown
Contributor

@sudara sudara commented Jan 3, 2026

This CMake warning appears on configure, with angry red text and a warning symbol next to the build:

CLion - 2026-01-03 38@2x

The CMake warning is fixed in clap-helpers, so I've bumped that dependency to the latest main, which resolves the error, see: free-audio/clap-helpers#88

@sudara sudara marked this pull request as draft January 3, 2026 18:43
@sudara
Copy link
Copy Markdown
Contributor Author

sudara commented Jan 3, 2026

Hm, clearly in over my head here. My guess is both clap and clap-helpers must be bumped in sync, so maybe bumping the clap dependency to 1.2.7 would make sense if bumping clap-helpers to main.

I have 0 idea of the impact of all of that, if clap-juce-extensions is on an 2-year old version of clap for Reasons, etc...

@sudara
Copy link
Copy Markdown
Contributor Author

sudara commented Jan 5, 2026

I'll QA, deploy in production, and report back in a week or so :)

@baconpaul
Copy link
Copy Markdown
Collaborator

Oh yeah we can fully go to latest on clap and clap helpers for cje

just laziness that I didn’t - and that cje didn’t use the new features

I’m on vaca now but will merge when I’m back thanks

@baconpaul baconpaul marked this pull request as ready for review January 16, 2026 17:35
@baconpaul baconpaul merged commit 67a063b into free-audio:main Jan 16, 2026
15 checks passed
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.

2 participants