Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 631914 - media-sound/qtractor - drop qt4 support
Summary: media-sound/qtractor - drop qt4 support
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Professional Audio Applications Maintainers
URL:
Whiteboard:
Keywords:
Depends on: 633490
Blocks: qt4-removal
  Show dependency tree
 
Reported: 2017-09-24 09:07 UTC by Michael Palimaka (kensington)
Modified: 2017-10-14 12:25 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 Michael Palimaka (kensington) gentoo-dev 2017-09-24 09:07:25 UTC
It looks like media-sound/qtractor supports both qt4 and qt5. Since qt4 is dead, please drop qt4 support and just enable qt5 unconditionally.
Comment 1 Alexis Ballier gentoo-dev 2017-09-24 12:56:18 UTC
feel free to drop the flag
Comment 2 Simon 2017-09-24 21:33:03 UTC
Came here from https://github.com/gentoo/gentoo/pull/5788.
Already posted the same comment there, but also wanted to mention it here

Enabling Qt5 on Qtractor means one has to enable Qt5 on all (lv2) plugins which use Qt as well, otherwise Qtractor crashes.

Unfortunately the released versions of suil don't support Qt5 inside gtk, which means that enabling Qt5 would make those plugins useless in suil based hosts (like jalv).
So I'd prefer to either wait until there's a new suil release or add a datestamped ebuild for suil from current master (which does support Qt5 inside gtk).

Also Qtractor upstream still supports Qt4. Why not just keep the option?
Comment 3 Michael Palimaka (kensington) gentoo-dev 2017-09-25 12:52:03 UTC
(In reply to Simon from comment #2)
> Came here from https://github.com/gentoo/gentoo/pull/5788.
> Already posted the same comment there, but also wanted to mention it here
> 
> Enabling Qt5 on Qtractor means one has to enable Qt5 on all (lv2) plugins
> which use Qt as well, otherwise Qtractor crashes.
> 
> Unfortunately the released versions of suil don't support Qt5 inside gtk,
> which means that enabling Qt5 would make those plugins useless in suil based
> hosts (like jalv).
> So I'd prefer to either wait until there's a new suil release or add a
> datestamped ebuild for suil from current master (which does support Qt5
> inside gtk).

Sure, we can wait for a while, it's not particularly urgent. Thanks for the feedback. 

> Also Qtractor upstream still supports Qt4. Why not just keep the option?
Qt 4 is dead and we (Qt team) have no intentions of starting a local fork to try and keep it alive. We're already seeing problems building it with newer libraries and compilers so want to remove it when we can. Investigating low hanging fruit (ie. packages that support both qt4 and qt5) for cleanup is a good way to start.
Comment 4 Andreas Sturmlechner gentoo-dev 2017-09-26 18:57:54 UTC
(In reply to Simon from comment #2)
> So I'd prefer to either wait until there's a new suil release or add a
> datestamped ebuild for suil from current master (which does support Qt5
> inside gtk).

Please let us know when you got a response from suil upstream about a new release. Otherwise I'd say we throw a snapshot @ ~arch.
Comment 5 Simon 2017-09-27 08:53:56 UTC
(In reply to Andreas Sturmlechner from comment #4)
> Please let us know when you got a response from suil upstream about a new
> release. Otherwise I'd say we throw a snapshot @ ~arch.

I've received a response from drobilla, he expects to cut a release within the next two weeks, so even if it will be slightly longer it won't be a very long wait :)
Comment 6 Andreas Sturmlechner gentoo-dev 2017-10-11 18:01:47 UTC
If this works fine with suil-0.10.0 now I guess we can drop qt4 after all.
Comment 7 Andreas Sturmlechner gentoo-dev 2017-10-14 12:25:25 UTC
Thanks, Qt4 support removed in git commits d08d7f15fb125e6df0af0f291182637d873b5274 and bcf1d5102b2ad55fb6f2d39defc0b76bdf08e362