Skip to contentSkip to conversation

read-edid: bump version to 3.0.2

PATCH New Peter Seiderer 2 participants 3 messages Dec 7, 2015 – Dec 8, 2015

lore
Share & tools
lore

Peter Seiderer

author
#0 lore raw

PATCH 1/1 read-edid: bump version to 3.0.2

- update hash
- change from autotools to cmake package
- update license info
- enable get-edid command (i2c build)
- add patch fixing install error because of missing file COPYING

Signed-off-by Peter Seiderer <[email protected]>
---
 package/read-edid/0001-Fix-install-file-list.patch | 29 ++++++++++++++++++++++
 package/read-edid/Config.in                        |  2 --
 package/read-edid/read-edid.hash                   |  2 +-
 package/read-edid/read-edid.mk                     | 13 +++++-----
 4 files changed, 36 insertions(+), 10 deletions(-)
 create mode 100644 package/read-edid/0001-Fix-install-file-list.patch
package/read-edid/0001-Fix-install-file-list.patch new file +29 -0
new file mode 100644
index 0000000..622e97d
@@ -0,0 +1,29 @@
+From 562f48bee3443bda0ac257f2b0c52a72208163e0 Mon Sep 17 00:00:00 2001
+From: Peter Seiderer <[email protected]>
+Date: Mon, 7 Dec 2015 21:20:38 +0100
+Subject: [PATCH] Fix install file list.
+
+Fixes:
+  CMake Error at cmake_install.cmake:40 (file):
+    file INSTALL cannot find
+    ".../build/read-edid-3.0.2/COPYING".
+
+Signed-off-by: Peter Seiderer <[email protected]>
+---
+ CMakeLists.txt | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/CMakeLists.txt b/CMakeLists.txt
+index 7bf4561..2b9bc86 100644
+--- a/CMakeLists.txt
++++ b/CMakeLists.txt
+@@ -13,5 +13,5 @@ endif ()
+ add_subdirectory (parse-edid)
+ 
+ INSTALL(FILES get-edid.1 DESTINATION share/man/man1)
+-INSTALL(FILES AUTHORS ChangeLog COPYING README DESTINATION
++INSTALL(FILES AUTHORS ChangeLog README DESTINATION
+ 	share/doc/read-edid)
+-- 
+2.1.4
+
package/read-edid/Config.in +0 -2
index a4668e6..c830e17 100644
@@ -10,6 +10,4 @@ config BR2_PACKAGE_READ_EDID
 	  which turns the raw binary information into an
 	  XF86Config-compatible monitor section.
 
-	  Notice that the get-edid tool is only available on x86.
-
 	  http://polypux.org/projects/read-edid/
package/read-edid/read-edid.hash +1 -1
index 2c5501c..3005217 100644
@@ -1,2 +1,2 @@
 # Locally calculated
-sha256	c1b5d839fe6349efc183b3a3d33a8be1cd0dfea3c94015a86159d8aaeb6e9dd7	read-edid-1.4.2.tar.gz
+sha256 c7c6d8440f5b90f98e276829271ccea5b2ff5a3413df8a0f87ec09f834af186f read-edid-3.0.2.tar.gz
package/read-edid/read-edid.mk +6 -7
index f047e79..3e38f50 100644
@@ -4,13 +4,12 @@
 #
 ################################################################################
 
-READ_EDID_VERSION = 1.4.2
+READ_EDID_VERSION = 3.0.2
 READ_EDID_SITE = http://www.polypux.org/projects/read-edid
-READ_EDID_LICENSE = GPLv2
-READ_EDID_LICENSE_FILES = COPYING
+READ_EDID_LICENSE = BSD-like
+READ_EDID_LICENSE_FILES = LICENSE
 
-define READ_EDID_BUILD_CMDS
-	$(MAKE) $(TARGET_CONFIGURE_OPTS) -C $(@D)
-endef
+# disable classic get-edid support (needs libx86)
+READ_EDID_CONF_OPTS += -DCLASSICBUILD=OFF
 
-$(eval $(autotools-package))
+$(eval $(cmake-package))
Signature · 1 line
2.1.4
#1 lore raw

PATCH 1/1 read-edid: bump version to 3.0.2

Dear Peter Seiderer,

Quoted 5 lines · Peter Seiderer

- update hash
- change from autotools to cmake package
- update license info
- enable get-edid command (i2c build)
- add patch fixing install error because of missing file COPYING

Out of curiosity, have you reported that problem upstream? I think it always good to know the upstream status of the patches we add in Buildroot.

Quoted 2 lines
Signed-off-by Peter Seiderer <[email protected]>
Reviewed-by Vicente Olivert Riera <[email protected]>
Tested-by Vicente Olivert Riera <[email protected]>

Build test for MIPS architecture:

$ grep -E ^READ_EDID_VERSION package/read-edid/read-edid.mk
READ_EDID_VERSION = 3.0.2
$ file output/target/usr/bin/get-edid
output/target/usr/bin/get-edid: ELF 32-bit LSB executable, MIPS, MIPS32
rel2 version 1, dynamically linked, interpreter /lib/ld.so.1, for
GNU/Linux 2.6.32, not stripped

Regards,

Vincent.

author
#2 lore raw

PATCH 1/1 read-edid: bump version to 3.0.2

Hello Vincent,

Quoted 12 lines · Vicente Olivert Riera

Dear Peter Seiderer,

Quoted 5 lines · Peter Seiderer

- update hash
- change from autotools to cmake package
- update license info
- enable get-edid command (i2c build)
- add patch fixing install error because of missing file COPYING

Out of curiosity, have you reported that problem upstream? I think it always good to know the upstream status of the patches we add in Buildroot.

O.k, done now...

Quoted 7 lines
Quoted 2 lines
Signed-off-by Peter Seiderer <[email protected]>
Reviewed-by Vicente Olivert Riera <[email protected]>
Tested-by Vicente Olivert Riera <[email protected]>

Build test for MIPS architecture:

Quoted 8 lines
$ grep -E ^READ_EDID_VERSION package/read-edid/read-edid.mk
READ_EDID_VERSION = 3.0.2
$ file output/target/usr/bin/get-edid
output/target/usr/bin/get-edid: ELF 32-bit LSB executable, MIPS, MIPS32
rel2 version 1, dynamically linked, interpreter /lib/ld.so.1, for
GNU/Linux 2.6.32, not stripped

Thanks for testing...

Regards,
Peter

Quoted 3 lines

Regards,

Vincent.