[Buildroot] [PATCH 1/1] set minimum cmake version to 3.10

[Buildroot] [PATCH 1/1] set minimum cmake version to 3.10

[Adam Duskett]

The CMakeLists.txt file uses the CMP0091 which is an MSVC runtime library flag
abstraction macro.

Because we are not building ninja for Microsoft, it is safe to remove this
macro and set the minimum version to 3.10.

Fixes:
http://autobuild.buildroot.net/results/992b34c5625ec733d8dce678aa7a7540c4768ca1

Signed-off-by: Adam Duskett <aduskett@gmail.com>
---
 ...01-set-minimum-cmake-version-to-3.10.patch | 30 +++++++++++++++++++
 1 file changed, 30 insertions(+)
 create mode 100644 package/ninja/0001-set-minimum-cmake-version-to-3.10.patch

diff --git a/package/ninja/0001-set-minimum-cmake-version-to-3.10.patch b/package/ninja/0001-set-minimum-cmake-version-to-3.10.patch
new file mode 100644
index 0000000000..d75b78fde9
--- /dev/null
+++ b/package/ninja/0001-set-minimum-cmake-version-to-3.10.patch
@@ -0,0 +1,30 @@
+From 4645f74eb668cc1cf876d1e9bef933591da7575c Mon Sep 17 00:00:00 2001
+From: Adam Duskett <aduskett@gmail.com>
+Date: Mon, 3 Feb 2020 06:21:57 -0800
+Subject: [PATCH] set minimum cmake version to 3.10
+
+The CMakeLists.txt file uses the CMP0091 which is an MSVC runtime library flag
+abstraction macro.
+
+Because we are not building ninja for Microsoft, it is safe to remove this
+macro and set the minimum version to 3.10.
+
+Signed-off-by: Adam Duskett <aduskett@gmail.com>
+---
+ CMakeLists.txt | 3 +--
+ 1 file changed, 1 insertion(+), 2 deletions(-)
+
+diff --git a/CMakeLists.txt b/CMakeLists.txt
+index 60fd8a1..a040e0b 100644
+--- a/CMakeLists.txt
++++ b/CMakeLists.txt
+@@ -1,5 +1,4 @@
+-cmake_minimum_required(VERSION 3.15)
+-cmake_policy(SET CMP0091 NEW)
++cmake_minimum_required(VERSION 3.10)
+ project(ninja)
+ 
+ if(CMAKE_BUILD_TYPE MATCHES "Release")
+-- 
+2.24.1
+
-- 
2.24.1

[Buildroot] [PATCH 1/1] set minimum cmake version to 3.10

[Thomas Petazzoni]

On Mon,  3 Feb 2020 06:27:38 -0800
Adam Duskett <aduskett@gmail.com> wrote:

> The CMakeLists.txt file uses the CMP0091 which is an MSVC runtime library flag
> abstraction macro.
> 
> Because we are not building ninja for Microsoft, it is safe to remove this
> macro and set the minimum version to 3.10.
> 
> Fixes:
> http://autobuild.buildroot.net/results/992b34c5625ec733d8dce678aa7a7540c4768ca1
> 
> Signed-off-by: Adam Duskett <aduskett@gmail.com>
> ---
>  ...01-set-minimum-cmake-version-to-3.10.patch | 30 +++++++++++++++++++
>  1 file changed, 30 insertions(+)
>  create mode 100644 package/ninja/0001-set-minimum-cmake-version-to-3.10.patch

Fixed the commit title and applied. Thanks!

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

[Buildroot] [PATCH 1/1] set minimum cmake version to 3.10

[Peter Korsgaard]

>>>>> "Adam" == Adam Duskett <aduskett@gmail.com> writes:

 > The CMakeLists.txt file uses the CMP0091 which is an MSVC runtime library flag
 > abstraction macro.

 > Because we are not building ninja for Microsoft, it is safe to remove this
 > macro and set the minimum version to 3.10.

There's unfortunately still another issue:

http://autobuild.buildroot.net/results/615/6150430d396b74a81be87d942ca571c7c15dfc52/build-end.log

CMakeLists.txt hardcodes -fdiagnostic-color, which is only supported
from gcc 4.9, breaking builds on old autobuilders:

https://gcc.gnu.org/gcc-4.9/changes.html

We don't really care about coloured error messages, so I guess we can
just patch away that line.

Care to send a patch doing that?

-- 
Bye, Peter Korsgaard