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

Remote bug watches

Bug watches keep track of this bug in other bug trackers.