-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Prepare LE to build with cmake 4.0.0 #9916
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
Comments
You may or may not be aware of https://cmake.org/cmake/help/latest/variable/CMAKE_POLICY_VERSION_MINIMUM.html It allows to set a "minimum" that will override what a cmake project has its min version at
Essentially avoiding having to patch for projects that arent actively maintained (lzo2 is a prime example from kodi perspective). Cmake state the following in their pre release notes about the new variable
Do be aware, cmake devs have stated they intend to make further breaking changes from the 3.x branch (deprecated removals), so setting the minimum to 3.5 in the example above, may need to be revisited in future cmake 4 version bumps if they drop deprecated features in > 3.5 |
I did think about it. I’m regretting not patching some of the gcc-12,13,14… workarounds I had made when we bumped previously, as remembering to clean them up / or revisiting the CFLAGS. So have been on the upstream patch bandwagon for both the gcc15 and now the cmake4. Some of the patches I have written are 3.10 targeted, which seems to be the not yet deprecated version. When there has been inconsistencies in the version (aka gmmlib and media-driver) then I have just been updating to the consistent. |
All good. Internally in kodi for other platforms i'll just be using the override for things like lzo. Just wanted to make sure it was known about a way so you dont have to patch things that are abandoned (or less maintained) upstream and then carry more patches for good |
You are right. Once we get down to the patches that are/will be accepted, then agree will be time to use the |
@fuzzard - 29 patches required - so not to bad, pretty much all upstreamed, most were simple patches, a couple required rework of the CMakeLists.txt. Let’s see how much traction we get on the upstream merges. |
Uh oh!
There was an error while loading. Please reload this page.
Upstream repo already in repo
libarchiveUpstream patches raised
vdr-plugin-robotv: allow build with cmake-4.0.0yajl: allow build with cmake-4.0.0audiodecoder.dumb: allow build with cmake-4.0.0librespotTBA
Local patch
The text was updated successfully, but these errors were encountered: