Page MenuHomeSolus

KeepassXC
Closed, InvalidPublic

Description

KeepassXC is an updated fork of KeepassX, a password manager.

Here is why they forked it: https://github.com/keepassxreboot/keepassxc/issues/43#issuecomment-254045934

This is my first package, please tell me if anything need to be changed.

Related Objects

StatusAssignedTask
ResolvedJoshStrobl
InvalidNone

Event Timeline

couchquid created this task.Mar 6 2017, 9:52 AM
couchquid added a subtask: T2788: KeePassXC.
couchquid updated the task description. (Show Details)Mar 6 2017, 10:12 AM

Why is a patch submission being provided for this when the request has yet to be accepted for inclusion?

I did not know that was the way it was supposed to be handled. So wait for a decision on a request before submitting patches?

I did not know that was the way it was supposed to be handled. So wait for a decision on a request before submitting patches?

Yes, wait to see if it's actually approved first. We already have Keepass and KeepassX, I'm going to need to review it against those, because ideally I don't want three Keepass clients in the repo.

Ok. Should i delete this?

No I'll mark it as invalid if the package request isn't accepted for inclusion.

couchquid updated the task description. (Show Details)Mar 7 2017, 8:07 AM
JoshStrobl closed this task as Invalid.Mar 7 2017, 7:31 PM
JoshStrobl mentioned this in T2788: KeePassXC.

I appreciate the patch, however I am opting to replace KeepassX source with KeepassXC, thus invalidating this patch. Please see my comment on the parent task for more info :)

I do want to note that I will be basing the changes off your work, so it's not all for nothing =)