Bug 493223

Summary:	Rewrite man page search handler to remove the dependency on Perl
Product:	[Applications] khelpcenter	Reporter:	Timothée Ravier <travier>
Component:	general	Assignee:	Documentation Editorial Team <kde-doc-english>
Status:	RESOLVED FIXED
Severity:	normal	CC:	aacid
Priority:	NOR
Version:	20.08
Target Milestone:	---
Platform:	Fedora RPMs
OS:	Linux
Latest Commit:	https://invent.kde.org/system/khelpcenter/-/commit/dcc31023bd3a26bb33e468de97ba2e635ad33f8e	Version Fixed In:
Sentry Crash Report:

Description Timothée Ravier 2024-09-16 18:06:02 UTC

SUMMARY

The man page search handler (khc_mansearch.pl [1]) is a small Perl program that search for man pages.

This is currently the last user of Perl in the KDE Stack on an installed Fedora KDE / Kinoite system.

Rewriting it in Python or anything else already in use by KDE package would remove that dependency.

[1] https://invent.kde.org/system/khelpcenter/-/blob/master/searchhandlers/khc_mansearch.pl

STEPS TO REPRODUCE
1. Install khelpcenter

OBSERVED RESULT
Perl is being installed as a dependency

EXPECTED RESULT
Perl is not installed as a dependency

SOFTWARE/OS VERSIONS
Linux/KDE Plasma: Fedora KDE 40

Comment 1 Albert Astals Cid 2024-09-16 22:12:33 UTC

Or C++ which is what we do :D

Comment 2 Bug Janitor Service 2024-10-11 11:33:53 UTC

A possibly relevant merge request was started @ https://invent.kde.org/system/khelpcenter/-/merge_requests/53

Comment 3 Timothée Ravier 2024-10-18 15:14:43 UTC

Git commit dcc31023bd3a26bb33e468de97ba2e635ad33f8e by Timothée Ravier.
Committed on 11/10/2024 at 12:13.
Pushed by ravier into branch 'master'.

searchhandlers: Replace Perl manpage script with Python

See:
- https://bugs.kde.org/show_bug.cgi?id=493223
- https://pagure.io/fedora-kde/SIG/issue/566

M  +1    -1    searchhandlers/CMakeLists.txt
D  +0    -107  searchhandlers/khc_mansearch.pl
A  +79   -0    searchhandlers/khc_mansearch.py
M  +1    -1    searchhandlers/man.desktop.cmake

https://invent.kde.org/system/khelpcenter/-/commit/dcc31023bd3a26bb33e468de97ba2e635ad33f8e