Hello!

The top-level build script says "test: too many arguments" if 
EXTRA_ALSA_DRIVER_CONFIG_FLAGS is already defined to a string that 
contains spaces.

Also, since that string can contain options beginning with "-", it's 
better that we play safe and prepend "x" to it to avoid the possibility of 
confusing "test" by those options.

Fix:

=========================
--- build
+++ build
@@ -6,7 +6,7 @@
 
 # Comment this out if you don't want isapnp support in ALSA driver RPM
 #
-test -z $EXTRA_ALSA_DRIVER_CONFIG_FLAGS &&
+test x"$EXTRA_ALSA_DRIVER_CONFIG_FLAGS" = x &&
     EXTRA_ALSA_DRIVER_CONFIG_FLAGS="--with-isapnp=yes --with-sequencer=yes"
 export EXTRA_ALSA_DRIVER_CONFIG_FLAGS
 
=========================

-- 
Regards,
Pavel Roskin



-------------------------------------------------------
This sf.net email is sponsored by: OSDN - Tired of that same old
cell phone?  Get a new here for FREE!
https://www.inphonic.com/r.asp?r=sourceforge1&refcode1=vs3390
_______________________________________________
Alsa-devel mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/alsa-devel

Reply via email to