Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 618370 - net-im/skypeforlinux: missing proper ABI_X86 specification
Summary: net-im/skypeforlinux: missing proper ABI_X86 specification
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Raymond Jennings
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: x32
  Show dependency tree
 
Reported: 2017-05-13 15:16 UTC by Dainius Masiliūnas
Modified: 2017-10-04 18:27 UTC (History)
3 users (show)

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Dainius Masiliūnas 2017-05-13 15:16:04 UTC
The net-im/skypeforlinux ebuilds explicitly download and install the x86_64 RPM of Skype for Linux, however, the dependencies are not defined as being of that ABI. That results in the package being broken at runtime on the x32 ABI, since the x32 versions of the dependencies are pulled in instead of the x86_64 ones, causing library not found errors.

The old (x86) net-im/skype ebuilds declared the dependency ABI explicitly with [abi_x86_32(-)], so there was no issue with those.
Comment 1 Karol Grudziński 2017-05-19 20:23:27 UTC
I don't know the right way to do this:
1. use MULTILIB_COMPAT=( abi_x86_64 ) and add [${MULTILIB_USEDEP}] to relevant dependencies
2. don't set MULTILIB_COMPAT and just add [abi_x86_64] to dependencies
Comment 2 Raymond Jennings 2017-05-19 22:38:25 UTC
I am honestly stumped.

My blessing on whoever can fix this the cleanest.

I will ask a couple of questions though:

1.  Can skypeforlinux work on both x86 and amd64 archs?
Comment 3 Karol Grudziński 2017-05-19 23:19:13 UTC
There is only x86_64 RPM available and in official FAQ they say "We are only building Skype for Linux Alpha for a 64-bit system, as this is a first version and we want to see how people react to it. There may be a 32-bit version in the future." so I think the answer is "no".
Comment 4 Raymond Jennings 2017-05-20 00:31:03 UTC
Then shouldn't we just arch-mask x86 for skypeforlinux?
Comment 5 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2017-05-20 07:20:13 UTC
I think Karol's first solution is correct — it's pretty much the design intended by the eclass design. Karol, could you submit a PR so Raymond can review it by the code? ;-)
Comment 6 Karol Grudziński 2017-05-20 09:27:25 UTC
Here is my PR: https://github.com/gentoo/gentoo/pull/4687
Comment 7 Raymond Jennings 2017-10-04 18:26:54 UTC
I got sidetracked by real life and depression for awhile, plus family drama.  My apologies for being lazy.

I checked the PR itself on github and I'm not sure if anything actually got merged or not.

Since you're now co-pmaint for this package can you go ahead and let me know if the changes in question have been merged?  If they have, I can close this bug as resolved.
Comment 8 Raymond Jennings 2017-10-04 18:27:52 UTC
My mistake, it did get merged.

Marking resolved, and note to self to lern2github.