diff mbox series

[v2] package/bluez-alsa: bump to version 2.0.0

Message ID 20200108141808.21387-1-joerg.krause@embedded.rocks
State Accepted
Headers show
Series [v2] package/bluez-alsa: bump to version 2.0.0 | expand

Commit Message

Jörg Krause Jan. 8, 2020, 2:18 p.m. UTC
Version 2.0.0 adds optional support for MP3 with lame and full MPEG
support with mpg123.

Signed-off-by: Jörg Krause <joerg.krause@embedded.rocks>
---
v2:
 - do not make enabling aplay tool a user choice, but built it by
   default
---
 package/bluez-alsa/bluez-alsa.hash |  2 +-
 package/bluez-alsa/bluez-alsa.mk   | 16 +++++++++++++++-
 2 files changed, 16 insertions(+), 2 deletions(-)

Comments

Yann E. MORIN Jan. 8, 2020, 4:55 p.m. UTC | #1
Jörg, All,

On 2020-01-08 15:18 +0100, Jörg Krause spake thusly:
> Version 2.0.0 adds optional support for MP3 with lame and full MPEG
> support with mpg123.
> 
> Signed-off-by: Jörg Krause <joerg.krause@embedded.rocks>

Applied to master, thanks.

Regards,
Yann E. MORIN.

> ---
> v2:
>  - do not make enabling aplay tool a user choice, but built it by
>    default
> ---
>  package/bluez-alsa/bluez-alsa.hash |  2 +-
>  package/bluez-alsa/bluez-alsa.mk   | 16 +++++++++++++++-
>  2 files changed, 16 insertions(+), 2 deletions(-)
> 
> diff --git a/package/bluez-alsa/bluez-alsa.hash b/package/bluez-alsa/bluez-alsa.hash
> index afc84329b3..37c4919f91 100644
> --- a/package/bluez-alsa/bluez-alsa.hash
> +++ b/package/bluez-alsa/bluez-alsa.hash
> @@ -1,3 +1,3 @@
>  # Locally calculated:
> -sha256  6287ae6135795a78470068ef9d9ea660d556cefc409905bd9211fda04d1e1f8e  bluez-alsa-1.4.0.tar.gz
> +sha256  451c99c2045a1ec1ffe0d054be4774fbd01af34d6d1a2757d346d02718381697  bluez-alsa-2.0.0.tar.gz
>  sha256  45fe8648faa0e735d4aaf60c32ddd37fc7f027d34ccedcf0f69d8aa961b63b25  LICENSE
> diff --git a/package/bluez-alsa/bluez-alsa.mk b/package/bluez-alsa/bluez-alsa.mk
> index 4552be82f3..5c07a1a5e8 100644
> --- a/package/bluez-alsa/bluez-alsa.mk
> +++ b/package/bluez-alsa/bluez-alsa.mk
> @@ -4,7 +4,7 @@
>  #
>  ################################################################################
>  
> -BLUEZ_ALSA_VERSION = 1.4.0
> +BLUEZ_ALSA_VERSION = 2.0.0
>  BLUEZ_ALSA_SITE = $(call github,Arkq,bluez-alsa,v$(BLUEZ_ALSA_VERSION))
>  BLUEZ_ALSA_LICENSE = MIT
>  BLUEZ_ALSA_LICENSE_FILES = LICENSE
> @@ -32,6 +32,20 @@ else
>  BLUEZ_ALSA_CONF_OPTS += --disable-aac
>  endif
>  
> +ifeq ($(BR2_PACKAGE_LAME),y)
> +BLUEZ_ALSA_DEPENDENCIES += lame
> +BLUEZ_ALSA_CONF_OPTS += --enable-mp3lame
> +else
> +BLUEZ_ALSA_CONF_OPTS += --disable-mp3lame
> +endif
> +
> +ifeq ($(BR2_PACKAGE_MPG123),y)
> +BLUEZ_ALSA_DEPENDENCIES += mpg123
> +BLUEZ_ALSA_CONF_OPTS += --enable-mpg123
> +else
> +BLUEZ_ALSA_CONF_OPTS += --disable-mpg123
> +endif
> +
>  # no build dependency, disables internal HFP in favor of oFonos HFP profile
>  ifeq ($(BR2_PACKAGE_OFONO),y)
>  BLUEZ_ALSA_CONF_OPTS += --enable-ofono
> -- 
> 2.24.1
> 
> _______________________________________________
> buildroot mailing list
> buildroot@busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
Yann E. MORIN Jan. 8, 2020, 5:01 p.m. UTC | #2
Jörg, All,

On 2020-01-08 17:55 +0100, Yann E. MORIN spake thusly:
> On 2020-01-08 15:18 +0100, Jörg Krause spake thusly:
> > Version 2.0.0 adds optional support for MP3 with lame and full MPEG
> > support with mpg123.
> > 
> > Signed-off-by: Jörg Krause <joerg.krause@embedded.rocks>
> 
> Applied to master, thanks.

Oh, I forgot to say that I slightly reworded the commit log, to make it
obvious that both mp3 and full mpeg are optional (the original commit
log suggested that only mp3 was optional, when in fact both are).

Regards,
Yann E. MORIN.

> Regards,
> Yann E. MORIN.
> 
> > ---
> > v2:
> >  - do not make enabling aplay tool a user choice, but built it by
> >    default
> > ---
> >  package/bluez-alsa/bluez-alsa.hash |  2 +-
> >  package/bluez-alsa/bluez-alsa.mk   | 16 +++++++++++++++-
> >  2 files changed, 16 insertions(+), 2 deletions(-)
> > 
> > diff --git a/package/bluez-alsa/bluez-alsa.hash b/package/bluez-alsa/bluez-alsa.hash
> > index afc84329b3..37c4919f91 100644
> > --- a/package/bluez-alsa/bluez-alsa.hash
> > +++ b/package/bluez-alsa/bluez-alsa.hash
> > @@ -1,3 +1,3 @@
> >  # Locally calculated:
> > -sha256  6287ae6135795a78470068ef9d9ea660d556cefc409905bd9211fda04d1e1f8e  bluez-alsa-1.4.0.tar.gz
> > +sha256  451c99c2045a1ec1ffe0d054be4774fbd01af34d6d1a2757d346d02718381697  bluez-alsa-2.0.0.tar.gz
> >  sha256  45fe8648faa0e735d4aaf60c32ddd37fc7f027d34ccedcf0f69d8aa961b63b25  LICENSE
> > diff --git a/package/bluez-alsa/bluez-alsa.mk b/package/bluez-alsa/bluez-alsa.mk
> > index 4552be82f3..5c07a1a5e8 100644
> > --- a/package/bluez-alsa/bluez-alsa.mk
> > +++ b/package/bluez-alsa/bluez-alsa.mk
> > @@ -4,7 +4,7 @@
> >  #
> >  ################################################################################
> >  
> > -BLUEZ_ALSA_VERSION = 1.4.0
> > +BLUEZ_ALSA_VERSION = 2.0.0
> >  BLUEZ_ALSA_SITE = $(call github,Arkq,bluez-alsa,v$(BLUEZ_ALSA_VERSION))
> >  BLUEZ_ALSA_LICENSE = MIT
> >  BLUEZ_ALSA_LICENSE_FILES = LICENSE
> > @@ -32,6 +32,20 @@ else
> >  BLUEZ_ALSA_CONF_OPTS += --disable-aac
> >  endif
> >  
> > +ifeq ($(BR2_PACKAGE_LAME),y)
> > +BLUEZ_ALSA_DEPENDENCIES += lame
> > +BLUEZ_ALSA_CONF_OPTS += --enable-mp3lame
> > +else
> > +BLUEZ_ALSA_CONF_OPTS += --disable-mp3lame
> > +endif
> > +
> > +ifeq ($(BR2_PACKAGE_MPG123),y)
> > +BLUEZ_ALSA_DEPENDENCIES += mpg123
> > +BLUEZ_ALSA_CONF_OPTS += --enable-mpg123
> > +else
> > +BLUEZ_ALSA_CONF_OPTS += --disable-mpg123
> > +endif
> > +
> >  # no build dependency, disables internal HFP in favor of oFonos HFP profile
> >  ifeq ($(BR2_PACKAGE_OFONO),y)
> >  BLUEZ_ALSA_CONF_OPTS += --enable-ofono
> > -- 
> > 2.24.1
> > 
> > _______________________________________________
> > buildroot mailing list
> > buildroot@busybox.net
> > http://lists.busybox.net/mailman/listinfo/buildroot
> 
> -- 
> .-----------------.--------------------.------------------.--------------------.
> |  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
> | +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
> | +33 561 099 427 `------------.-------:  X  AGAINST      |  \e/  There is no  |
> | http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
> '------------------------------^-------^------------------^--------------------'
> _______________________________________________
> buildroot mailing list
> buildroot@busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
diff mbox series

Patch

diff --git a/package/bluez-alsa/bluez-alsa.hash b/package/bluez-alsa/bluez-alsa.hash
index afc84329b3..37c4919f91 100644
--- a/package/bluez-alsa/bluez-alsa.hash
+++ b/package/bluez-alsa/bluez-alsa.hash
@@ -1,3 +1,3 @@ 
 # Locally calculated:
-sha256  6287ae6135795a78470068ef9d9ea660d556cefc409905bd9211fda04d1e1f8e  bluez-alsa-1.4.0.tar.gz
+sha256  451c99c2045a1ec1ffe0d054be4774fbd01af34d6d1a2757d346d02718381697  bluez-alsa-2.0.0.tar.gz
 sha256  45fe8648faa0e735d4aaf60c32ddd37fc7f027d34ccedcf0f69d8aa961b63b25  LICENSE
diff --git a/package/bluez-alsa/bluez-alsa.mk b/package/bluez-alsa/bluez-alsa.mk
index 4552be82f3..5c07a1a5e8 100644
--- a/package/bluez-alsa/bluez-alsa.mk
+++ b/package/bluez-alsa/bluez-alsa.mk
@@ -4,7 +4,7 @@ 
 #
 ################################################################################
 
-BLUEZ_ALSA_VERSION = 1.4.0
+BLUEZ_ALSA_VERSION = 2.0.0
 BLUEZ_ALSA_SITE = $(call github,Arkq,bluez-alsa,v$(BLUEZ_ALSA_VERSION))
 BLUEZ_ALSA_LICENSE = MIT
 BLUEZ_ALSA_LICENSE_FILES = LICENSE
@@ -32,6 +32,20 @@  else
 BLUEZ_ALSA_CONF_OPTS += --disable-aac
 endif
 
+ifeq ($(BR2_PACKAGE_LAME),y)
+BLUEZ_ALSA_DEPENDENCIES += lame
+BLUEZ_ALSA_CONF_OPTS += --enable-mp3lame
+else
+BLUEZ_ALSA_CONF_OPTS += --disable-mp3lame
+endif
+
+ifeq ($(BR2_PACKAGE_MPG123),y)
+BLUEZ_ALSA_DEPENDENCIES += mpg123
+BLUEZ_ALSA_CONF_OPTS += --enable-mpg123
+else
+BLUEZ_ALSA_CONF_OPTS += --disable-mpg123
+endif
+
 # no build dependency, disables internal HFP in favor of oFonos HFP profile
 ifeq ($(BR2_PACKAGE_OFONO),y)
 BLUEZ_ALSA_CONF_OPTS += --enable-ofono