From 3f859c10afd3c5ad5c0f3b7978acf2f97464ca79 Mon Sep 17 00:00:00 2001 From: Bernd Kuhls Date: Tue, 6 Oct 2026 18:20:18 +0200 Subject: [PATCH] package/luvi: fix build without c++ MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Buildroot commit 00317f0aff243986983650671fd220ab040af50f bumped the package to 2.15.0 which first included upstream commit https://github.com/luvit/luvi/commit/262d5f7dff3484285cd8b3bf2ed32ff0f13c8741 that added the miniz submodule. The version of miniz used by luvi contains commit https://github.com/richgel999/miniz/commit/05ab4dc05c9d1e1f951f5849b659e2c7e291a620 which removed "C" from the project variable causing a dependency to c++ which causes build errors when the toolchain does not support c++: -- Check for working CXX compiler: /bin/false - broken This dependency was not intended so we add an upstream commit to fix the problem. Fixes: https://autobuild.buildroot.net/results/2a5/2a5cf09dc9303b11e2a2769c5b0b319284ec1bbb/ Signed-off-by: Bernd Kuhls Signed-off-by: Thomas Petazzoni (cherry picked from commit d267b59fa9219f3a6127dd687cf68a0c51e189ac) Signed-off-by: Raphaël Mélotte --- ...eLists.txt-only-require-a-C-compiler.patch | 45 +++++++++++++++++++ 1 file changed, 45 insertions(+) create mode 100644 package/luvi/0003-CMakeLists.txt-only-require-a-C-compiler.patch diff --git a/package/luvi/0003-CMakeLists.txt-only-require-a-C-compiler.patch b/package/luvi/0003-CMakeLists.txt-only-require-a-C-compiler.patch new file mode 100644 index 0000000000..6f4ecdea80 --- /dev/null +++ b/package/luvi/0003-CMakeLists.txt-only-require-a-C-compiler.patch @@ -0,0 +1,45 @@ +From c6a1f6a3650e8487e506080eb34a244e75e37a78 Mon Sep 17 00:00:00 2001 +From: Thomas Devoogdt +Date: Tue, 1 Apr 2025 22:56:00 +0200 +Subject: [PATCH] CMakeLists.txt: only require a C compiler + +Commit 05ab4dc05c9d1e1f951f5849b659e2c7e291a620 +"Add some catch2 tests" dropped that C flag, +but should instead only have added a simple +enable_language call for tests only. + +Signed-off-by: Thomas Devoogdt +Upstream: https://github.com/richgel999/miniz/commit/383e551cccf41fc2ce1177712d6a37a85d936642 +Signed-off-by: Bernd Kuhls +--- + CMakeLists.txt | 5 +++-- + 1 file changed, 3 insertions(+), 2 deletions(-) + +diff --git a/deps/miniz/CMakeLists.txt b/deps/miniz/CMakeLists.txt +index 8978910..55d1cde 100644 +--- a/deps/miniz/CMakeLists.txt ++++ b/deps/miniz/CMakeLists.txt +@@ -7,10 +7,10 @@ if(DEFINED PROJECT_NAME) + endif() + + if(CMAKE_MINOR_VERSION LESS 12) +- project(miniz) ++ project(miniz C) + # see issue https://gitlab.kitware.com/cmake/cmake/merge_requests/1799 + else() +- project(miniz) ++ project(miniz C) + set(CMAKE_C_STANDARD 90) + set(CMAKE_VERBOSE_MAKEFILE ON) + # set(CMAKE_C_VISIBILITY_PRESET hidden) +@@ -298,6 +298,7 @@ if(BUILD_FUZZERS) + endif() + + if(BUILD_TESTS) ++ enable_language(CXX) + set(CMAKE_CXX_STANDARD 20) + set(CMAKE_CXX_STANDARD_REQUIRED YES) + +-- +2.55.0 +