[patch] startBristol process directly use sound device.

Bug #688847 reported by Takashi Sakamoto
12
This bug affects 1 person
Affects Status Importance Assigned to Milestone
monobristol (Ubuntu)
Fix Released
Medium
Stefan Ebner
Maverick
Fix Released
Medium
Unassigned

Bug Description

Binary package hint: monobristol

I think monobristol should open startBristol process with "-audiodev default" option.

Current version of startBristol directly opens sound device (hw(0,0)) without this option, not via PulseAudio. With "-audiodev default" option, startbristol opens default pcm device of ALSA. It's PulseAudio. Then we can see bristol pcm output in gnome-sound window. This is due to ALSA plugin of PulseAudio.

In my opinion, application's direct access to sound device is not good. If one application directly use the sound device, the other application cannot use it and there is no possibility to mix sounds from different appications.

To solve this issue, I attach 03_audiodev.patch.

Thanks to include my patch #660627 and It's my glad if you include this audiodev patch to distributed package.
Regards.

ProblemType: Bug
DistroRelease: Ubuntu 10.10
Package: monobristol 0.60.1-2~maverick1
ProcVersionSignature: Ubuntu 2.6.35-24.42-generic 2.6.35.8
Uname: Linux 2.6.35-24-generic i686
Architecture: i386
Date: Sat Dec 11 10:00:21 2010
InstallationMedia: Ubuntu-Studio 10.10 "Maverick Meerkat" - Beta i386 (20100902.1)
PackageArchitecture: all
ProcEnviron:
 LANGUAGE=ja_JP:ja:en
 LANG=ja_JP.utf8
 SHELL=/bin/bash
SourcePackage: monobristol

Revision history for this message
Takashi Sakamoto (mocchi) wrote :
Revision history for this message
Takashi Sakamoto (mocchi) wrote :
tags: added: patch
Revision history for this message
Stefan Ebner (sebner) wrote :

I'll take a look and talk to upstream the next few days.
Thanks for your work in advance!

Changed in monobristol (Ubuntu):
status: New → In Progress
assignee: nobody → Stefan Ebner (sebner)
Revision history for this message
Stefan Ebner (sebner) wrote :

@Takashi:

The upstream developer agrees with you so I will get an updated version uploaded in the next few days.
Thanks again!

Revision history for this message
Martin Pitt (pitti) wrote :

Stefan, please fix this in natty as soon as possible, before this SRU can go to maverick-updates.

Changed in monobristol (Ubuntu Maverick):
status: New → Fix Committed
tags: added: verification-needed
Revision history for this message
Martin Pitt (pitti) wrote : Please test proposed package

Accepted monobristol into maverick-proposed, the package will build now and be available in a few hours. Please test and give feedback here. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

Revision history for this message
Takashi Sakamoto (mocchi) wrote :

I tested it from maverick proposed and get unexpected behaviour. Then i review my patch and found my mistake to the patch...

In advanced mode, the value in "adriver" item should be valuated but in my patch "easyariver" is still valuated (at 21st line in previous patch). This is due to my miss-operation to create the patch, maybe subtle copy mistake...

I'm sorry for package mentainers to take more time just before your vacation...

Changed in monobristol (Ubuntu):
importance: Undecided → Medium
Changed in monobristol (Ubuntu Maverick):
importance: Undecided → Medium
Revision history for this message
Martin Pitt (pitti) wrote :

Accepted monobristol into maverick-proposed, the package will build now and be available in a few hours. Please test and give feedback here. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

Revision history for this message
Takashi Sakamoto (mocchi) wrote :

Now works fine in my maverick. Thanks and have a good year!

tags: added: verification-done
removed: verification-needed
Revision history for this message
Martin Pitt (pitti) wrote :

Still needs to be fixed in natty before this can be released.

Stefan Ebner (sebner)
Changed in monobristol (Ubuntu):
status: In Progress → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package monobristol - 0.60.1-2~maverick3

---------------
monobristol (0.60.1-2~maverick3) maverick-proposed; urgency=low

  * Fix small mistake in debian/patches/03_audiodev (LP: #688847)
 -- Stefan Ebner <email address hidden> Tue, 21 Dec 2010 01:01:58 +0100

Changed in monobristol (Ubuntu Maverick):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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