Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 447720 - net-im/skype should not DEPEND on x11-libs/qt-core:4
Summary: net-im/skype should not DEPEND on x11-libs/qt-core:4
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: AMD64 Linux
: Normal normal (vote)
Assignee: No maintainer - Look at https://wiki.gentoo.org/wiki/Project:Proxy_Maintainers if you want to take care of it
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-12-18 15:08 UTC by Hristo Venev
Modified: 2012-12-21 01:09 UTC (History)
1 user (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 Hristo Venev 2012-12-18 15:08:41 UTC
net-im/skype has a build time dependency on qt. On amd64 it's not needed.

Reproducible: Always
Comment 1 Pacho Ramos gentoo-dev 2012-12-18 20:24:28 UTC
(In reply to comment #0)
> net-im/skype has a build time dependency on qt. On amd64 it's not needed.
> 
> Reproducible: Always

Why not? I guess it will need emul qtlibs then, no?
Comment 2 Julian Ospald 2012-12-18 20:37:52 UTC
(In reply to comment #1)
> (In reply to comment #0)
> > net-im/skype has a build time dependency on qt. On amd64 it's not needed.
> > 
> > Reproducible: Always
> 
> Why not? I guess it will need emul qtlibs then, no?

yes, there is no linking to amd64 libs.

also, there is no build-time dependency at all
Comment 3 Eric F. GARIOUD 2012-12-18 21:17:24 UTC
I think that the following snippet of code in skype's ebuild is responsible for this dependency :

src_compile() {
	type -P lrelease >/dev/null && lrelease lang/*.ts
	rm -f lang/*.ts
}

This code is not arch dependent and an - equery belongs lrelease - will tell you that lrelease comes with the qt-core package.

So qt-core might not be require for skype to run on amd64, however... it is actually needed to build the package.
Comment 4 Pacho Ramos gentoo-dev 2012-12-18 21:27:56 UTC
It's explained at:
https://bugs.gentoo.org/show_bug.cgi?id=443440#c14

per that, we should only add a RDEPEND on emul qtlibs (and keep qt-core for lrelease)
Comment 5 Pacho Ramos gentoo-dev 2012-12-18 21:28:44 UTC
(In reply to comment #4)
> It's explained at:
> https://bugs.gentoo.org/show_bug.cgi?id=443440#c14
> 
> per that, we should only add a RDEPEND on emul qtlibs (and keep qt-core for
> lrelease)

If that is only for translations, probably we could put it behind a "nls" USE flag
Comment 6 Julian Ospald 2012-12-18 21:38:24 UTC
*.qm files are always present regardless of "lrelease lang/*.ts"

I don't know if it has any particular advantage to regenerate them, so I'd just punt the qt-core dependency.
Comment 7 Julian Ospald 2012-12-21 01:09:16 UTC
+  21 Dec 2012; Julian Ospald <hasufell@gentoo.org> skype-4.1.0.20.ebuild:
+  drop hard-dep on qt-core wrt #447720
+