diff mbox series

[1/1] mbuffer: new package

Message ID 20200211134652.2909-1-gliga.mircea@gmail.com
State Accepted
Headers show
Series [1/1] mbuffer: new package | expand

Commit Message

Mircea Gliga Feb. 11, 2020, 1:46 p.m. UTC
Signed-off-by: Mircea Gliga <gliga.mircea@gmail.com>
---
 package/Config.in            |  1 +
 package/mbuffer/Config.in    |  8 ++++++++
 package/mbuffer/mbuffer.hash |  5 +++++
 package/mbuffer/mbuffer.mk   | 19 +++++++++++++++++++
 4 files changed, 33 insertions(+)
 create mode 100644 package/mbuffer/Config.in
 create mode 100644 package/mbuffer/mbuffer.hash
 create mode 100644 package/mbuffer/mbuffer.mk

Comments

Thomas Petazzoni April 12, 2020, 8:42 p.m. UTC | #1
Hello Mircea,

On Tue, 11 Feb 2020 15:46:52 +0200
Mircea Gliga <gliga.mircea@gmail.com> wrote:

> Signed-off-by: Mircea Gliga <gliga.mircea@gmail.com>

Sorry for the long delay. I have now applied your patch, with a few
changes.

First, I changed the commit title to "package/mbuffer: new package", to
follow our convention.

>  package/Config.in            |  1 +
>  package/mbuffer/Config.in    |  8 ++++++++
>  package/mbuffer/mbuffer.hash |  5 +++++
>  package/mbuffer/mbuffer.mk   | 19 +++++++++++++++++++

I've added an entry for this package in the DEVELOPERS file.

> diff --git a/package/mbuffer/Config.in b/package/mbuffer/Config.in
> new file mode 100644
> index 0000000000..5c733c0ec5
> --- /dev/null
> +++ b/package/mbuffer/Config.in
> @@ -0,0 +1,8 @@
> +config BR2_PACKAGE_MBUFFER
> +	bool "mbuffer"
> +	depends on !BR2_STATIC_LIBS

I've added a comment that explains why we have this dependency:

	depends on !BR2_STATIC_LIBS # dlopen()

> +	help
> +	  mbuffer is a tool for buffering data streams with a large set
> +	  of unique features.
> +
> +	  http://www.maier-komor.de/mbuffer.html

And here I've added a Config.in comment that shows up in
BR2_STATIC_LIBS configurations, so that users now why mbuffer is not
available.

Applied with those changes. Thanks!

Thomas
diff mbox series

Patch

diff --git a/package/Config.in b/package/Config.in
index dfa3f34b9d..ed173de1d6 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -1703,6 +1703,7 @@  menu "Networking"
 	source "package/libwebsockets/Config.in"
 	source "package/libyang/Config.in"
 	source "package/lksctp-tools/Config.in"
+	source "package/mbuffer/Config.in"
 	source "package/mongoose/Config.in"
 	source "package/nanomsg/Config.in"
 	source "package/neon/Config.in"
diff --git a/package/mbuffer/Config.in b/package/mbuffer/Config.in
new file mode 100644
index 0000000000..5c733c0ec5
--- /dev/null
+++ b/package/mbuffer/Config.in
@@ -0,0 +1,8 @@ 
+config BR2_PACKAGE_MBUFFER
+	bool "mbuffer"
+	depends on !BR2_STATIC_LIBS
+	help
+	  mbuffer is a tool for buffering data streams with a large set
+	  of unique features.
+
+	  http://www.maier-komor.de/mbuffer.html
diff --git a/package/mbuffer/mbuffer.hash b/package/mbuffer/mbuffer.hash
new file mode 100644
index 0000000000..80b7b1f26c
--- /dev/null
+++ b/package/mbuffer/mbuffer.hash
@@ -0,0 +1,5 @@ 
+# From http://www.maier-komor.de/mbuffer.html
+md5 e4acaa1e6a9a879e7394f04e02e1ae83  mbuffer-20191016.tgz
+
+# Locally computed after checking signature upstream
+sha256 8dc210454765c18901074bc16e126c655135a486e73d69855caf74a157ddbe17  mbuffer-20191016.tgz
diff --git a/package/mbuffer/mbuffer.mk b/package/mbuffer/mbuffer.mk
new file mode 100644
index 0000000000..6ff6e27a39
--- /dev/null
+++ b/package/mbuffer/mbuffer.mk
@@ -0,0 +1,19 @@ 
+################################################################################
+#
+# mbuffer
+#
+################################################################################
+
+MBUFFER_VERSION = 20191016
+MBUFFER_SOURCE = mbuffer-$(MBUFFER_VERSION).tgz
+MBUFFER_SITE = http://www.maier-komor.de/software/mbuffer
+MBUFFER_LICENSE = GPL-3.0+
+MBUFFER_LICENSE_FILES = LICENSE
+MBUFFER_CONF_OPTS = --disable-debug
+MBUFFER_CONF_ENV = CFLAGS="$(TARGET_CFLAGS) -std=c99"
+
+# we don't need tests & co. so we specify a target
+# so that the others don't get built, e.g idev.so
+MBUFFER_MAKE_OPTS += mbuffer
+
+$(eval $(autotools-package))