[Buildroot] [PATCH v3] package/ninja: fix to be cmake-3.10 compatible again

* [Buildroot] [PATCH v3] package/ninja: fix to be cmake-3.10 compatible again
@ 2020-12-12 20:54 Peter Seiderer
  2020-12-23 14:18 ` Philippe REYNES
  2020-12-25 15:28 ` [Buildroot] [PATCH] package/go: enable ARMv7 optimizations for ARMv8 target Yann E. MORIN
  0 siblings, 2 replies; 6+ messages in thread
From: Peter Seiderer @ 2020-12-12 20:54 UTC (permalink / raw)
  To: buildroot

Use cmake-3.10 conform COMPILE_FLAGS instead of 3.15 INCLUDE_DIRECTORIES
property (as this one is unknown/unused for older cmake versions and
has no effect, resuling in a build failure because of missing extra
include path).

Fixes:

  - http://lists.busybox.net/pipermail/buildroot/2020-December/298159.html

  [ 96%] Building CXX object CMakeFiles/ninja.dir/src/browse.cc.o
  .../build/host-ninja-1.10.2/src/browse.cc:23:10: fatal error: build/browse_py.h: No such file or directory
     23 | #include "build/browse_py.h"
        |          ^~~~~~~~~~~~~~~~~~~
  compilation terminated.

Reported-by: Yegor Yefremov <yegorslists@googlemail.com>
Signed-off-by: Peter Seiderer <ps.report@gmx.net>
Reviewed-by: Yegor Yefremov <yegorslists@googlemail.com>
---
Changes v1 -> v2:
  - really use COMPILE_FLAGS (as the patch subject says)
    (suggested by Yegor Yefremov)

Changes v2 -> v3:
  - add reported-by, reviewed-by Yegor Yefremov
  - update commit description
---
 ...onform-COMPILE_FLAGS-instead-of-3.15.patch | 40 +++++++++++++++++++
 1 file changed, 40 insertions(+)
 create mode 100644 package/ninja/0002-use-cmake-3.10-conform-COMPILE_FLAGS-instead-of-3.15.patch

diff --git a/package/ninja/0002-use-cmake-3.10-conform-COMPILE_FLAGS-instead-of-3.15.patch b/package/ninja/0002-use-cmake-3.10-conform-COMPILE_FLAGS-instead-of-3.15.patch
new file mode 100644
index 0000000000..83ec77c76c
--- /dev/null
+++ b/package/ninja/0002-use-cmake-3.10-conform-COMPILE_FLAGS-instead-of-3.15.patch
@@ -0,0 +1,40 @@
+From 80c34cb11c093f3f0c4217ecf078a1a626da50bd Mon Sep 17 00:00:00 2001
+From: Peter Seiderer <ps.report@gmx.net>
+Date: Thu, 10 Dec 2020 20:29:19 +0100
+Subject: [PATCH] use cmake-3.10 conform COMPILE_FLAGS instead of 3.15
+ INCLUDE_DIRECTORIES property
+
+Use cmake-3.10 conform COMPILE_FLAGS instead of 3.15 INCLUDE_DIRECTORIES
+property (as this one is unknown/unused for older cmake versions and
+has no effect, resuling in a build failure because of missing extra
+include path).
+
+Fixes:
+
+  [ 96%] Building CXX object CMakeFiles/ninja.dir/src/browse.cc.o
+  .../build/host-ninja-1.10.2/src/browse.cc:23:10: fatal error: build/browse_py.h: No such file or directory
+     23 | #include "build/browse_py.h"
+        |          ^~~~~~~~~~~~~~~~~~~
+  compilation terminated.
+
+Signed-off-by: Peter Seiderer <ps.report@gmx.net>
+---
+ CMakeLists.txt | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/CMakeLists.txt b/CMakeLists.txt
+index 7f03c35..f582d2f 100644
+--- a/CMakeLists.txt
++++ b/CMakeLists.txt
+@@ -156,7 +156,7 @@ if(platform_supports_ninja_browse)
+ 	set_source_files_properties(src/browse.cc
+ 		PROPERTIES
+ 			OBJECT_DEPENDS "${CMAKE_BINARY_DIR}/build/browse_py.h"
+-			INCLUDE_DIRECTORIES "${CMAKE_BINARY_DIR}"
++			COMPILE_FLAGS "-I${CMAKE_BINARY_DIR}"
+ 			COMPILE_DEFINITIONS NINJA_PYTHON="python"
+ 	)
+ endif()
+-- 
+2.29.2
+
-- 
2.29.2

^ permalink raw reply related	[flat|nested] 6+ messages in thread