[28/49,v2] package/spandsp: new package

Message ID d89d72386ef004f0fdb39074ac4f8814411fe7c8.1504993178.git.yann.morin.1998@free.fr
State Changes Requested
Headers show
Series
  • [01/49,v2] package/asterisk: new package
Related show

Commit Message

Yann E. MORIN Sept. 9, 2017, 9:39 p.m.
Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
---
 package/Config.in                      |  1 +
 package/spandsp/0001-fix-msse4.x.patch | 29 +++++++++++++++++++++++++++++
 package/spandsp/Config.in              |  9 +++++++++
 package/spandsp/spandsp.hash           |  2 ++
 package/spandsp/spandsp.mk             | 28 ++++++++++++++++++++++++++++
 5 files changed, 69 insertions(+)
 create mode 100644 package/spandsp/0001-fix-msse4.x.patch
 create mode 100644 package/spandsp/Config.in
 create mode 100644 package/spandsp/spandsp.hash
 create mode 100644 package/spandsp/spandsp.mk

Comments

Arnout Vandecappelle Sept. 23, 2017, 4:40 p.m. | #1
On 09-09-17 23:39, Yann E. MORIN wrote:
[snip]
> diff --git a/package/spandsp/0001-fix-msse4.x.patch b/package/spandsp/0001-fix-msse4.x.patch
> new file mode 100644
> index 0000000000..ba3a3e55e3
> --- /dev/null
> +++ b/package/spandsp/0001-fix-msse4.x.patch
> @@ -0,0 +1,29 @@
> +diff -durN spandsp-20170713.orig/configure.ac spandsp-20170713/configure.ac
> +--- spandsp-20170713.orig/configure.ac	2014-08-09 10:19:27.000000000 +0200
> ++++ spandsp-20170713/configure.ac	2017-09-07 21:38:36.990854021 +0200
> +@@ -354,10 +354,10 @@

 Missing message + SoB. I'd add it myself but I'm not sure what to add :-)

> +         COMP_VENDOR_CFLAGS="-msse4a $COMP_VENDOR_CFLAGS"
> +     fi
> +     if test "$enable_sse4_2" = "yes" ; then
> +-        COMP_VENDOR_CFLAGS="-msse42 $COMP_VENDOR_CFLAGS"
> ++        COMP_VENDOR_CFLAGS="-msse4.2 $COMP_VENDOR_CFLAGS"

 Are we sure that e.g. GCC 4.3 didn't have the -msse42 flag? So I checked, and
the flag was added in 4.3, so exactly right :-)


[snip]
> +
> +SPANDSP_LICENSE = LGPLv2.1 (library), GPLv2 (test suite)

 Please use SPDX license codes.

> +SPANDSP_LICENSE_FILES = COPYING

 And add license hashes.

 I'm doing that for the other packages while I'm committing, but since you're
going to respin this one...

 Regards,
 Arnout

> +
> +SPANDSP_AUTORECONF = YES
> +SPANDSP_DEPENDENCIES = tiff
> +SPANDSP_INSTALL_STAGING = YES
> +
> +SPANDSP_CONF_OPTS = \
> +	--disable-doc \
> +	--disable-builtin-tiff \
> +	$(if $(BR2_X86_CPU_HAS_MMX),--enable-mmx,--disable-mmx) \
> +	$(if $(BR2_X86_CPU_HAS_SSE),--enable-sse,--disable-sse) \
> +	$(if $(BR2_X86_CPU_HAS_SSE2),--enable-sse2,--disable-sse2) \
> +	$(if $(BR2_X86_CPU_HAS_SSE3),--enable-sse3,--disable-sse3) \
> +	$(if $(BR2_X86_CPU_HAS_SSSE3),--enable-ssse3,--disable-ssse3) \
> +	$(if $(BR2_X86_CPU_HAS_SSE4),--enable-sse4-1,--disable-sse4-1) \
> +	$(if $(BR2_X86_CPU_HAS_SSE42),--enable-sse4-2,--disable-sse4-2)
> +
> +$(eval $(autotools-package))
>
Yann E. MORIN Sept. 24, 2017, 6:39 p.m. | #2
Arnout, All,

On 2017-09-23 18:40 +0200, Arnout Vandecappelle spake thusly:
> On 09-09-17 23:39, Yann E. MORIN wrote:
> [snip]
> > diff --git a/package/spandsp/0001-fix-msse4.x.patch b/package/spandsp/0001-fix-msse4.x.patch
> > new file mode 100644
> > index 0000000000..ba3a3e55e3
> > --- /dev/null
> > +++ b/package/spandsp/0001-fix-msse4.x.patch
> > @@ -0,0 +1,29 @@
> > +diff -durN spandsp-20170713.orig/configure.ac spandsp-20170713/configure.ac
> > +--- spandsp-20170713.orig/configure.ac	2014-08-09 10:19:27.000000000 +0200
> > ++++ spandsp-20170713/configure.ac	2017-09-07 21:38:36.990854021 +0200
> > +@@ -354,10 +354,10 @@
> 
>  Missing message + SoB. I'd add it myself but I'm not sure what to add :-)

Well, there's not much to say: the flags are incorrectly named in the
configure script, so we just fix them...

> > +         COMP_VENDOR_CFLAGS="-msse4a $COMP_VENDOR_CFLAGS"
> > +     fi
> > +     if test "$enable_sse4_2" = "yes" ; then
> > +-        COMP_VENDOR_CFLAGS="-msse42 $COMP_VENDOR_CFLAGS"
> > ++        COMP_VENDOR_CFLAGS="-msse4.2 $COMP_VENDOR_CFLAGS"
> 
>  Are we sure that e.g. GCC 4.3 didn't have the -msse42 flag? So I checked, and
> the flag was added in 4.3, so exactly right :-)

Thanks for answering your own question! ;-)

> > +
> > +SPANDSP_LICENSE = LGPLv2.1 (library), GPLv2 (test suite)
>  Please use SPDX license codes.

ACK.

> > +SPANDSP_LICENSE_FILES = COPYING
>  And add license hashes.

ACK.

>  I'm doing that for the other packages while I'm committing, but since you're
> going to respin this one...

Thanks! :-)

Regards,
Yann E. MORIN.

Patch

diff --git a/package/Config.in b/package/Config.in
index 764ae73c8b..2243d6bdcd 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -964,6 +964,7 @@  menu "Audio/Sound"
 	source "package/opusfile/Config.in"
 	source "package/portaudio/Config.in"
 	source "package/sbc/Config.in"
+	source "package/spandsp/Config.in"
 	source "package/speex/Config.in"
 	source "package/taglib/Config.in"
 	source "package/tinyalsa/Config.in"
diff --git a/package/spandsp/0001-fix-msse4.x.patch b/package/spandsp/0001-fix-msse4.x.patch
new file mode 100644
index 0000000000..ba3a3e55e3
--- /dev/null
+++ b/package/spandsp/0001-fix-msse4.x.patch
@@ -0,0 +1,29 @@ 
+diff -durN spandsp-20170713.orig/configure.ac spandsp-20170713/configure.ac
+--- spandsp-20170713.orig/configure.ac	2014-08-09 10:19:27.000000000 +0200
++++ spandsp-20170713/configure.ac	2017-09-07 21:38:36.990854021 +0200
+@@ -354,10 +354,10 @@
+         COMP_VENDOR_CFLAGS="-msse4a $COMP_VENDOR_CFLAGS"
+     fi
+     if test "$enable_sse4_2" = "yes" ; then
+-        COMP_VENDOR_CFLAGS="-msse42 $COMP_VENDOR_CFLAGS"
++        COMP_VENDOR_CFLAGS="-msse4.2 $COMP_VENDOR_CFLAGS"
+     fi
+     if test "$enable_sse4_1" = "yes" ; then
+-        COMP_VENDOR_CFLAGS="-msse41 $COMP_VENDOR_CFLAGS"
++        COMP_VENDOR_CFLAGS="-msse4.1 $COMP_VENDOR_CFLAGS"
+     fi
+     if test "$enable_ssse3" = "yes" ; then
+         COMP_VENDOR_CFLAGS="-mssse3 $COMP_VENDOR_CFLAGS"
+@@ -418,10 +418,10 @@
+         COMP_VENDOR_CFLAGS="-msse4a $COMP_VENDOR_CFLAGS"
+     fi
+     if test "$enable_sse4_2" = "yes" ; then
+-        COMP_VENDOR_CFLAGS="-msse42 $COMP_VENDOR_CFLAGS"
++        COMP_VENDOR_CFLAGS="-msse4.2 $COMP_VENDOR_CFLAGS"
+     fi
+     if test "$enable_sse4_1" = "yes" ; then
+-        COMP_VENDOR_CFLAGS="-msse41 $COMP_VENDOR_CFLAGS"
++        COMP_VENDOR_CFLAGS="-msse4.1 $COMP_VENDOR_CFLAGS"
+     fi
+     if test "$enable_ssse3" = "yes" ; then
+         COMP_VENDOR_CFLAGS="-mssse3 $COMP_VENDOR_CFLAGS"
diff --git a/package/spandsp/Config.in b/package/spandsp/Config.in
new file mode 100644
index 0000000000..a2d5b85d79
--- /dev/null
+++ b/package/spandsp/Config.in
@@ -0,0 +1,9 @@ 
+config BR2_PACKAGE_SPANDSP
+	bool "spandsp"
+	select BR2_PACKAGE_TIFF
+	help
+	  Spandsp is a library of many DSP functions for telephony.
+	  These range from simple modules, such as DTMF detection, to
+	  a complete software FAX machine.
+
+	  http://www.soft-switch.org/
diff --git a/package/spandsp/spandsp.hash b/package/spandsp/spandsp.hash
new file mode 100644
index 0000000000..6c7b772ac7
--- /dev/null
+++ b/package/spandsp/spandsp.hash
@@ -0,0 +1,2 @@ 
+# sha256 locally computed
+sha256  fcdc49aafbd4999514f2851f0dcfdc0bfa497eda600e66335a621cdead8b44cb  spandsp-20170713.tar.gz
diff --git a/package/spandsp/spandsp.mk b/package/spandsp/spandsp.mk
new file mode 100644
index 0000000000..5e7034e340
--- /dev/null
+++ b/package/spandsp/spandsp.mk
@@ -0,0 +1,28 @@ 
+################################################################################
+#
+# spandsp
+#
+################################################################################
+
+SPANDSP_VERSION = 20170713
+SPANDSP_SITE = https://www.soft-switch.org/downloads/spandsp/snapshots
+
+SPANDSP_LICENSE = LGPLv2.1 (library), GPLv2 (test suite)
+SPANDSP_LICENSE_FILES = COPYING
+
+SPANDSP_AUTORECONF = YES
+SPANDSP_DEPENDENCIES = tiff
+SPANDSP_INSTALL_STAGING = YES
+
+SPANDSP_CONF_OPTS = \
+	--disable-doc \
+	--disable-builtin-tiff \
+	$(if $(BR2_X86_CPU_HAS_MMX),--enable-mmx,--disable-mmx) \
+	$(if $(BR2_X86_CPU_HAS_SSE),--enable-sse,--disable-sse) \
+	$(if $(BR2_X86_CPU_HAS_SSE2),--enable-sse2,--disable-sse2) \
+	$(if $(BR2_X86_CPU_HAS_SSE3),--enable-sse3,--disable-sse3) \
+	$(if $(BR2_X86_CPU_HAS_SSSE3),--enable-ssse3,--disable-ssse3) \
+	$(if $(BR2_X86_CPU_HAS_SSE4),--enable-sse4-1,--disable-sse4-1) \
+	$(if $(BR2_X86_CPU_HAS_SSE42),--enable-sse4-2,--disable-sse4-2)
+
+$(eval $(autotools-package))