Message ID | 20210615005923.51837-1-charles.hardin@chargepoint.com |
---|---|
State | Accepted |
Headers | show |
Series | [1/1] package/alsa-plugins: add new package alsa-plugins version 1.2.5 | expand |
Hello Charles, On Mon, 14 Jun 2021 17:59:23 -0700 Charles Hardin <ckhardin@gmail.com> wrote: > From: Charles Hardin <ckhardin@gmail.com> > > During testing of bluez-alsa in particular, there is no compilation > dependency - but, some configurations against boards will request > the load of a particular plugin: > > ALSA lib dlmisc.c:337:(snd_dlobj_cache_get0) \ > Cannot open shared library libasound_module_rate_samplerate_best.so \ > (/usr/lib/alsa-lib/libasound_module_rate_samplerate_best.so: \ > cannot open shared object file: No such file or directory) > > Adding in the alsa-plugins package will build the appropriate dlobject > for loading and configuration. > > Signed-off-by: Charles Hardin <ckhardin@gmail.com> Thanks for your patch. I have applied, but after doing a number of changes, see below for the details. First of all, alsa-plugins 1.2.5 did not build against alsa-lib 1.2.4, as it uses a new API that only appeared in alsa-lib 1.2.5, so I'm not sure how you could have built alsa-plugins 1.2.5. So as a preparation patch I have bumped alsa-lib to 1.2.5.1. > package/Config.in | 1 + > package/alsa-plugins/Config.in | 8 ++++++++ > package/alsa-plugins/alsa-plugins.hash | 4 ++++ > package/alsa-plugins/alsa-plugins.mk | 23 +++++++++++++++++++++++ > 4 files changed, 36 insertions(+) You forgot to add an entry in the DEVELOPERS file. > diff --git a/package/alsa-plugins/Config.in b/package/alsa-plugins/Config.in > new file mode 100644 > index 0000000000..44aaa7997c > --- /dev/null > +++ b/package/alsa-plugins/Config.in > @@ -0,0 +1,8 @@ > +config BR2_PACKAGE_ALSA_PLUGINS > + bool "alsa-plugins" > + depends on BR2_PACKAGE_ALSA_LIB select BR2_PACKAGE_ALSA_LIB_HWDEP was missing. > + select BR2_PACKAGE_LIBSAMPLERATE This is not needed: libsamplerate is not a mandatory dependency. > diff --git a/package/alsa-plugins/alsa-plugins.mk b/package/alsa-plugins/alsa-plugins.mk > new file mode 100644 > index 0000000000..70d3cdb87c > --- /dev/null > +++ b/package/alsa-plugins/alsa-plugins.mk > @@ -0,0 +1,23 @@ > +############################################################# > +# > +# alsa-plugins > +# > +############################################################# The formatting of this comment was not matching our coding style: make sure to use "make check-package" when you create a new package. > +ALSA_PLUGINS_VERSION = 1.2.5 > +ALSA_PLUGINS_SOURCE = alsa-plugins-$(ALSA_PLUGINS_VERSION).tar.bz2 > +ALSA_PLUGINS_SITE = https://www.alsa-project.org/files/pub/plugins > +ALSA_PLUGINS_LICENSE = LGPL-2.1+ (library), GPL-2.0+ (samplerate) > +ALSA_PLUGINS_LICENSE_FILES = COPYING COPYING.GPL > +ALSA_PLUGINS_INSTALL_STAGING = NO This was not needed as it's the default. > +ALSA_PLUGINS_AUTORECONF = YES Not needed, there is an already generate configure script in the tarball. > +ALSA_PLUGINS_DEPENDENCIES = alsa-lib libsamplerate > + > +ALSA_PLUGINS_CONF_OPTS += \ > + --enable-samplerate \ > + --disable-jack \ > + --disable-usbstream \ > + --disable-pulseaudio \ > + --disable-avcodec \ > + --with-speex=builtin Then the major change was to make the libsamplerate dependency optional, as such: +ifeq ($(BR2_PACKAGE_LIBSAMPLERATE),y) +ALSA_PLUGINS_CONF_OPTS += --enable-samplerate +ALSA_PLUGINS_DEPENDENCIES += libsamplerate +ALSA_PLUGINS_LICENSE += , GPL-2.0+ (samplerate plugin) +ALSA_PLUGINS_LICENSE_FILES += COPYING.GPL +else +ALSA_PLUGINS_CONF_OPTS += --disable-samplerate +endif And also I adjusted the list of options to disable a few other things, or differently. See the final commit at https://git.buildroot.org/buildroot/commit/?id=e85bde9cc672fc228eeb079e23ef302468ae3826 If there are build failures on alsa-plugins, you will receive an e-mail, and it would be nice if you could investigate those build failures and submit patches fixing them. Thanks a lot for your contribution! Thomas
diff --git a/package/Config.in b/package/Config.in index e42f579494..26adcd53f2 100644 --- a/package/Config.in +++ b/package/Config.in @@ -1291,6 +1291,7 @@ menu "Libraries" menu "Audio/Sound" source "package/alsa-lib/Config.in" + source "package/alsa-plugins/Config.in" source "package/alure/Config.in" source "package/aubio/Config.in" source "package/bcg729/Config.in" diff --git a/package/alsa-plugins/Config.in b/package/alsa-plugins/Config.in new file mode 100644 index 0000000000..44aaa7997c --- /dev/null +++ b/package/alsa-plugins/Config.in @@ -0,0 +1,8 @@ +config BR2_PACKAGE_ALSA_PLUGINS + bool "alsa-plugins" + depends on BR2_PACKAGE_ALSA_LIB + select BR2_PACKAGE_LIBSAMPLERATE + help + Advanced Linux Sound Architecture Plugins + + http://www.alsa-project.org/ diff --git a/package/alsa-plugins/alsa-plugins.hash b/package/alsa-plugins/alsa-plugins.hash new file mode 100644 index 0000000000..8ecbd19965 --- /dev/null +++ b/package/alsa-plugins/alsa-plugins.hash @@ -0,0 +1,4 @@ +# Locally calculated +sha256 42eef98433d2c8d11f1deeeb459643619215a75aa5a5bbdd06a794e4c413df20 alsa-plugins-1.2.5.tar.bz2 +sha256 32434afcc8666ba060e111d715bfdb6c2d5dd8a35fa4d3ab8ad67d8f850d2f2b COPYING +sha256 231f7edcc7352d7734a96eef0b8030f77982678c516876fcb81e25b32d68564c COPYING.GPL diff --git a/package/alsa-plugins/alsa-plugins.mk b/package/alsa-plugins/alsa-plugins.mk new file mode 100644 index 0000000000..70d3cdb87c --- /dev/null +++ b/package/alsa-plugins/alsa-plugins.mk @@ -0,0 +1,23 @@ +############################################################# +# +# alsa-plugins +# +############################################################# +ALSA_PLUGINS_VERSION = 1.2.5 +ALSA_PLUGINS_SOURCE = alsa-plugins-$(ALSA_PLUGINS_VERSION).tar.bz2 +ALSA_PLUGINS_SITE = https://www.alsa-project.org/files/pub/plugins +ALSA_PLUGINS_LICENSE = LGPL-2.1+ (library), GPL-2.0+ (samplerate) +ALSA_PLUGINS_LICENSE_FILES = COPYING COPYING.GPL +ALSA_PLUGINS_INSTALL_STAGING = NO +ALSA_PLUGINS_AUTORECONF = YES +ALSA_PLUGINS_DEPENDENCIES = alsa-lib libsamplerate + +ALSA_PLUGINS_CONF_OPTS += \ + --enable-samplerate \ + --disable-jack \ + --disable-usbstream \ + --disable-pulseaudio \ + --disable-avcodec \ + --with-speex=builtin + +$(eval $(autotools-package))