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
Add keysmith #88614
Add keysmith #88614
Conversation
Your name is still missing from the maintainers list. |
Fixed |
buildInputs = [ oathToolkit kirigami2 qtquickcontrols2 qtbase ]; | ||
|
||
meta = { | ||
description = "OTP client for Plasma Mobile and Desktop."; |
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.
This shouldn't have a period at the end, as the contributing guidelines state.
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.
please cleanup git history as well, should only have 2 commits:
maintainers: add shamilton
keysmith: init at 0.1
Also, thanks for opening your first PR with nixpkgs :) |
You're welcome 👍 (Even though it's my third) |
Rebased to new branch, moved to #89413 |
Discovered git force pushing so coming back to this thread and closing the new one. |
811e397
to
d8fb6c5
Compare
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 get issues trying to launch it:
[nix-shell:/home/jon/.cache/nixpkgs-review/pr-88614]$ ./results/keysmith/bin/org.kde.keysmith
QQmlApplicationEngine failed to load component
qrc:/main.qml:29 Type Kirigami.ApplicationWindow unavailable
file:///nix/store/8jhqa8q59r448c94yj88w2f1s1l4cmb8-kirigami2-5.68.0/lib/qt-5.12.7/qml/org/kde/kirigami.2/styles/org.kde.desktop/ApplicationWindow.qml:12 Type Base.ApplicationWindow unavailable
file:///nix/store/8jhqa8q59r448c94yj88w2f1s1l4cmb8-kirigami2-5.68.0/lib/qt-5.12.7/qml/org/kde/kirigami.2/ApplicationWindow.qml:10 module "QtGraphicalEffects" is not installed
also, do you mind adding a link:
postInstall = ''
ln -s $out/bin/org.kde.keysmith $out/bin/keysmith
'';
deb7828
to
75c2ca5
Compare
I've wrapped the needed QT and QML ENV vars in postInstall, I think this should fix your issue (even though I couldn't reproduce it). |
75c2ca5
to
ab19002
Compare
I don't run a DE, so I do usually have issues with quite a few KDE / Qt / GTK apps |
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.
diff LGTM
gui launches fine
[2 built, 1 copied (0.5 MiB), 0.1 MiB DL]
https://github.com/NixOS/nixpkgs/pull/88614
1 package built:
keysmith
Motivation for this change
Needed a simple otp client, while otpgen is still in stand-by.
Things done
Successfully build, tested and added to all-packages.nix
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)