Created attachment 414172 [details] clementine-9999.ebuild I am adding Qt5 support, needed for testing
Thanks for your work! When you submit changes, then please always attach a unified diff against the original ebuild so your changes can be reviewed.
Created attachment 432848 [details] clementine-9999.ebuild Fix and attach it again.
Created attachment 432850 [details, diff] clementine-9999.ebuild.diff Fix and attach it again.
I've updated ebuild for 9999 to match current ebuild for 1.3.1-r6 and tested it with qt5 - at a glance it works fine. Patch needs to be fixed too. I'll attach new ebuild, ebuild diff and new patch. Clementine was the last one Qt4 app on my system! :)
Created attachment 467030 [details] clementine-9999.ebuild
Created attachment 467032 [details, diff] diff with clementine-1.3.1-r6.ebuild
Created attachment 467034 [details, diff] files/clementine-9999-fix-tokenizer.patch
Comment on attachment 467032 [details, diff] diff with clementine-1.3.1-r6.ebuild >+ # If qtchooser is installed, it may break the build, because moc,rcc and uic binaries for wrong qt version may be used. >+ # Setting QT_SELECT environment variable will enforce correct binaries. >+ if use qt4; then >+ export QT_SELECT=qt4 >+ elif use qt5; then >+ export QT_SELECT=qt5 >+ ewarn "Please note that Qt5 support is still experimental." >+ ewarn "If you find anything to not work with Qt5, please report a bug." >+ ewarn "«Work (slowly) in progress.»© https://github.com/clementine-player/Clementine/issues/3463#issuecomment-90714420" >+ fi Any idea of which/where binaries are called directly? We can patch the build system to use the correct version directly instead of relying on qtchooser.
(In reply to Michael Palimaka (kensington) from comment #8) > Any idea of which/where binaries are called directly? We can patch the build > system to use the correct version directly instead of relying on qtchooser. No, I've just copy&paste this from previous ebuild by Alexander. I can't test this on my system because I've no Qt4 anymore.
I want to postpone this a bit more: I'll try to push fix-tokenizer to upstream, so not to clutter the live version with one more patch... As for QT_SELECT stuff, I suspect it's redundant due to cmake should handle it on its own... Problems with the current ebuild are: - this fix-tokenizer patch won't apply to master branch.. - -DENABLE_VK=OFF is redundant for live - Dependencies should be verified one more time.
Created attachment 468110 [details] clementine-9999.ebuild Drop media-libs/libechonest dependency because of bug 587972 (current repo master already removed related code, no patches needed). P.S. If you excuse me I'll be a bit lazy and won't upload updated ebuild diff.
Created attachment 469046 [details] clementine-9999.ebuild Drop dev-libs/qjson dependency - it doesn't used by current clementine, at least with USE=qt5 (I didn't tried to build clementine with USE=qt4 without this dependency). This is important because dev-libs/qjson require a couple of qt4 libs, so should be either removed or moved to qt4-only deps block in ebuild.
Rather than adding Qt5 support, it should entirely replace Qt4 at this point.
The bug has been closed via the following commit(s): https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=f36c3c321d5e383adbfb2ed448ad1e9d1f598179 commit f36c3c321d5e383adbfb2ed448ad1e9d1f598179 Author: Andreas Sturmlechner <asturm@gentoo.org> AuthorDate: 2017-11-19 12:39:38 +0000 Commit: Andreas Sturmlechner <asturm@gentoo.org> CommitDate: 2017-11-19 15:09:08 +0000 media-sound/clementine: Switch 9999 to qt5 Forward-port fts3-tokenizer fix to qt5 branch. Add patch to properly find linguist-tools for lconvert. Closes: https://bugs.gentoo.org/562610 Package-Manager: Portage-2.3.14, Repoman-2.3.6 media-sound/clementine/clementine-9999.ebuild | 47 ++++++++++++++++------ .../files/clementine-fts3-tokenizer.patch | 36 +++++++++++++++++ .../clementine/files/clementine-qt5-lconvert.patch | 28 +++++++++++++ 3 files changed, 98 insertions(+), 13 deletions(-)