-
-
Notifications
You must be signed in to change notification settings - Fork 14.8k
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
pineflash: init at 0.5.4 #243312
pineflash: init at 0.5.4 #243312
Conversation
#227474 has to get merged before this one since blisp is needed to flash the pincil v2 |
blisp is now packaged. Does anything else need to be done before this PR is ready for merging? |
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.
My suggestions for the latest version.
Thanks for the review and the ping, I'll try to work on it this weekend :D |
I applied your suggested patches and building works fine, but executing |
188576b
to
6403e0f
Compare
Works fine for me on Arch Linux with home-manager in a Gnome wayland session. I don't have a NixOS system to test this on at the moment. Maybe @Luflosi can help? |
Hmm I'm on Nixos unstable with sway |
emilk/egui#3104 seems related to your sway problem. Pineflash 0.5.3 bumped egui from version 0.21 to 0.22. |
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.
How about moving this from pkgs/tools/misc/pineflash/default.nix
to pkgs/by-name/pi/pineflash/package.nix
? Then you don't need to change pkgs/top-level/all-packages.nix
.
I wonder whats different on your host. I tried bisecting egui earlier but to no avail all the hello_world examples even the one from the 0.21 release fail with:
There will be a pr with automated tooling to do this. No need for extra manual labor |
On sway, x86_64-linux. |
I tried this on NixOS unstable for my desktop (x86_64-linux) running KDE Plasma 5.27.10; I also got the |
6403e0f
to
b81fc4c
Compare
updated, but still getting the same error on wayland :( |
I actually got a different error this time, same system. I also forgot to mention in my previous comment that my desktop is using X11 and not Wayland
|
b81fc4c
to
5ad3090
Compare
I'm also getting
I'm running GNOME with Wayland. |
Hey I gave it a try once again after updating it and it's still erroring out on X11
|
I guess this is a upstream error and not one with our packaging but not sure. |
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.
Upstream reworked its wayland code.
|
||
rustPlatform.buildRustPackage rec { | ||
pname = "pineflash"; | ||
version = "0.5.4"; |
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.
version = "0.5.4"; | |
version = "0.5.5"; |
owner = "Spagett1"; | ||
repo = "PineFlash"; | ||
rev = version; | ||
hash = "sha256-4izfxApaCLbetZmMMKZIAknb+88B7z2WO+sBafehKHI="; |
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.
hash = "sha256-4izfxApaCLbetZmMMKZIAknb+88B7z2WO+sBafehKHI="; | |
hash = "sha256-4tcwEok36vuXbtlZNUkLNw1kHFQPBEJM/gWRhRWNLPg="; |
categories = [ "Utility" ]; | ||
})]; | ||
|
||
cargoHash = "sha256-VtUq/scRnx52ibqvUIqvaloaQpZyyXUlJYd2UIKQTjg="; |
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.
cargoHash = "sha256-VtUq/scRnx52ibqvUIqvaloaQpZyyXUlJYd2UIKQTjg="; | |
cargoHash = "sha256-l01It6mUflENlADW6PpOQvK1o4qOjbTsMLB6n+OIl0U="; |
--replace 'Command::new("dfu-util")' 'Command::new("${dfu-util}/bin/dfu-util")' \ | ||
--replace 'Command::new("blisp")' 'Command::new("${blisp}/bin/blisp")' | ||
''; | ||
|
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.
postFixup = '' | |
patchelf $out/bin/${pname} \ | |
--add-rpath ${lib.makeLibraryPath [libxkbcommon wayland]} | |
''; | |
With the new version I'm still getting the error :(
I get the same error when I rebase the PR on top of the version of Nixpkgs I'm currently running on my NixOS system (5c24cf2) I also tried rebasing on top of the latest master with the same result. |
Whoops. I just found this PR. The 0 exit status with no stack trace was in part an upstream bug, which should be fixed in the next version.
How should we go about merging these—can anyone test if either of these function for them currently? My PR doesn’t have the desktop icons currently, and has not been tested on Darwin, but functions for me with X11 Cinnamon on Note: The license is now out of date here, it was changed to gpl2Only. |
Description of changes
https;//github.com/spagett1/pineFlash/
Things done
sandbox = true
set innix.conf
? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)I have some problems with dfu-util when trying to flash a pinecil with pineflash it complaints about not finding dfu-util no matter iif I put it into buildInputs, nativeBuildInputs or propagatedBuildInputs what does work is spawning a nix-shell like that:
nix-shell -p dfu-util
and then opening pineflash from that nix-shell. I'm guessing this is caused by some wired behavior of $PATH but I don't know.