Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 644728 - app-i18n/*: remove deprecated Qt4 support
Summary: app-i18n/*: remove deprecated 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: CJK Team
URL:
Whiteboard: scheduled for removal on 2018-07-26
Keywords: PMASKED
Depends on: 644898
Blocks: qt4-removal
  Show dependency tree
 
Reported: 2018-01-16 01:44 UTC by Andreas Sturmlechner
Modified: 2018-08-07 07:27 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 Andreas Sturmlechner gentoo-dev 2018-01-16 01:44:55 UTC
Qt4 is dead and going away. Debian has dropped this package, please check if upstream has a Qt5 port available, if not, if there is any need for it in tree.
Comment 1 Andreas Sturmlechner gentoo-dev 2018-05-10 17:36:10 UTC
Let's use this bug to kick off removal of app-i18n/* Qt4 support before it is blocking Qt4 removal itself. At this point, the majority of leaf Qt4 reverse-dependencies is already pending removal or has updates queued up for stabilisation.

Affected packages:

app-i18n/fcitx
app-i18n/fcitx-unikey
app-i18n/ibus-qt
app-i18n/qimhangul
app-i18n/scim
app-i18n/scim-bridge
app-i18n/tagainijisho
app-i18n/uim
Comment 2 Andreas Sturmlechner gentoo-dev 2018-05-10 18:59:34 UTC
PR is removing optional Qt4 dependencies.

I could not find newer releases for the following packages where Qt4 dependency is mandatory:

app-i18n/ibus-qt
app-i18n/qimhangul
app-i18n/tagainijisho
Comment 3 Arfrever Frehtes Taifersar Arahesis 2018-05-10 20:37:51 UTC
app-i18n/tagainijisho -> bug #655416
Comment 4 Arfrever Frehtes Taifersar Arahesis 2018-05-10 21:03:01 UTC
Fcitx, IBus, SCIM, UIM are special. They are not GTK+/Qt/... applications, but plugins for GTK+/Qt/... which provide critical ability to write in GTK+/Qt/... applications. Therefore they need to be the very last place where GTK+/Qt/... support can be deleted.


(Old version of app-i18n/fcitx-unikey will be deleted during next cleanup...)


In case of app-i18n/fcitx (and app-i18n/fcitx-meta), the plan is to package.use.mask qt4 USE flag, without dropping support for Qt 4 in ebuilds for now.
I think that the same solution is reasonable also for:
app-i18n/ibus-qt       (package.mask of package for longer time)
app-i18n/scim          (package.use.mask of qt4 USE flag for longer time)
app-i18n/scim-bridge   (package.use.mask of qt4 USE flag for longer time)
app-i18n/uim           (package.use.mask of qt4 USE flag for longer time)
(IBus, SCIM and UIM currently have no dedicated maintainers. If a new dedicated maintainer for IBus, SCIM or UIM appears and wants to drop support for Qt 4 earlier, then I will not oppose.)


In case of app-i18n/qimhangul, upstream repository (https://github.com/choehwanjin/qimhangul) has no commits after 2011 and no sign of work on support for Qt 5, so package.masking of this package for deletion in 30 days seems to be reasonable.
Comment 5 Akinori Hattori gentoo-dev 2018-05-12 09:53:38 UTC
iBus immodule for Qt5 is integrated in QtGui (dev-qt/qtgui[ibus]), so app-i18n/ibus-qt can be removed.

As already Arfrever mentioned, app-i18n/qimhangul seems dead. There is a Qt5 port (https://github.com/peremen/qimhangul-qt5), but I don't know its status.
Comment 6 Arfrever Frehtes Taifersar Arahesis 2018-05-12 18:08:17 UTC
Presumably app-i18n/ibus-qt should be added to kde-sunset repository.
Comment 7 Andreas Sturmlechner gentoo-dev 2018-05-13 18:21:04 UTC
(In reply to Arfrever Frehtes Taifersar Arahesis from comment #4)
> Fcitx, IBus, SCIM, UIM are special. They are not GTK+/Qt/... applications,
> but plugins for GTK+/Qt/... which provide critical ability to write in
> GTK+/Qt/... applications. Therefore they need to be the very last place
> where GTK+/Qt/... support can be deleted.
Technically we can remove Qt4 support in ~arch as soon as everything has a Qt5 upgrade available in ~arch. Not opposed to package.use.mask though.

> (Old version of app-i18n/fcitx-unikey will be deleted during next cleanup...)
Why not do it now?
Comment 8 Larry the Git Cow gentoo-dev 2018-06-27 19:00:51 UTC
The bug has been referenced in the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=0c1a787876320cd3488d7f80878c2cf54ed2897b

commit 0c1a787876320cd3488d7f80878c2cf54ed2897b
Author:     Andreas Sturmlechner <asturm@gentoo.org>
AuthorDate: 2018-05-10 18:03:49 +0000
Commit:     Andreas Sturmlechner <asturm@gentoo.org>
CommitDate: 2018-06-27 19:00:21 +0000

    profiles: Mask Qt4-depending app-i18n/* packages for removal
    
    Bug: https://bugs.gentoo.org/644728

 profiles/package.mask | 6 ++++++
 1 file changed, 6 insertions(+)
Comment 9 Larry the Git Cow gentoo-dev 2018-08-07 07:27:55 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=6a543ac64d80e5ddb860e1653a269261f5d0e016

commit 6a543ac64d80e5ddb860e1653a269261f5d0e016
Author:     Andreas Sturmlechner <asturm@gentoo.org>
AuthorDate: 2018-07-01 16:44:28 +0000
Commit:     Andreas Sturmlechner <asturm@gentoo.org>
CommitDate: 2018-08-07 07:27:03 +0000

    app-i18n/*: Remove last-rited ibus-qt, qimhangul, tagainijisho
    
    Closes: https://bugs.gentoo.org/644728

 app-i18n/ibus-qt/Manifest                          |  1 -
 app-i18n/ibus-qt/files/ibus-qt-doc.patch           | 11 ----
 app-i18n/ibus-qt/ibus-qt-1.3.3.ebuild              | 53 -------------------
 app-i18n/ibus-qt/metadata.xml                      | 14 -----
 app-i18n/qimhangul/Manifest                        |  1 -
 app-i18n/qimhangul/metadata.xml                    |  8 ---
 app-i18n/qimhangul/qimhangul-0.2.0.ebuild          | 17 -------
 app-i18n/tagainijisho/Manifest                     |  1 -
 .../files/tagainijisho-1.0.3-sqlite.patch          | 15 ------
 app-i18n/tagainijisho/tagainijisho-1.0.3-r1.ebuild | 59 ----------------------
 profiles/package.mask                              |  6 ---
 11 files changed, 186 deletions(-)