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
qjoypad: 4.1.0 -> 4.3.0 #74330
qjoypad: 4.1.0 -> 4.3.0 #74330
Conversation
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.
running it as is creates:
[nix-shell:/home/jon/.cache/nix-review/pr-74330-1]$ ./results/qjoypad/bin/qjoypad
qt.qpa.plugin: Could not find the Qt platform plugin "xcb" in ""
This application failed to start because no Qt platform plugin could be initialized. Reinstalling the application may fix this problem.
Aborted (core dumped)
please read: https://nixos.org/nixpkgs/manual/#sec-language-qt
dont forget to squash the commits
|
c8889d1
to
9cac0e4
Compare
Now when I run nix-build, I get this error: |
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 squash all the commits to adhere to contributing.md
git reset HEAD~4
git add pkgs/
git commit --amend --no-edit
git push <fork> <branch> --force
6cbb4c9
to
cb8d1d2
Compare
Done, thank you for your help ! |
please resolve conflict, otherwise it LGTM |
ping @radraw for resolving merge conflict |
I see the change hasn't been picked yet. Is something missing I can help to proceed with it ? |
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.
to have clean git histories, please rebase on top of the target branch:
git pull -r origin master
instead of merging upstream into your branch :)
should just be 1 commit
pkgs/tools/misc/qjoypad/default.nix
Outdated
license = stdenv.lib.licenses.gpl2; | ||
maintainers = with stdenv.lib.maintainers; [ astsmtl ]; | ||
platforms = with stdenv.lib.platforms; linux; | ||
homepage = https://github.com/panzi/qjoypad; |
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.
rfc 45
homepage = https://github.com/panzi/qjoypad; | |
homepage = "https://github.com/panzi/qjoypad"; |
2cfb336
to
6fc6ef3
Compare
I marked this as stale due to inactivity. → More info |
Co-authored-by: Sandro <sandro.jaeckel@gmail.com>
Motivation for this change
Switching to Qt5 makes the package play well with modern desktop environments. Tested on Ubuntu 18.04, default Ubuntu session.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @astsmtl