-
Notifications
You must be signed in to change notification settings - Fork 3.7k
Mosquitto cmake min version fix #27801
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
Conversation
Patch CMakeLists.txt for cmake 4.x compatibility. New cmake versions require at least 3.5 as 'cmake_minimum_required' in CMakeLists.txt. In future 3.10 will be required. Link: openwrt#20265 Signed-off-by: Rani Hod <[email protected]>
|
I feel that for mosquitto, the better solution would be to update the package itself from 2.0.18 (of Sep 2023) to 2.0.22 (of July 2025) which already has the minimum cmake bump built-in |
|
Fair enough. Should I close this PR then? |
|
Well, in meanwhile I already tried to do the mosquitto update by myself, but it seems to require first the libwebsockets upgrade, and possibly something else. Just leave the PR open for a while. This might still be stopgap solution. EDIT: Curiously, the SSL-version compiles ok, but nossl version of 2.0.22 complains about openssl... EDIT2: |
I actually compiled and installed the nossl version on my device (not a complete build, just as a single package). Build log |
You compiled the old 2.0.18, not the 2.0.22 that I was talking about. It seems to need also this backport to 2.0.22: |
📦 Package Details
Maintainer: @karlp
Description: Fix CMake min version. See openwrt/openwrt#20265.
🧪 Run Testing Details
ramips/mt7621wodesys,wd-r1802u✅ Formalities
If your PR contains a patch:
git am(e.g., subject line, commit description, etc.)
We must try to upstream patches to reduce maintenance burden.