Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 816093 - kde-apps/okular-21.08.1 can't open password protected PDF file
Summary: kde-apps/okular-21.08.1 can't open password protected PDF file
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo KDE team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: kde-gear-21.08.3-stable
  Show dependency tree
 
Reported: 2021-10-03 17:35 UTC by Ostashevskyi Viktor
Modified: 2021-11-25 00:57 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 Ostashevskyi Viktor 2021-10-03 17:35:53 UTC
kde-apps/okular-21.08.1 can't open password protected PDF file stating "Can't open file"

Previous version kde-apps/okular-21.04.3-r2 works fine.

Reproducible: Always

Steps to Reproduce:
1. Get a password protected PDF
2. Open it with Okular

Actual Results:  
Error message

Expected Results:  
Password prompt
Comment 1 Ostashevskyi Viktor 2021-10-03 17:41:50 UTC
Ok, it looks like enabling 'kwallet' USE flag helps.

But this is confusing:

kwallet       : Enable encrypted storage of passwords with kde-frameworks/kwallet

I do not want to store password, I want to enter it each time.
Comment 2 Jonas Stein gentoo-dev 2021-10-03 17:44:38 UTC
Thank you for the report. We need to have all information at hand before ticket assignment. Please provide
* the complete build.log as attachment and
* paste the emerge info 
as described on https://wiki.gentoo.org/wiki/Attach_the_logs_to_the_bug_ticket

Please attach also one .pdf file with password to reproduce.

Can you remove the password with one of the described methods?
https://www.cyberciti.biz/faq/removing-password-from-pdf-on-linux/

@kde team: is the kwallet description correct?
Comment 3 Andreas Sturmlechner gentoo-dev 2021-10-03 20:44:59 UTC
(In reply to Jonas Stein from comment #2)
> @kde team: is the kwallet description correct?
No. It is in this case doing something entirely different than what the dependency has been traditionally used for.

If this functionality should be kept at all, it must become default enabled; but since many people are likely disabling kwallet, they will get that error - making me lean towards dropping the use flag again.
Comment 4 Esteve Varela Colominas 2021-10-04 01:08:50 UTC
Default-enable it or enable it in the plasma profile. The description thing is a general issue with gentoo USE flags, where enabling the "gtk" use flag can mean a whole range of different things depending on the app/library, and similar for all sorts of different flags.
I doubt many people would be affected by this when they have to make the conscious decision to disable it, and dropping the flag just genuinely seems dumb when upstream supports it and the work's been done... There's many reasons to drop a flag, a wrong default ain't one.
Comment 5 Andreas Sturmlechner gentoo-dev 2021-10-04 07:47:26 UTC
No, it's not just a matter of description, it is also a matter of USE flag naming that in this case is synonymous with the package in question.

USE=kwallet is already default enabled in plasma profile.

To keep it, it must be a different flag name and default enabled.
Comment 6 Esteve Varela Colominas 2021-10-05 09:56:50 UTC
Renaming is a fine solution, I'm just too used to flags like "sdl" having different meanings (sdl frontend? sdl audio backend replacing the regular one?) depending on context to let this one slide.
Another option (besides a package-local description for the use flag) is adding a post-install message if the use flag combination is uncommon/not recommended, explaining the side-effects. net-im/telegram-desktop[-gtk] had this for the longest time as that flag did little more than fix image copy-paste problems.
Comment 7 Andreas Sturmlechner gentoo-dev 2021-10-05 15:35:38 UTC
Bad examples do not make a policy for conflicting IUSE meanings, and disabling of a global IUSE going by its global description must not cause wholly unrelated unexpected behaviour.
Comment 8 Larry the Git Cow gentoo-dev 2021-11-24 19:07:42 UTC
The bug has been closed via the following commit(s):

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

commit aa1e11ad23a2fa0bcb307bcff7a92c654dd8420c
Author:     Andreas Sturmlechner <asturm@gentoo.org>
AuthorDate: 2021-11-24 18:41:47 +0000
Commit:     Andreas Sturmlechner <asturm@gentoo.org>
CommitDate: 2021-11-24 19:07:19 +0000

    kde-apps/okular: Rename IUSE kwallet -> crypt and fix description
    
    - kwallet was enabled by plasma profile
    - crypt is enabled by linux profile, so much wider coverage by default
    
    Closes: https://bugs.gentoo.org/816093
    Package-Manager: Portage-3.0.28, Repoman-3.0.3
    Signed-off-by: Andreas Sturmlechner <asturm@gentoo.org>

 kde-apps/okular/metadata.xml          | 2 +-
 kde-apps/okular/okular-21.08.3.ebuild | 6 +++---
 2 files changed, 4 insertions(+), 4 deletions(-)
Comment 9 Larry the Git Cow gentoo-dev 2021-11-24 19:41:07 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/proj/kde.git/commit/?id=7ffe879af1206d5e6c908622b71b6116f8ee01d1

commit 7ffe879af1206d5e6c908622b71b6116f8ee01d1
Author:     Andreas Sturmlechner <asturm@gentoo.org>
AuthorDate: 2021-11-24 18:41:47 +0000
Commit:     Andreas Sturmlechner <asturm@gentoo.org>
CommitDate: 2021-11-24 19:40:11 +0000

    kde-apps/okular: Rename IUSE kwallet -> crypt and fix description
    
    - kwallet was enabled by plasma profile
    - crypt is enabled by linux profile, so much wider coverage by default
    
    Closes: https://bugs.gentoo.org/816093
    Package-Manager: Portage-3.0.28, Repoman-3.0.3
    Signed-off-by: Andreas Sturmlechner <asturm@gentoo.org>

 kde-apps/okular/metadata.xml                | 2 +-
 kde-apps/okular/okular-21.11.80.ebuild      | 6 +++---
 kde-apps/okular/okular-21.12.49.9999.ebuild | 6 +++---
 kde-apps/okular/okular-9999.ebuild          | 6 +++---
 4 files changed, 10 insertions(+), 10 deletions(-)