Bug in asoundconf

Bug #271899 reported by Joe_Bishop
84
Affects Status Importance Assigned to Milestone
alsa-utils (Ubuntu)
Fix Released
Low
Unassigned

Bug Description

Binary package hint: alsa-utils

/usr/bin/asoundconf uses configuration file /usr/share/alsa/alsa.conf, but it can't work with it properly: alsa.conf has line:
defaults.pcm.file_format "raw"
where ....file_format and "raw" is separated with tabular character, not spaces. But in current state /usr/bin/asoundconf supposes such lines are separated with spaces. The attached patch solves this issue. Or, you can replace \t in alsa.conf with spaces.

Revision history for this message
Joe_Bishop (denis-cheremisov-gmail) wrote :
Revision history for this message
Daniel T Chen (crimsun) wrote :

Thanks for catching this bug! I've been away from reading the conffile for some time, so I didn't see the additional whitespace changes.

Actually, for the separator, we want to handle any amount of whitespace of at least one character, so '\s+' is more appropriate.

Changed in alsa-utils:
importance: Undecided → Low
status: New → In Progress
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package alsa-utils - 1.0.17-0ubuntu2

---------------
alsa-utils (1.0.17-0ubuntu2) intrepid; urgency=low

  * Merge changes from asoundconf trunk to fix a crash in the asoundconf
    utility. (LP: #271899)

 -- Luke Yelavich <email address hidden> Thu, 25 Sep 2008 16:04:05 +1000

Changed in alsa-utils:
status: In Progress → Fix Released
Revision history for this message
dontiego (gauthier-fleutot) wrote :

I am running 8.04 (studio) with alsa-utils-1.0.18, and still get into:

$ asoundconf set-default-card EMU1010
Traceback (most recent call last):
  File "/usr/bin/asoundconf", line 429, in <module>
    exit_code(set_default_card(sys.argv[2]))
  File "/usr/bin/asoundconf", line 354, in set_default_card
    (j, k) = sep.split(i)
ValueError: need more than 1 value to unpack

Isn't this fix supposed to be in alsa-utils-1.0.18?
Or does it work only with intrepid?

I made the modification in "Bug attachments" by hand and it works, the affected lines were:
sep = re.compile(r'[ ]')
instead of
sep = re.compile(r'[ \t]')

Just curious about why the change isn't in alsa 1.0.18.

Revision history for this message
Joe_Bishop (denis-cheremisov-gmail) wrote : Re: [Bug 271899] Re: Bug in asoundconf

OMG, linux sound system is one big bug!
I made 3 bugreports yesterdau linked with sound system, it's too much!
And this buggy pulseaudio really annoy me. :(

2009/1/13 dontiego <email address hidden>

> I am running 8.04 (studio) with alsa-utils-1.0.18, and still get into:
>
> $ asoundconf set-default-card EMU1010
> Traceback (most recent call last):
> File "/usr/bin/asoundconf", line 429, in <module>
> exit_code(set_default_card(sys.argv[2]))
> File "/usr/bin/asoundconf", line 354, in set_default_card
> (j, k) = sep.split(i)
> ValueError: need more than 1 value to unpack
>
> Isn't this fix supposed to be in alsa-utils-1.0.18?
> Or does it work only with intrepid?
>
> I made the modification in "Bug attachments" by hand and it works, the
> affected lines were:
> sep = re.compile(r'[ ]')
> instead of
> sep = re.compile(r'[ \t]')
>
> Just curious about why the change isn't in alsa 1.0.18.
>
> --
> Bug in asoundconf
> https://bugs.launchpad.net/bugs/271899
> You received this bug notification because you are a direct subscriber
> of the bug.
>

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers