Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

change Kwallet on undetected Desktops to Kwallet5 #63

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

maces
Copy link

@maces maces commented Feb 1, 2016

This changes the used kwallet version to Backend_Kwallet5 on undetected Desktops.
This enables users of LXQT to use Kwallet5.
Tested and working on Archlinux https://www.archlinux.org/packages/community/x86_64/qtkeychain-qt5/ just including the above change.

This changes the used kwallet version to Backend_Kwallet5 on undetected Desktops.
This enables users of LXQT to use Kwallet5. 
Tested and working on Archlinux https://www.archlinux.org/packages/community/x86_64/qtkeychain-qt5/ just including the above change.
@maces
Copy link
Author

maces commented Feb 1, 2016

My thought was: When Distributions renew their version of qtkeychain, they probably also update KDE and KWallet with it. On Arch KDE4 is already deprecated.
A configurable option or detection if kwallet4/5 is installed, preferring the later if both are installed, would be a better solution of course.

@maces maces mentioned this pull request Mar 9, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants