Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 531350

Summary: net-im/skype: add wrapper for running with media-sound/apulse
Product: Gentoo Linux Reporter: Jauhien Piatlicki (RETIRED) <jauhien>
Component: Current packagesAssignee: Amy Liffey <amynka>
Status: RESOLVED FIXED    
Severity: normal CC: martin, ulm, zlg
Priority: Normal    
Version: unspecified   
Hardware: All   
OS: Linux   
See Also: https://bugs.gentoo.org/show_bug.cgi?id=529098
Whiteboard:
Package list:
Runtime testing required: ---
Attachments: ebuild diff
skype

Description Jauhien Piatlicki (RETIRED) gentoo-dev 2014-12-01 16:03:10 UTC
Concider adding a wrapper script to call skype when apulse USE is set as media-sound/apulse no longer installs libraries in system location because of bug #529098. Wrapper should call skype using apulse.

I will add wrapper script and ebuild diff later.
Comment 1 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2014-12-04 19:28:36 UTC
Preferably use 'i686-pc-linux-gnu-apulse' since that will work correctly both on x86 and multilib amd64.
Comment 2 Jauhien Piatlicki (RETIRED) gentoo-dev 2014-12-05 16:13:19 UTC
Created attachment 391022 [details, diff]
ebuild diff
Comment 3 Jauhien Piatlicki (RETIRED) gentoo-dev 2014-12-05 16:15:36 UTC
Created attachment 391024 [details]
skype

skype wrapper
Comment 4 Jauhien Piatlicki (RETIRED) gentoo-dev 2014-12-05 16:17:38 UTC
@amynka: please, review and test these changes for x86 and amd64 (I have tested only for amd64, but should be ok for x86 as well) and commit them if you are ok with this.

Or comment here, so I can commit these changes.
Comment 5 zlg (RETIRED) gentoo-dev 2014-12-07 09:18:36 UTC
(In reply to Jauhien Piatlicki from comment #4)
> @amynka: please, review and test these changes for x86 and amd64 (I have
> tested only for amd64, but should be ok for x86 as well) and commit them if
> you are ok with this.
> 
> Or comment here, so I can commit these changes.

I can confirm the following to work:

`/usr/bin/i686-pc-linux-gnu-apulse skype`

I wasn't aware of the recent changes to apulse (or perhaps multilib?), and the default build of apulse on amd64 doesn't work for Skype since it's 32-bit.

It seems that your wrapper does the exact same thing as what I discovered, so consider me another vote in favor of the wrapper!
Comment 6 Amy Liffey gentoo-dev 2014-12-07 09:53:13 UTC
I tested it on amd64. However I don't have right now any machine for testing on x86. 

So for amd64 I can confirm it seems to be working fine. I think you can try to commit it (or should I?). We will see if someone will test it on x86 (it should work). 
They can still report bug I guess.

Thanks

Amynka
Comment 7 Ulrich Müller gentoo-dev 2014-12-07 10:03:56 UTC
Please commit Jauhien's changes. With USE=apulse, sound is currently broken on both amd64 and x86.

Also ~arch _is_ called "testing" for a reason. :)
Comment 8 Amy Liffey gentoo-dev 2014-12-07 10:52:15 UTC
(In reply to Ulrich Müller from comment #7)
> Please commit Jauhien's changes. With USE=apulse, sound is currently broken
> on both amd64 and x86.
> 
> Also ~arch _is_ called "testing" for a reason. :)

Yes Sir :)
Comment 9 Martin Väth 2014-12-07 17:30:56 UTC
Although I welcome the change, the implementation has a problem, see bug 531914
(I was not sure whether I should report this as a new bug).

As a second remark, it would be nice to pass arguments through the wrapper script
(see also bug 531914).