Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 531350 - net-im/skype: add wrapper for running with media-sound/apulse
Summary: net-im/skype: add wrapper for running with media-sound/apulse
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Amy Liffey
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-12-01 16:03 UTC by Jauhien Piatlicki (RETIRED)
Modified: 2014-12-07 17:30 UTC (History)
3 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
ebuild diff (skype-4.3.0.37.diff,536 bytes, patch)
2014-12-05 16:13 UTC, Jauhien Piatlicki (RETIRED)
Details | Diff
skype (skype,46 bytes, text/plain)
2014-12-05 16:15 UTC, Jauhien Piatlicki (RETIRED)
Details

Note You need to log in before you can comment on or make changes to this bug.
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).