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
appimage: replace radare2+jq with readelf+awk+sha256sum #89122
Conversation
6d27b3a
to
cf96bad
Compare
cf96bad
to
c7dfd31
Compare
I tried to build some appimage packages from your code and I get I'm just curious about the problem your solving, when would the existing radare based code fail? Would it happen from spaces in the name of the appimage or package name or something else? |
$ appimage-run "Any Filename With Spaces.AppImage" |
Is the issue fixed when you replace |
Yup works perfectly now. And I tested all of my appimage packages and they work perfectly. The PR is perfect with the change. |
c7dfd31
to
56ec381
Compare
Changed |
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 very much prefer that. radare2 has quite frequent releases, which I need to test and sometimes break the API.
56ec381
to
46a8d42
Compare
radare2 does not play nicely with filenames containing spaces radareorg/radare2#16958
46a8d42
to
9444756
Compare
Motivation for this change
radare2 does not play nicely with filenames containing spaces: radareorg/radare2#16958
so I replaced radare2+jq with readelf+awk+sha256sum
I also cleaned up default.nix with nixpkgs-fmt and removed bashisms from appimage-exec.sh
Things done
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)