Bug 33564 - kde-pdf-servicemenu is mostly broken
Summary: kde-pdf-servicemenu is mostly broken
Status: NEW
Alias: None
Product: Mageia
Classification: Unclassified
Component: RPM Packages (show other bugs)
Version: 9
Hardware: All Linux
Priority: Normal normal
Target Milestone: ---
Assignee: David GEIGER
QA Contact:
URL:
Whiteboard:
Keywords: advisory
Depends on:
Blocks:
 
Reported: 2024-09-17 15:26 CEST by Aurelian R
Modified: 2024-09-21 00:10 CEST (History)
2 users (show)

See Also:
Source RPM: kde-pdf-servicemenu-0.6-10.mga9.src
CVE:
Status comment:


Attachments
spec patch (3.34 KB, patch)
2024-09-17 15:26 CEST, Aurelian R
Details | Diff
source patch (4.51 KB, patch)
2024-09-18 21:13 CEST, Aurelian R
Details | Diff

Description Aurelian R 2024-09-17 15:26:52 CEST
Created attachment 14662 [details]
spec patch

Description of problem:
kde-pdf-servicemenu, a KDE service menu for dolphin that enables/provides menu entries of various tasks for handling PDF files (https://store.kde.org/p/998465), is mostly broken in Mageia 9, while it is not loading on Cauldron. Using the latest version (2.3) of kde-pdf-servicemenu and dealing with some quirks, it can be made to work properly on Mageia 9, while on Cauldron only the booklet feature is not working, and that has to do with pdfjam script of texlive and not the kde-pdf-servicemenu.

How reproducible: Always.

Steps to reproduce the issue:
1. Install kde-pdf-servicemenu and/or kde-pdfnup-servicemenu (this one will pull the texlive main packages, over 2GB). 
2. Open Dolphin, select a pdf file, press mouse right-button on it, open the Actions submenu and start testing the listed tasks.
3. Most pdf tasks fail; only converting to djvu or to pictures work.

Attached is a patch for the "kde-pdf-servicemenu.spec" file, which works for me on Mageia9/Cauldron.

N.B.: On Cauldron, the pdfjam (v3.10) script fails, but upgrading it to version 3.12 seems to make it work properly.

Regards.
Comment 1 Lewis Smith 2024-09-17 21:45:05 CEST
Thank you once again for your thorough report with a correction.

Mageia 9 SRPMs are cited, although this is for Cauldron. Note both:
- kde-pdf-servicemenu
- texlive-texmf

Assigning as KDE.

Assignee: bugsquad => kde
Source RPM: kde-pdf-servicemenu-0.6-10.mga9.src => kde-pdf-servicemenu-0.6-10.mga9.src, texlive-texmf-20220321-12.mga9.src.rpm

Comment 2 David GEIGER 2024-09-18 10:08:35 CEST
Fixed both mga9 and Cauldron!

Assigning to QA,

Packages in 9/Core/Updates_testing:
======================
kde-pdf-servicemenu-2.3-1.mga9.noarch.rpm
kde-pdfnup-servicemenu-2.3-1.mga9.noarch.rpm


From SRPMS
kde-pdf-servicemenu-2.3-1.mga9.src.rpm

CC: (none) => geiger.david68210
Version: Cauldron => 9
Assignee: kde => qa-bugs

Comment 3 Morgan Leijström 2024-09-18 17:58:00 CEST
mga9-64 mini test OK of kde-pdf-servicemenu-2.3-1.mga9.noarch.rpm

I tried right clicking a pdf file in Dolphin, > Actions > Convert
And tried html, ps and some more

Before update nothing happened, and in system journal I spotted:
   kdialog: Unknown option 'caption'.

After update, html page was correctly generated with separate image files, and ps files works too.  Output in same folder as source pdf.

CC: (none) => fri

katnatek 2024-09-18 19:09:25 CEST

Source RPM: kde-pdf-servicemenu-0.6-10.mga9.src, texlive-texmf-20220321-12.mga9.src.rpm => kde-pdf-servicemenu-0.6-10.mga9.src

katnatek 2024-09-18 19:32:10 CEST

Keywords: (none) => advisory

Comment 4 Aurelian R 2024-09-18 21:13:56 CEST
Created attachment 14666 [details]
source patch

It turns out that I have missed two bugs, one for customized repagination option, which is not going to be easily fixed and it would be better to remove the option, and the other for djvu custom dpi option, which looks more like a typo bug.

Here are the additional sed instructions for the spec file, which should fix them:
# Remove option for custom repaginate
sed -e '/Action nup_custom/,/^$/{/^#/!d};s/nup_custom;//' \
    -i ServiceMenus/pdf-repaginate.desktop
# Fix djvu custom convert
sed -e "s/dpi %F/custom %F/" -i ServiceMenus/pdf-convert.desktop

For convenience, I attached a patch that is equivalent to all sed commands in the spec file.
Comment 5 katnatek 2024-09-21 00:10:32 CEST
Return to David due comment#4

Assignee: qa-bugs => geiger.david68210


Note You need to log in before you can comment on or make changes to this bug.