Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 610372 - dev-libs/kdiagram-2.6.0 - missing dependencies on dev-qt/qtsql and dev-qt/linguist-tools
Summary: dev-libs/kdiagram-2.6.0 - missing dependencies on dev-qt/qtsql and dev-qt/lin...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo KDE team
URL:
Whiteboard:
Keywords: PATCH
Depends on:
Blocks:
 
Reported: 2017-02-21 13:24 UTC by Pablo Cholaky
Modified: 2017-02-21 22:47 UTC (History)
0 users

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


Attachments
Build log displaying qtsql requirement (build.log,4.41 KB, text/x-log)
2017-02-21 13:24 UTC, Pablo Cholaky
Details
Build log displaying linguist dependency (build.log.2,4.36 KB, text/x-log)
2017-02-21 13:26 UTC, Pablo Cholaky
Details
Proposed patch (kdiagram-2.6.0.patch,58 bytes, patch)
2017-02-21 13:30 UTC, Pablo Cholaky
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Pablo Cholaky 2017-02-21 13:24:47 UTC
Created attachment 464512 [details]
Build log displaying qtsql requirement

Hi there,

I'm recompiling my system from zero due a partition problem, and I got a problem compiling kdiagram due qtsql error. Attaching build.log.
Comment 1 Pablo Cholaky 2017-02-21 13:26:04 UTC
Created attachment 464514 [details]
Build log displaying linguist dependency

After adding $(add_qt_dep qtsql) on the ebuild, I got the same problem with linguist also
Comment 2 Pablo Cholaky 2017-02-21 13:30:48 UTC
Created attachment 464516 [details, diff]
Proposed patch

For now I made this patch to see if can solve it.
Comment 3 Pablo Cholaky 2017-02-21 17:14:22 UTC
With the patch worked for me.
Comment 4 Pablo Cholaky 2017-02-21 22:45:22 UTC
By some reason, dev-qt/linguist wasn't necessary, but dev-qt/linguist-tools
Comment 5 Andreas Sturmlechner gentoo-dev 2017-02-21 22:47:36 UTC
dev-qt/qtsql is only used in examples, we don't build examples, so the warning is irrelevant/has no effect.

dev-qt/linguist-tools was indeed missing from DEPEND, thanks for reporting!

Fixed in git commit f7e2e329aba224322017ae67d930dceb0074a7c7.