Page MenuHomeSolus

KeePassXC
Closed, ResolvedPublic

Description

Name: KeePassXC
Home Page: https://keepassxc.org/
Open Source: yes
Source: https://keepassxc.org/download#linux

Related Objects

StatusAssignedTask
ResolvedJoshStrobl
InvalidNone

Event Timeline

zjuhasz created this task.Mar 3 2017, 6:52 PM
Unknown Object (User) added a subscriber: Unknown Object (User).EditedMar 4 2017, 10:19 AM

I can't speak for the Solus devs, but Solus does have quite a few password managers already, including Keepass and KeepassX. What's different about this app?

timoll added a subscriber: timoll.Mar 4 2017, 11:00 AM

KeePassXC is a community fork of keepassx. It was forked because keepassX is hardly maintained(81 open pull requests right now, many without a comment)

Here is a discussion why it was forked:

https://github.com/keepassxreboot/keepassxc/issues/43#issuecomment-254058825

So my opinion would be to replace keepassx with keepassxc.

Unknown Object (User) added a comment.EditedMar 4 2017, 11:05 AM

Ah, in that case it's understandable. But you didn't say you wanted to replace it, that's why I replied the way I did ;) If it's maintained better than KeePassX, then I'm all for it to replace it.

@Vistaus He wasn't the one that filed the bug. Also, please stop acting like part of the triage team and assessing tasks. It is our responsibility to evaluate package requests.

Unknown Object (User) added a comment.EditedMar 4 2017, 11:08 AM

@Vistaus He wasn't the one that filed the bug. Also, please stop acting like part of the triage team and assessing tasks. It is our responsibility to evaluate package requests.

I'm not assessing tasks, I'm asking questions and vote in favor. I see other people on here doing the same, we're all just trying to help.

Unknown Object (User) awarded a token.Mar 4 2017, 11:11 AM
palto42 added a subscriber: palto42.Mar 5 2017, 2:53 PM
Okusa awarded a token.Mar 6 2017, 6:53 AM
Okusa added a subscriber: Okusa.
couchquid added a subscriber: couchquid.
JoshStrobl triaged this task as Normal priority.
JoshStrobl added a project: Package Requests.

At this moment in time, I'm likely going to go down the patch of replacing the keepassx source tarball with the tarball of keepassxc, thus invalidating T2816 since it won't be a separate package, nor do I view it necessary to have three separate keepass packages (one being Keepass in Mono, two being Qt clients where one is simply a fork). After review, I've determined that KeepassX is indeed unmaintained and the concerns from the KeepassXC fork are valid.

JoshStrobl closed this task as Resolved.Mar 7 2017, 8:13 PM

I have moved our KeepassX to using KeepassXC as of https://git.solus-project.com/packages/keepassx/commit/?id=ec46049e466099910ace98e4ec27a9e51447479b

Thanks @couchquid for your very helpful contribution with enabling fancy HTTP bits and the Yubikey support =)

ben added a subscriber: ben.Mar 7 2017, 10:27 PM

Can we explain that the keepassx package is using keepassxc sources in the package description?

Can we explain that the keepassx package is using keepassxc sources in the package description?

Yea, I did that if you look at the git commit.

ben added a comment.Mar 7 2017, 10:34 PM

Oh, great! I was looking in eopkg sr results, but this is not in shannon yet