Skip to contentSkip to conversation

package/libdvbsi: new package

PATCH New Carsten Schoenert 2 participants 3 messages Jun 23, 2013

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 package/libdvbsi: new package

From: Carsten Schoenert <[email protected]>

libdvbsi++ is a C++ library for parsing DVB SI sections and providing the output in C++ objects. It's usefull in any case you need prepared and preselected DVB SI data from the raw stream.

---
 package/Config.in            |    1 +
 package/libdvbsi/Config.in   |    9 +++++++++
 package/libdvbsi/libdvbsi.mk |   17 +++++++++++++++++
 3 files changed, 27 insertions(+)
 create mode 100644 package/libdvbsi/Config.in
 create mode 100644 package/libdvbsi/libdvbsi.mk
package/Config.in +1 -0
index 0eb6a9c..dfc4584 100644
@@ -513,6 +513,7 @@ source "package/json-javascript/Config.in"
 endmenu
 
 menu "Multimedia"
+source "package/libdvbsi/Config.in"
 source "package/libdvdread/Config.in"
 source "package/libdvdnav/Config.in"
 source "package/libebml/Config.in"
package/libdvbsi/Config.in new file +9 -0
new file mode 100644
index 0000000..2f58d2a
@@ -0,0 +1,9 @@
+config BR2_PACKAGE_LIBDVBSI
+	bool "libdvbsi"
+	depends on BR2_INSTALL_LIBSTDCPP && BR2_USE_WCHAR && BR2_TOOLCHAIN_HAS_THREADS
+	help
+	  C++ parsing library for Service Information (SI) in DVB systems.
+	  With libdvbsi++ you can convert raw DVB service information to
+	  a set of C++ objects, removing the need to parse data manually.
+
+	  http://www.saftware.de/
package/libdvbsi/libdvbsi.mk new file +17 -0
new file mode 100644
index 0000000..1fc0701
@@ -0,0 +1,17 @@
+################################################################################
+#
+# libdvbsi++
+#
+################################################################################
+
+LIBDVBSI_VERSION = 0.3.6
+LIBDVBSI_SOURCE = libdvbsi++-$(LIBDVBSI_VERSION).tar.bz2
+LIBDVBSI_SITE = http://www.saftware.de/
+LIBDVBSI_INSTALL_STAGING = YES
+LIBDVBSI_LICENSE = LGPLv2.1
+LIBDVBSI_LICENSE_FILE = COPYING
+
+# sometimes no Makefile is in the archive
+LIBDVBSI_AUTORECONF=YES
+
+$(eval $(autotools-package))
Signature · 1 line
1.7.10.4
#1 lore raw

PATCH 1/1 package/libdvbsi: new package

Dear Carsten Schoenert,

package/libdvbsi/Config.in line 3

Quoted 6 lines · Carsten Schoenert
package/libdvbsi/Config.in new file +3 -0
@@ -0,0 +1,9 @@
+config BR2_PACKAGE_LIBDVBSI
+	bool "libdvbsi"
+	depends on BR2_INSTALL_LIBSTDCPP && BR2_USE_WCHAR && BR2_TOOLCHAIN_HAS_THREADS

We generally put these dependencies each on a separate line, for some reason.

package/libdvbsi/Config.in line 9

Quoted 6 lines
package/libdvbsi/Config.in new file +6 -0
+	help
+	  C++ parsing library for Service Information (SI) in DVB systems.
+	  With libdvbsi++ you can convert raw DVB service information to
+	  a set of C++ objects, removing the need to parse data manually.
+
+	  http://www.saftware.de/

Missing comment when the package cannot be selected due to the absence of the toolchain dependencies.

package/libdvbsi/Config.in

Quoted 11 lines
package/libdvbsi/Config.in new file +11 -0
+LIBDVBSI_VERSION = 0.3.6
+LIBDVBSI_SOURCE = libdvbsi++-$(LIBDVBSI_VERSION).tar.bz2
+LIBDVBSI_SITE = http://www.saftware.de/
+LIBDVBSI_INSTALL_STAGING = YES
+LIBDVBSI_LICENSE = LGPLv2.1
+LIBDVBSI_LICENSE_FILE = COPYING
+
+# sometimes no Makefile is in the archive
+LIBDVBSI_AUTORECONF=YES
+
+$(eval $(autotools-package))

Otherwise, looks good.

Thanks,

Thomas

Signature · 4 lines
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
#2 lore raw

PATCH 1/1 package/libdvbsi: new package

Hello Thomas,

Am 23.06.2013 16:42, schrieb Thomas Petazzoni:

Quoted 2 lines

We generally put these dependencies each on a separate line, for some reason.

O.k. put this from another Config.in. :-)
But I agree.

Quoted 2 lines

Missing comment when the package cannot be selected due to the absence of the toolchain dependencies.

Dito^^

Quoted 1 line

Otherwise, looks good.

Yann found another issue, I prepare a new patch.

Thx for the quick re looking!

Regards
Carsten