Skip to contentSkip to conversation

PATCH: Add more config options to alsa, bump version

Buildroot Will Wagner 3 participants 4 messages Jan 27, 2010 – Jan 28, 2010

lore
Share & tools
lore
#0 lore raw

This patch adds a lot of extra config options to alsa-lib (default should be the same as before options existed), with these you can substantially reduce the size of the alsa-lib library.

It also changes alsa-utils to only depend on ncurses if you have alsa-mixer selected.

Finally it bumps the version of alsa-lib and alsa-utils to 1.0.22
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: 0015-bump-alsa-version-add-options.patch
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20100127/56484916/attachment.diff>

#1 lore raw

Will Wagner a ?crit :

Quoted 8 lines

This patch adds a lot of extra config options to alsa-lib (default should be the same as before options existed), with these you can substantially reduce the size of the alsa-lib library.

It also changes alsa-utils to only depend on ncurses if you have alsa-mixer selected.

Finally it bumps the version of alsa-lib and alsa-utils to 1.0.22

<cut>

Review

Quoted 5 lines
(quoted diff) +5 -0
+ifneq ($(BR2_PACKAGE_ALSA_LIB_ALOAD),y)
+ALSA_LIB_CONF_OPT += --disable-aload
+endif
+ifneq ($(BR2_PACKAGE_ALSA_LIB_MIXER),y)
+ALSA_LIB_CONF_OPT += --disable-mizer

--disable-mixer, no ?

Regards,
Julien

#2 lore raw
In reply to Will Wagner

Have revised patch to fix the typo

Quoted 14 lines · Will Wagner

This patch adds a lot of extra config options to alsa-lib (default should be the same as before options existed), with these you can substantially reduce the size of the alsa-lib library.

It also changes alsa-utils to only depend on ncurses if you have alsa-mixer selected.

Finally it bumps the version of alsa-lib and alsa-utils to 1.0.22

_______________________________________________
buildroot mailing list
buildroot at busybox.net
http://lists.busybox.net/mailman/listinfo/buildroot

Signature · 13 lines
------------------------------------------------------------------------
Will Wagner                                     will_wagner at carallon.com
Development Manager                      Office Tel: +44 (0)20 7371 2032
Carallon Ltd, Studio G20, Shepherds Building, Rockley Rd, London W14 0DA
------------------------------------------------------------------------

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20100128/4ebd4ad9/attachment-0001.htm>
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: 0001-bump-alsa-version-add-options.patch
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20100128/4ebd4ad9/attachment-0001.diff>
#3 lore raw
Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line

"William" == William Wagner <[email protected]> writes:

William> Have revised patch to fix the typo

The removal of .la / .a files is wrong. That's globally handled in target-finalize if HAVE_DEVFILES isn't enabled. There were also some whitespace issues.

I've fixed both and committed - Thanks.

Signature · 1 line
Bye, Peter Korsgaard