Bug#761023: bb: Visual stops when audio starts under pulseaudio

2015-04-12 Thread Antoine Amarilli
On Sun, Apr 12, 2015 at 12:03:00AM +0200, Axel Beckert wrote:
> Antoine Amarilli wrote:
> > It makes me wonder; would it make sense, instead of disabling the music
> > by default, to have a wrapper that runs bb with this hack, so that the
> > music can work? (Otherwise the user need to find and read the README to
> > find out about this, which is not ideal...)
> 
> Yes, that idea came to me, too, and may be a valid fix for this bug
> (which is still open), BUT it's definitely out of scope for the Jessie
> release.

Sure. I was suggesting this in general, I don't know anything about
specific milestones. :)

> The fix I'm imagining would be a wrapper script which does the
> following:
> 
> * Checking if it runs under X and PulseAudio is active.
> * If so, use the above mention commandline.
> * Else just start bb normaly.

Indeed I had missed that pulseaudio depended on pulseaudio-utils, so
it's reasonable to assume that pasuspender is available whenever
pulseaudio is run and there was no reason to worry. Sorry for the
confusion. Your suggestion looks fine to me.

-- 
Antoine Amarilli



signature.asc
Description: Digital signature


Bug#761023: bb: Visual stops when audio starts under pulseaudio

2015-04-11 Thread Axel Beckert
Hi,

Antoine Amarilli wrote:
> On Fri, Apr 10, 2015 at 07:29:57PM -0400, Anthony DeRobertis wrote:
> > If you do a README.Debian update to fix that typo, here is another
> > thing to add—this makes bb work on my PA system:
> > 
> > pasuspender -- env PULSE_SERVER= bb
> > 
> > pasuspender makes PulseAudio release the sound hardware, then setting
> > PULSE_SERVER= to blank makes bb not connect to PA (falling back to ALSA
> > instead, which works).
> 
> That's very nice, thanks!
> 
> It makes me wonder; would it make sense, instead of disabling the music
> by default, to have a wrapper that runs bb with this hack, so that the
> music can work? (Otherwise the user need to find and read the README to
> find out about this, which is not ideal...)

Yes, that idea came to me, too, and may be a valid fix for this bug
(which is still open), BUT it's definitely out of scope for the Jessie
release. I've also discussed that with Niels of the Release Team and
he agrees that anything else than a documentation update is out of
scope that close to the release.

> Of course, the devil is in the details: implementing this solution would
> require pasuspender,

No, it wouldn't. The fix I'm imagining would be a wrapper script which
does the following:

* Checking if it runs under X and PulseAudio is active.
* If so, use the above mention commandline.
* Else just start bb normaly.

That would not need any additional dependencies as far as I can see.

Regards, Axel
-- 
 ,''`.  |  Axel Beckert , http://people.debian.org/~abe/
: :' :  |  Debian Developer, ftp.ch.debian.org Admin
`. `'   |  4096R: 2517 B724 C5F6 CA99 5329  6E61 2FF9 CD59 6126 16B5
  `-|  1024D: F067 EA27 26B9 C3FC 1486  202E C09E 1D89 9593 0EDE


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org



Bug#761023: bb: Visual stops when audio starts under pulseaudio

2015-04-11 Thread Anthony DeRobertis

On 04/11/2015 01:05 PM, Antoine Amarilli wrote:


It makes me wonder; would it make sense, instead of disabling the music
by default, to have a wrapper that runs bb with this hack, so that the
music can work? (Otherwise the user need to find and read the README to
find out about this, which is not ideal...)

Of course, the devil is in the details: implementing this solution would
require pasuspender, hence pulseaudio-utils; however having bb depend on
pulseaudio-utils would be strange as the hack is only needed because of
pulseaudio itself.


Well, pulseaudio Depends: pulseaudio-utils. So you're not going to find 
many PA systems without it. I suppose it might be possible if the system 
is using a remote PA daemon, but that's by far not a normal setup.


So you could do something like:

   if command -v pasuspender > /dev/null; then
pasuspender -- env PULSE_SERVER= bb
   else
bb
   fi

Downsides: Anything else playing audio through PA is suddenly going to 
go silent.


The PA mixer still works, so things like desktop volume keys should 
continue to work.



--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org



Bug#761023: bb: Visual stops when audio starts under pulseaudio

2015-04-11 Thread Antoine Amarilli
On Fri, Apr 10, 2015 at 07:29:57PM -0400, Anthony DeRobertis wrote:
> If you do a README.Debian update to fix that typo, here is another
> thing to add—this makes bb work on my PA system:
> 
> pasuspender -- env PULSE_SERVER= bb
> 
> pasuspender makes PulseAudio release the sound hardware, then setting
> PULSE_SERVER= to blank makes bb not connect to PA (falling back to ALSA
> instead, which works).

That's very nice, thanks!

It makes me wonder; would it make sense, instead of disabling the music
by default, to have a wrapper that runs bb with this hack, so that the
music can work? (Otherwise the user need to find and read the README to
find out about this, which is not ideal...)

Of course, the devil is in the details: implementing this solution would
require pasuspender, hence pulseaudio-utils; however having bb depend on
pulseaudio-utils would be strange as the hack is only needed because of
pulseaudio itself. Ideally the wrapper could be smarter and only disable
the music when pulseaudio seems in use but pasuspender is not installed,
but that's not so easy because bb does not seems to have flags to
control the music from the command-line when running it.

So I don't know if that's a reasonable idea, but I wanted to make the
suggestion.

-- 
Antoine Amarilli



signature.asc
Description: Digital signature


Bug#761023: bb: Visual stops when audio starts under pulseaudio

2015-04-11 Thread Axel Beckert
Hi Anthony,

Anthony DeRobertis wrote:
> If you do a README.Debian update to fix that typo, here is another
> thing to add—this makes bb work on my PA system:
> 
> pasuspender -- env PULSE_SERVER= bb
> 
> pasuspender makes PulseAudio release the sound hardware, then setting
> PULSE_SERVER= to blank makes bb not connect to PA (falling back to ALSA
> instead, which works).

Thanks for that information. I wouldn't have done another upload just
because of that typo, but the information you've sent is possible a
reason to do another upload.

Regards, Axel
-- 
 ,''`.  |  Axel Beckert , http://people.debian.org/~abe/
: :' :  |  Debian Developer, ftp.ch.debian.org Admin
`. `'   |  4096R: 2517 B724 C5F6 CA99 5329  6E61 2FF9 CD59 6126 16B5
  `-|  1024D: F067 EA27 26B9 C3FC 1486  202E C09E 1D89 9593 0EDE


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org



Bug#761023: bb: Visual stops when audio starts under pulseaudio

2015-04-10 Thread Anthony DeRobertis
Package: bb
Followup-For: Bug #761023

-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

If you do a README.Debian update to fix that typo, here is another
thing to add—this makes bb work on my PA system:

pasuspender -- env PULSE_SERVER= bb

pasuspender makes PulseAudio release the sound hardware, then setting
PULSE_SERVER= to blank makes bb not connect to PA (falling back to ALSA
instead, which works).

- -- System Information:
Debian Release: 8.0
  APT prefers testing
  APT policy: (500, 'testing'), (130, 'unstable'), (120, 'experimental')
Architecture: amd64 (x86_64)
Foreign Architectures: i386

Kernel: Linux 3.16.0-4-amd64 (SMP w/8 CPU cores)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/bash
Init: systemd (via /run/systemd/system)

Versions of packages bb depends on:
ii  libaa1  1.4p5-43
ii  libc6   2.19-17
ii  libmikmod3  3.3.7-1
ii  oss-compat  6

bb recommends no packages.

bb suggests no packages.

- -- no debconf information

-BEGIN PGP SIGNATURE-
Version: GnuPG v1

iEYEARECAAYFAlUoXPIACgkQ+z+IwlXqWf4jhACgisipycEFHwdCVlA1kxQtyW6C
M2wAniiFqSHa+qvDUub6sRTO6bcpleLg
=/Yoz
-END PGP SIGNATURE-


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org