Skip to contentSkip to conversation

package/read-edid: Fix compiler check, c++ is not needed

PATCH New Bernd Kuhls 3 participants 3 messages Dec 13, 2015

lore
Share & tools
lore

Bernd Kuhls

author
#0 lore raw

PATCH v2 1/1 package/read-edid: Fix compiler check, c++ is not needed

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

Signed-off-by Bernd Kuhls <[email protected]>

---
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
package/read-edid/0002-Fix-compiler-check.patch new file +19 -0
new file mode 100644
index 0000000..6c5f58c
@@ -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 <[email protected]>
+
+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)
Signature · 1 line
2.6.2
#1 lore raw

PATCH v2 1/1 package/read-edid: Fix compiler check, c++ is not needed

Bernd, All,

On 2015-12-13 16:16 +0100, Bernd Kuhls spake thusly:

Quoted 5 lines

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

Signed-off-by Bernd Kuhls <[email protected]>
Reviewed-by "Yann E. MORIN" <[email protected]>

Regards,
Yann E. MORIN.

Quoted 39 lines

---
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
package/read-edid/0002-Fix-compiler-check.patch new file +19 -0
new file mode 100644
index 0000000..6c5f58c
@@ -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 <[email protected]>
+
+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)
Signature · 6 lines
2.6.2

_______________________________________________
buildroot mailing list
buildroot at busybox.net
http://lists.busybox.net/mailman/listinfo/buildroot
Signature · 6 lines
.-----------------.--------------------.------------------.--------------------.
|  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.  |
'------------------------------^-------^------------------^--------------------'
Applied
#2 lore raw

PATCH v2 1/1 package/read-edid: Fix compiler check, c++ is not needed

In reply to Bernd Kuhls

Dear Bernd Kuhls,

Quoted 7 lines · Bernd Kuhls

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

Signed-off-by Bernd Kuhls <[email protected]>

---
v2: read-edid does not need c++, added new patch suggested by Thomas

Applied, thanks. Please submit the patch upstream :)

Thomas

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