Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH v2 1/1] package/read-edid: Fix compiler check, c++ is not needed
@ 2015-12-13 15:16 Bernd Kuhls
  2015-12-13 17:20 ` Yann E. MORIN
  2015-12-13 21:38 ` Thomas Petazzoni
  0 siblings, 2 replies; 3+ messages in thread
From: Bernd Kuhls @ 2015-12-13 15:16 UTC (permalink / raw)
  To: buildroot

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

Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
---
v2: read-edid does not need c++, added new patch suggested by Thomas

 package/read-edid/0002-Fix-compiler-check.patch | 19 +++++++++++++++++++
 1 file changed, 19 insertions(+)
 create mode 100644 package/read-edid/0002-Fix-compiler-check.patch

diff --git a/package/read-edid/0002-Fix-compiler-check.patch b/package/read-edid/0002-Fix-compiler-check.patch
new file mode 100644
index 0000000..6c5f58c
--- /dev/null
+++ b/package/read-edid/0002-Fix-compiler-check.patch
@@ -0,0 +1,19 @@
+Fix compiler check
+
+By default CMake looks for c++ which the project does not need.
+
+Patch by Thomas:
+http://lists.busybox.net/pipermail/buildroot/2015-December/146865.html
+
+Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
+
+diff -uNr read-edid-3.0.2.org/CMakeLists.txt read-edid-3.0.2/CMakeLists.txt
+--- read-edid-3.0.2.org/CMakeLists.txt	2014-02-05 17:27:26.000000000 +0100
++++ read-edid-3.0.2/CMakeLists.txt	2015-12-13 16:08:43.000000000 +0100
+@@ -1,5 +1,5 @@
+ cmake_minimum_required (VERSION 2.6)
+-project (read-edid)
++project (read-edid C)
+ 
+ option(I2CBUILD "Build I2C get-edid implementation" ON)
+ option(CLASSICBUILD "Build VBE get-edid implementation" ON)
-- 
2.6.2

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

* [Buildroot] [PATCH v2 1/1] package/read-edid: Fix compiler check, c++ is not needed
  2015-12-13 15:16 [Buildroot] [PATCH v2 1/1] package/read-edid: Fix compiler check, c++ is not needed Bernd Kuhls
@ 2015-12-13 17:20 ` Yann E. MORIN
  2015-12-13 21:38 ` Thomas Petazzoni
  1 sibling, 0 replies; 3+ messages in thread
From: Yann E. MORIN @ 2015-12-13 17:20 UTC (permalink / raw)
  To: buildroot

Bernd, All,

On 2015-12-13 16:16 +0100, Bernd Kuhls spake thusly:
> Fixes
> http://autobuild.buildroot.net/results/8c1/8c11e463b68121fe0331acf9f8a42a1d3fef40c7//
> and others
> 
> Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>

Reviewed-by: "Yann E. MORIN" <yann.morin.1998@free.fr>

Regards,
Yann E. MORIN.

> ---
> v2: read-edid does not need c++, added new patch suggested by Thomas
> 
>  package/read-edid/0002-Fix-compiler-check.patch | 19 +++++++++++++++++++
>  1 file changed, 19 insertions(+)
>  create mode 100644 package/read-edid/0002-Fix-compiler-check.patch
> 
> diff --git a/package/read-edid/0002-Fix-compiler-check.patch b/package/read-edid/0002-Fix-compiler-check.patch
> new file mode 100644
> index 0000000..6c5f58c
> --- /dev/null
> +++ b/package/read-edid/0002-Fix-compiler-check.patch
> @@ -0,0 +1,19 @@
> +Fix compiler check
> +
> +By default CMake looks for c++ which the project does not need.
> +
> +Patch by Thomas:
> +http://lists.busybox.net/pipermail/buildroot/2015-December/146865.html
> +
> +Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
> +
> +diff -uNr read-edid-3.0.2.org/CMakeLists.txt read-edid-3.0.2/CMakeLists.txt
> +--- read-edid-3.0.2.org/CMakeLists.txt	2014-02-05 17:27:26.000000000 +0100
> ++++ read-edid-3.0.2/CMakeLists.txt	2015-12-13 16:08:43.000000000 +0100
> +@@ -1,5 +1,5 @@
> + cmake_minimum_required (VERSION 2.6)
> +-project (read-edid)
> ++project (read-edid C)
> + 
> + option(I2CBUILD "Build I2C get-edid implementation" ON)
> + option(CLASSICBUILD "Build VBE get-edid implementation" ON)
> -- 
> 2.6.2
> 
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'

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

* [Buildroot] [PATCH v2 1/1] package/read-edid: Fix compiler check, c++ is not needed
  2015-12-13 15:16 [Buildroot] [PATCH v2 1/1] package/read-edid: Fix compiler check, c++ is not needed Bernd Kuhls
  2015-12-13 17:20 ` Yann E. MORIN
@ 2015-12-13 21:38 ` Thomas Petazzoni
  1 sibling, 0 replies; 3+ messages in thread
From: Thomas Petazzoni @ 2015-12-13 21:38 UTC (permalink / raw)
  To: buildroot

Dear Bernd Kuhls,

On Sun, 13 Dec 2015 16:16:02 +0100, Bernd Kuhls wrote:
> Fixes
> http://autobuild.buildroot.net/results/8c1/8c11e463b68121fe0331acf9f8a42a1d3fef40c7//
> and others
> 
> Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
> ---
> v2: read-edid does not need c++, added new patch suggested by Thomas

Applied, thanks. Please submit the patch upstream :)

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

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

end of thread, other threads:[~2015-12-13 21:38 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-12-13 15:16 [Buildroot] [PATCH v2 1/1] package/read-edid: Fix compiler check, c++ is not needed Bernd Kuhls
2015-12-13 17:20 ` Yann E. MORIN
2015-12-13 21:38 ` Thomas Petazzoni

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox