Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 512770 - [qt overlay] x11-themes/qtcurve-9999 needs KF5 compatibility
Summary: [qt overlay] x11-themes/qtcurve-9999 needs KF5 compatibility
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Unspecified (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Qt Bug Alias
URL:
Whiteboard:
Keywords:
: 518200 (view as bug list)
Depends on:
Blocks:
 
Reported: 2014-06-08 16:41 UTC by Elias Probst
Modified: 2014-12-25 00:13 UTC (History)
4 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 Elias Probst 2014-06-08 16:41:25 UTC
As kde-base/breeze uses qtcurve for the implementation of its widget style, x11-themes/qtcurve needs now to be KF5 compatible.

Right now, the 'kde' USE flag pulls in KDE 4 dependencies, but there's no option to make use of qtcurve's KDE integration in a KF5-only environment.
Comment 1 Michael Palimaka (kensington) gentoo-dev 2014-06-08 16:57:37 UTC
qtcurve does have an option and what appears to be an implementation of that option, but it's currently commented out.
Comment 2 Johannes Huber (RETIRED) gentoo-dev 2014-07-25 22:19:09 UTC
qtcurve is moved to the kde infrastructure:
https://github.com/QtCurve/qtcurve
https://projects.kde.org/projects/playground/base/qtcurve

@qt Maybe it would make sense to co-maintain the package or take it completly over to kde herd. What do you think?
Comment 3 Davide Pesavento (RETIRED) gentoo-dev 2014-07-26 01:03:25 UTC
(In reply to Johannes Huber from comment #2)
> @qt Maybe it would make sense to co-maintain the package or take it
> completly over to kde herd. What do you think?

I don't personally mind if you take it. If someone else in the qt team is interested, we can definitely co-maintain. In the latter case however, in which overlay should we maintain the live ebuild? Maybe we can move it to gx86?

[CC'ing also polynomial-c who is listed as maintainer of the in-tree package.]
Comment 4 Davide Pesavento (RETIRED) gentoo-dev 2014-07-28 11:44:37 UTC
*** Bug 518200 has been marked as a duplicate of this bug. ***
Comment 5 Johannes Huber (RETIRED) gentoo-dev 2014-07-28 20:41:01 UTC
(In reply to Davide Pesavento from comment #3)
> I don't personally mind if you take it. If someone else in the qt team is
> interested, we can definitely co-maintain. In the latter case however, in
> which overlay should we maintain the live ebuild? Maybe we can move it to
> gx86?
> 
> [CC'ing also polynomial-c who is listed as maintainer of the in-tree
> package.]

Live ebuilds should not be in tree in general, imho. If we take the maintainer ship i would suggest to maintain it in kde overlay. If you want to have a mirror i could keep them in sync.
Comment 6 Franz Trischberger 2014-09-04 09:12:57 UTC
On 28 of July Hugo Pereira Da Costa started implementing breeze style using current Oxygen code as a base. So breeze no longer depends on qtcurve, so it is not a needed dep for kde styles anymore. It IMHO should stay in qt overlay.

Would be nice to see some action here, as qtcurve-9999 is now broken since 23 of July.
Comment 7 Michael Palimaka (kensington) gentoo-dev 2014-09-04 11:37:37 UTC
I've updated the homepage and repo uri.

Not sure if there's anything further to do as per comment #6 and comment #1 still being true.
Comment 8 Davide Pesavento (RETIRED) gentoo-dev 2014-12-18 14:58:15 UTC
https://github.com/gentoo/qt/commit/ca968b37221985285a28e732896c5ea0f7dcd4ba

Can we close this? Anything else needs to be done?
Comment 9 Ben de Groot (RETIRED) gentoo-dev 2014-12-22 13:27:44 UTC
(In reply to Davide Pesavento from comment #8)
> Can we close this? Anything else needs to be done?

I didn't actually test this with KF5, so if anyone can give some feedback on that, that would be great!
Comment 10 Elias Probst 2014-12-22 14:09:35 UTC
I'd say this can be closed since ca968b37 implemented all required changes.
Comment 11 Ben de Groot (RETIRED) gentoo-dev 2014-12-25 00:13:33 UTC
If anything is missing, please let us know.