[Buildroot] [PATCH 1/1] package/read-edid: Needs c++

[Buildroot] [PATCH 1/1] package/read-edid: Needs c++

[Bernd Kuhls]

Fixes
http://autobuild.buildroot.net/results/8c1/8c11e463b68121fe0331acf9f8a42a1d3fef40c7//
and others

Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
---
 package/read-edid/Config.in | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/package/read-edid/Config.in b/package/read-edid/Config.in
index c830e17..6f39037 100644
--- a/package/read-edid/Config.in
+++ b/package/read-edid/Config.in
@@ -1,5 +1,6 @@
 config BR2_PACKAGE_READ_EDID
 	bool "read-edid"
+	depends on BR2_INSTALL_LIBSTDCPP
 	help
 	  Read-edid is a pair of tools for reading the EDID from a
 	  monitor. It should work with most monitors made since 1996
@@ -11,3 +12,6 @@ config BR2_PACKAGE_READ_EDID
 	  XF86Config-compatible monitor section.
 
 	  http://polypux.org/projects/read-edid/
+
+comment "read-edid needs a toolchain w/ C++"
+	depends on !BR2_INSTALL_LIBSTDCPP
-- 
2.6.2

[Buildroot] [PATCH 1/1] package/read-edid: Needs c++

[Thomas Petazzoni]

Dear Bernd Kuhls,

On Sun, 13 Dec 2015 10:35:43 +0100, Bernd Kuhls wrote:
> Fixes
> http://autobuild.buildroot.net/results/8c1/8c11e463b68121fe0331acf9f8a42a1d3fef40c7//
> and others
> 
> Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>

This is not the proper fix: there is no C++ code in this package. The
proper fix is to adjust CMakeLists.txt to remove the C++ check. This is
done by doing:

	project (read-edid C)

See https://cmake.org/cmake/help/v3.0/command/project.html for details,
and especially: "Optionally you can specify which languages your
project supports. Example languages are C, CXX (i.e. C++), Fortran,
etc. By default C and CXX are enabled if no language options are given."

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com