Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 632024 - media-gfx/kuickshow depends on kdelibs4
Summary: media-gfx/kuickshow depends on kdelibs4
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Christian Gerloff
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: kdelibs4-removal
  Show dependency tree
 
Reported: 2017-09-26 00:05 UTC by Andreas Sturmlechner
Modified: 2018-01-13 17:41 UTC (History)
0 users

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 2017-09-26 00:05:43 UTC
Without a port to KDE Frameworks 5 this will need to be removed eventually. All hope is not completely lost since it was ported away from kde3-/qt3support, but under the last known upstream maintainer there was no activity in many years.
Comment 1 Christian Gerloff 2017-09-26 13:35:33 UTC
AFAIK upstream only supports gwenview now.

I've created a Git repo and imported all commits related to kuickshow from KDE's SVN repo into that. It's not yet public because I'm still cleaning up the history of various translation files which should be done by the end of this week.

After that's done I'll put it on GitHub and continue to try to move the code to KDE5.

When do you plan to remove KDE4 support?
Comment 2 Andreas Sturmlechner gentoo-dev 2017-09-26 17:11:40 UTC
Did you talk with upstream to look into continued hosting of the repository on cgit.kde.org and thus benefiting from the existing translations workflow? I'm sure that would have been an option.

From KDE side, first Applications release without kdelibs4-based packages will be 17.12.0, and stabilisation of that probably won't happen before February '18.

When you look at the remaining list, there is still some way to go: https://qa-reports.gentoo.org/output/genrdeps/rindex/kde-frameworks/kdelibs

So this is an early notification.
Comment 3 Christian Gerloff 2017-09-26 18:59:06 UTC
(In reply to Andreas Sturmlechner from comment #2)
> Did you talk with upstream to look into continued hosting of the repository
> on cgit.kde.org and thus benefiting from the existing translations workflow?
> I'm sure that would have been an option.
Not yet. I'm still not sure if the port can be done easily.
There's also the use of imlib which seems to be problematic. imlib.h includes some X.Org header files which do nasty stuff like #define'ing "index", "KeyPress", "None" and various other names used by Qt in enums. So as long as imlib is used, there's at least a chance that things will suddenly just break.

Until I've figured out these issues and actually ported the app, I just wanted to focus on the code without dealing with any KDE bureaucracy, especially if it's just for the translations. (They can still be done later.)

> From KDE side, first Applications release without kdelibs4-based packages
> will be 17.12.0, and stabilisation of that probably won't happen before
> February '18.
> 
> When you look at the remaining list, there is still some way to go:
> https://qa-reports.gentoo.org/output/genrdeps/rindex/kde-frameworks/kdelibs
> 
> So this is an early notification.
Ah thank you, that's good to know. My goal was to have something concrete in a month or two.
Comment 4 Andreas Sturmlechner gentoo-dev 2017-12-02 21:19:00 UTC
A revbump dropping KDE_LINGUAS will be necessary in order not to be blocked by the upcoming KDE Applications 17.12 release.
Comment 5 Christian Gerloff 2017-12-03 16:34:55 UTC
(In reply to Andreas Sturmlechner from comment #4)
> A revbump dropping KDE_LINGUAS will be necessary in order not to be blocked
> by the upcoming KDE Applications 17.12 release.

I have created a pull request (https://github.com/gentoo/gentoo/pull/6426) for the changes.
Comment 6 Andreas Sturmlechner gentoo-dev 2018-01-11 00:42:49 UTC
What's the status on kuickshow? I remember watching a #kde-devel discussion about the kuickshow repository, did you create an upstream sysadmin ticket for that? Anything we can track from here?
Comment 7 Christian Gerloff 2018-01-11 22:50:28 UTC
(In reply to Andreas Sturmlechner from comment #6)
> What's the status on kuickshow? I remember watching a #kde-devel discussion
> about the kuickshow repository, did you create an upstream sysadmin ticket
> for that? Anything we can track from here?

Yes, I did. The creation of the final repository is still a work in progress.

I do however have a tarball with the ported code and a working ebuild for KDE 5 ready, if time is an issue.
Comment 8 Andreas Sturmlechner gentoo-dev 2018-01-12 16:33:27 UTC
It's certainly not the last kdelibs-revdep at this point, but it is always nice to be able to tick off another item. ;) The prospect of this being revived on cgit.kde.org is good enough for me to add it to tree. If it helps, I could upload the tarball to my devspace.
Comment 9 Christian Gerloff 2018-01-12 18:13:30 UTC
That would be great. I haven't found anything in the Gentoo docu yet on how to upload new tarballs on my own (I probably don't have the permissions to do so).

Should I send it to your Gentoo eMail address? It's a little under 1MB compressed.
Comment 10 Andreas Sturmlechner gentoo-dev 2018-01-13 12:33:15 UTC
Yes, just send it should be fine. I can then fix the ebuild SRC_URI while pushing.
Comment 11 Larry the Git Cow gentoo-dev 2018-01-13 17:17:11 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=89cc6c34b103b3478596be5f49a3abe1995a6d19

commit 89cc6c34b103b3478596be5f49a3abe1995a6d19
Author:     Christian Gerloff <chrgerloff@gmx.net>
AuthorDate: 2018-01-13 16:40:26 +0000
Commit:     Andreas Sturmlechner <asturm@gentoo.org>
CommitDate: 2018-01-13 17:16:54 +0000

    media-gfx/kuickshow: remove old
    
    Closes: https://bugs.gentoo.org/632024
    Package-Manager: Portage-2.3.13, Repoman-2.3.3

 media-gfx/kuickshow/Manifest                  |  1 -
 media-gfx/kuickshow/kuickshow-0.9.1-r3.ebuild | 29 ---------------------------
 2 files changed, 30 deletions(-)

Additionally, it has been referenced in the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=1764f421f0d31fa7d248a7af5b37fc9677e3c9b5

commit 1764f421f0d31fa7d248a7af5b37fc9677e3c9b5
Author:     Christian Gerloff <chrgerloff@gmx.net>
AuthorDate: 2018-01-13 16:38:37 +0000
Commit:     Andreas Sturmlechner <asturm@gentoo.org>
CommitDate: 2018-01-13 17:16:54 +0000

    media-gfx/kuickshow: 0.10.0 KF5-based version bump
    
    Bug: https://bugs.gentoo.org/632024
    Closes: https://github.com/gentoo/gentoo/pull/6851
    Package-Manager: Portage-2.3.13, Repoman-2.3.3

 media-gfx/kuickshow/Manifest                |  1 +
 media-gfx/kuickshow/kuickshow-0.10.0.ebuild | 36 +++++++++++++++++++++++++++++
 2 files changed, 37 insertions(+)}
Comment 12 Larry the Git Cow gentoo-dev 2018-01-13 17:41:51 UTC
The bug has been referenced in the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=1cec8f8226eff85b3614ceb5b1f60fb90aa2b91b

commit 1cec8f8226eff85b3614ceb5b1f60fb90aa2b91b
Author:     Andreas Sturmlechner <asturm@gentoo.org>
AuthorDate: 2018-01-13 17:33:59 +0000
Commit:     Andreas Sturmlechner <asturm@gentoo.org>
CommitDate: 2018-01-13 17:41:00 +0000

    media-gfx/kuickshow: Add implicit missing DEPENDs
    
    Bug: https://bugs.gentoo.org/632024
    Package-Manager: Portage-2.3.19, Repoman-2.3.6

 media-gfx/kuickshow/kuickshow-0.10.0.ebuild | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=34006270027c40fa6fbde0d955bfe0eaa4c2de94

commit 34006270027c40fa6fbde0d955bfe0eaa4c2de94
Author:     Andreas Sturmlechner <asturm@gentoo.org>
AuthorDate: 2018-01-13 17:28:34 +0000
Commit:     Andreas Sturmlechner <asturm@gentoo.org>
CommitDate: 2018-01-13 17:41:00 +0000

    media-gfx/kuickshow: Fix desktop file
    
    Bug: https://bugs.gentoo.org/632024
    Package-Manager: Portage-2.3.19, Repoman-2.3.6

 media-gfx/kuickshow/files/kuickshow-0.10.0-desktop-fix.patch | 11 +++++++++++
 media-gfx/kuickshow/kuickshow-0.10.0.ebuild                  |  2 ++
 2 files changed, 13 insertions(+)}