Bug 416460 - Replace Vokoscreen with VokoscreenNG
Summary: Replace Vokoscreen with VokoscreenNG
Status: RESOLVED FIXED
Alias: None
Product: frameworks-knewstuff
Classification: Frameworks and Libraries
Component: general (show other bugs)
Version: 5.66.0
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Jeremy Whiting
URL:
Keywords: junior-jobs
Depends on:
Blocks:
 
Reported: 2020-01-19 14:55 UTC by Olivier BELLEUX
Modified: 2020-04-07 19:28 UTC (History)
5 users (show)

See Also:
Latest Commit:
Version Fixed In: 5.70
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Olivier BELLEUX 2020-01-19 14:55:08 UTC
SUMMARY


STEPS TO REPRODUCE
1. 
2. 
3. 

OBSERVED RESULT
In the tool menu at the bottom there is an entry for "screen recorder" with submenu for 4 applications including vokoscreen.

Vokoscreen devellopment, which was stopped, was started again under the new name vokoscreenNG (see https://github.com/vkohaupt/vokoscreenNG and http://linuxecke.volkoh.de/vokoscreen/vokoscreen.html)

EXPECTED RESULT

I expected first that the menu is update to reflect the last devellopment of the project and last that Spectacle correctly detect VokoscreenNG when installed on operating system.

SOFTWARE/OS VERSIONS
Windows: 
macOS: 
Linux/KDE Plasma: 
(available in About System)
KDE Plasma Version: 
KDE Frameworks Version: 
Qt Version: 

ADDITIONAL INFORMATION

Thanks for your wonderfull works on Kde ! 

Cheers !
Comment 1 Nate Graham 2020-01-20 17:40:05 UTC
This is the commit that added vokoscreen in the first place: https://cgit.kde.org/knewstuff.git/commit/?id=bf82fe7c222fd635906f1c3e9380600559bfdc15

Changing some stuff there should be pretty easy. Would you like to have a go at it? I can help you.
Comment 2 Patrick Silva 2020-02-13 15:54:14 UTC
bug 417575 is related
Comment 3 Gastón Haro 2020-03-29 02:35:25 UTC
(In reply to Nate Graham from comment #1)
> This is the commit that added vokoscreen in the first place:
> https://cgit.kde.org/knewstuff.git/commit/
> ?id=bf82fe7c222fd635906f1c3e9380600559bfdc15
> 
> Changing some stuff there should be pretty easy. Would you like to have a go
> at it? I can help you.

Hi! I'll do it. I have one question though. The package is still called vokoscreen (at least in the AUR) for the vokoscreenNG version. Should I keep or update to the new name?

Maybe my question does not make a lot of sense. But hey it will be my first patch ;)

Cheers!
Comment 4 Gastón Haro 2020-03-29 07:06:13 UTC
I opened https://phabricator.kde.org/D28397
Comment 5 Nate Graham 2020-03-30 04:49:33 UTC
Thanks! You did great. Fantastic first patch.
Comment 6 Nate Graham 2020-04-07 18:31:05 UTC
Git commit 5613fd66c5f42c8acdcb72586292e5fd9346e7b3 by Nate Graham, on behalf of Gastón Haro.
Committed on 07/04/2020 at 18:30.
Pushed by ngraham into branch 'master'.

Replace Vokoscreen with VokoscreenNG

Summary:
The original Vokoscreen project has been abandoned
and superseeded by VokoscreenNG. This patch updates
the references.
FIXED-IN: 5.70

Test Plan:
Spectacle -> Tools -> Record Screen the new
VokoscreenNG entry should appear and the new
website referenced. (If other tools are installed
it will appear under "More")

Reviewers: #knewstuff, ngraham, leinir

Reviewed By: ngraham

Subscribers: apol, pino, IlyaBizyaev, kde-frameworks-devel

Tags: #frameworks

Differential Revision: https://phabricator.kde.org/D28397

M  +2    -2    data/kmoretools-desktopfiles/_update_all_files.sh
R  +3    -3    data/kmoretools-desktopfiles/vokoscreenNG.desktop [from: data/kmoretools-desktopfiles/vokoscreen.desktop - 098% similarity]
R  +-    --    data/kmoretools-desktopfiles/vokoscreenNG.png [from: data/kmoretools-desktopfiles/vokoscreen.png - 100% similarity]
M  +2    -2    src/kmoretools/kmoretoolspresets.cpp

https://commits.kde.org/knewstuff/5613fd66c5f42c8acdcb72586292e5fd9346e7b3
Comment 7 Olivier BELLEUX 2020-04-07 19:15:13 UTC
Good evening to all of you,

Once again, thank you all, for your work, you make Kde the best graphical interface for Gnu / Linux (in all objectivity, of course !)

Dear Nate, I can't wait to read your next blog post and find out what's new that all contributors have brought to KDE.

Cheers
Comment 8 Nate Graham 2020-04-07 19:28:06 UTC
:)