Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 682152 - app-text/calibre should DEPEND on dev-python/sip
Summary: app-text/calibre should DEPEND on dev-python/sip
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Ian Stakenvicius (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2019-03-31 16:15 UTC by Arthur Zamarin
Modified: 2019-04-02 06:00 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 Arthur Zamarin archtester Gentoo Infrastructure gentoo-dev Security 2019-03-31 16:15:09 UTC
After build failure and investigating why, found out that Calibre needs dev-python/sip as a dependency (build failure log without sip is attached). Installing sip fixed the issue.
Comment 1 Arthur Zamarin archtester Gentoo Infrastructure gentoo-dev Security 2019-03-31 16:21:38 UTC
Well, unable to attack because of unknown error with Bugs Gentoo...
Comment 2 Zac Medico gentoo-dev 2019-03-31 21:35:39 UTC
Since calibre already depends on dev-python/PyQt5, it already has an indirect dependency on dev-python/sip. Maybe the issue is that you had to rebuild dev-python/sip after a QT upgrade.
Comment 3 Arthur Zamarin archtester Gentoo Infrastructure gentoo-dev Security 2019-04-02 05:21:12 UTC
When I look into dev-python/PyQt5/PyQt5-5.12.1.ebuild, I see DEPEND on dev-python/sip, and no RDEPEND on it. calibre should also add sip as a DEPEND, because it is possible (as happened with me) that sip is cleaned as it isn't a runtime dependency, and when I want to rebuild calibre there is a missing build dependency.
It is the same as if a package needs virtual/pkgconfig, even if a dependency already build DEPENDS, you still need to add it to the local DEPEND.
Comment 4 Larry the Git Cow gentoo-dev 2019-04-02 05:58:54 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=5945e7c86317ee78064b05685ec18b122edf9d8b

commit 5945e7c86317ee78064b05685ec18b122edf9d8b
Author:     Zac Medico <zmedico@gentoo.org>
AuthorDate: 2019-04-02 05:54:17 +0000
Commit:     Zac Medico <zmedico@gentoo.org>
CommitDate: 2019-04-02 05:58:42 +0000

    app-text/calibre: add dev-python/sip to DEPEND
    
    Reported-by: Arthur Zamarin <arthurzam+gentoo@gmail.com>
    Closes: https://bugs.gentoo.org/682152
    Package-Manager: Portage-2.3.62, Repoman-2.3.12
    Signed-off-by: Zac Medico <zmedico@gentoo.org>

 app-text/calibre/calibre-3.26.1.ebuild | 3 ++-
 app-text/calibre/calibre-3.40.1.ebuild | 1 +
 2 files changed, 3 insertions(+), 1 deletion(-)
Comment 5 Zac Medico gentoo-dev 2019-04-02 05:59:40 UTC
Thanks!
Comment 6 Arthur Zamarin archtester Gentoo Infrastructure gentoo-dev Security 2019-04-02 06:00:19 UTC
Thank you too!