-
-
Notifications
You must be signed in to change notification settings - Fork 12.9k
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
qtpass: 1.2.3 -> 1.3.0 #67660
qtpass: 1.2.3 -> 1.3.0 #67660
Conversation
@GrahamcOfBorg build qtpass |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I see to just get a segfault when running this.
[nix-shell:~/.cache/nix-review/pr-67660]$ ./results/qtpass/bin/qtpass
Segmentation fault
@jonringer could you please providing e.g. an strace log about the issue? I just tested this localy and didn't encounter any issues :) |
Sure, strace logs: last 50ish lines
|
I can't reproduce the segfault. It's working fine for me. |
Patch has been upstreamed and should be safe to remove in versions > 1.3.0
44bbc93
to
d665d57
Compare
worked fine for me in nix-review on unstable. Even if it crashed, might be more of a program bug so I believe we can merge. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works for me and my password store under x11.
d665d57
to
8de860b
Compare
Motivation for this change
Updating to latest version. Also includes a patch to fix the missing app icon under Wayland (see IJHack/QtPass#468)
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @hrdinka