Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 578320 - x11-wm/lumina-0.8.8_p1: Incorrect dependencies
Summary: x11-wm/lumina-0.8.8_p1: Incorrect dependencies
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All All
: Normal normal (vote)
Assignee: Rubin Simons
URL:
Whiteboard:
Keywords: PMASKED
: 578348 (view as bug list)
Depends on:
Blocks:
 
Reported: 2016-03-26 18:33 UTC by Arfrever Frehtes Taifersar Arahesis
Modified: 2016-03-30 20:12 UTC (History)
2 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 Arfrever Frehtes Taifersar Arahesis 2016-03-26 18:33:19 UTC
x11-wm/lumina-0.8.8_p1 should depend on dev-qt/qtmultimedia:5[widgets] and dev-qt/qtsvg:5 (in DEPEND and RDEPEND).

In unpacked source archive:
$ grep -E "(multimediawidgets|svg)" **/*.pr[io]
libLumina/libLumina.pro:greaterThan(QT_MAJOR_VERSION, 4): QT += widgets x11extras multimedia concurrent svg
lumina-desktop/lumina-desktop.pro:greaterThan(QT_MAJOR_VERSION, 4): QT += widgets x11extras multimedia concurrent svg
lumina-fm/lumina-fm.pro:greaterThan(QT_MAJOR_VERSION, 4): QT += widgets multimedia multimediawidgets network concurrent svg


The only thing from dev-qt/linguist-tools:5 which is used here is lrelease and it is most likely used only at build time. So dependency on dev-qt/linguist-tools:5 should be only in DEPEND and not in RDEPEND. Currently it is also in RDEPEND due to 'RDEPEND="${DEPEND}'.
Comment 1 Rubin Simons 2016-03-28 08:53:50 UTC
Ah, thanks for the heads-up; these dependencies where missing from the upstream documentation and I had them installed already which is why I missed them. 

I'm setting up a container with a minimal gentoo installation to test packages as per idella4 and wraeth's recommendations. 

In the meantime, I've created a pull request here:
https://github.com/gentoo/gentoo/pull/1151
Comment 2 Anthony Basile gentoo-dev 2016-03-28 09:41:49 UTC
*** Bug 578348 has been marked as a duplicate of this bug. ***
Comment 3 Rubin Simons 2016-03-29 15:16:29 UTC
The pr is ready for testing and was already tested succesfully by Sam!
Comment 4 Anthony Basile gentoo-dev 2016-03-30 20:12:13 UTC
i committed this and tested.  its fine.  i'm going to remove the mask in a sec.  thanks for your work!