Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 538450 - dev-qt/qtwebkit-5.4.0 unconditionally depends on dev-lang/ruby
Summary: dev-qt/qtwebkit-5.4.0 unconditionally depends on dev-lang/ruby
Status: RESOLVED INVALID
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Library (show other bugs)
Hardware: All Linux
: Normal normal with 1 vote (vote)
Assignee: Qt Bug Alias
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-02-01 15:54 UTC by Andrei Slavoiu
Modified: 2015-10-04 11:04 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 Andrei Slavoiu 2015-02-01 15:54:17 UTC
Looking at the source code Ruby is only required for LLInt (that the ebuild does not enable anyway) and some dev tools. It builds fine without ruby installed so please make it an optional dependency.

Reproducible: Always
Comment 1 Davide Pesavento (RETIRED) gentoo-dev 2015-02-24 21:04:15 UTC
Bug 540124 disagrees... besides, configure.prf checks that ruby is installed, and I see a ruby dependency in LLIntOffsetsExtractor.pro, which appears to be required by JavaScriptCore.

Would you mind explaining why you think that ruby is not needed?
Comment 2 Andrei Slavoiu 2015-02-25 03:24:02 UTC
The reason why I thought that ruby was not needed is that I removed dev-lang/ruby from DEPEND and qtwebkit "built" without it.

But apparently without ruby an error will be printed by qmake that nothing will be built even though it returns success. And because of that emerge will generate an empty packet.

But, everything that I have installed that depends on qtwebkit:5 (dev-qt/designer-5.4.0 and dev-qt/qtquick1-5.4.0) built fine like that.

BTW, when I said that ruby is only a dependency if llint is explicitly enabled I was looking at the cmake files. Those do not enable llint by default. But qmake does enable it and it is indeed required.