Skip to content

Conversation

@gjedeer
Copy link

@gjedeer gjedeer commented Mar 9, 2016

LXQt is a Qt environment and gnome-keyring feels alien.

Also idea: would you accept a patch which allows user to override auto-detection using an environment variable? I know it's now possible to override XDG_CURRENT_DESKTOP but this has other unwanted implications on the app using qtkeychain.

LXQt is a Qt environment and gnome-keyring feels alien.

Also idea: would you accept a patch which allows user to override auto-detection using an environment variable? I know it's now possible to override XDG_CURRENT_DESKTOP but this has other unwanted implications on the app using qtkeychain.
@maces
Copy link

maces commented Mar 9, 2016

Shouldn't LXQt use kwallet5 instead of kwallet4 (also see: #63)

@gjedeer
Copy link
Author

gjedeer commented Mar 9, 2016 via email

@maces
Copy link

maces commented Mar 9, 2016

Decision by user would be best option for me, too.

@frankosterfeld
Copy link
Owner

Who's the "user" here? The application developer or the end user? I can make the wallet preference configurable, but the UI to configure that would be something the applications using qtkeychain would have to implement.

Another option is a config file option, which might help some people but is not very user-friendly either...

@gjedeer
Copy link
Author

gjedeer commented Mar 11, 2016

I'm talking about the end user here.

The GUI would be a nice thing too, but even supporting a QTKEYCHAIN_BACKEND environment variable would be helpful. Sure, it's not as user friendly as a GUI, but at least it would be an easy answer for people who would like to use a different wallet than the one QtKeychain selected for them - I see you have a plenty of ticket with the same issue.

A GUI would be a nice addition to that, but I guess it will be years before application developers keep up with this.

@maces
Copy link

maces commented Mar 11, 2016

+1 for the environment variable QTKEYCHAIN_BACKEND, would be sufficient for my needs (currently compiled a custom version)

@sandsmark
Copy link

#75 for that environment variable thing

@michaelk83
Copy link

QTKEYCHAIN_BACKEND support added in #264. This can be closed now.

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.

5 participants