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
patchbay, patchwork: init with appimage-run #53156
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.
This is not how appimage-run is meant to be used, it's meant to be a quick escape hatch, like steam-run. appimage-run unnecessarily extracts into the users home directory.
I built an alternative at some point, I'll see if I can find and fix it pr'd as #54693. For the moment, I'd like to find a better option instead of merging this
The appimageTools attrset contains utilities to prevent the usage of appimage-run to package AppImages, like done/attempted in NixOS#49370 and NixOS#53156. This has the advantage of allowing for per-package environment changes, and extracts into the store instead of the users home directory. The package list was extracted into appimageTools to prevent duplication.
The appimageTools attrset contains utilities to prevent the usage of appimage-run to package AppImages, like done/attempted in NixOS#49370 and NixOS#53156. This has the advantage of allowing for per-package environment changes, and extracts into the store instead of the users home directory. The package list was extracted into appimageTools to prevent duplication.
We already have patchwork in master. |
Patchbay could still actually go in. Even so, I do agree with @tilpner, we should not run an appimage. |
@astro Do you plan to finish this? |
Would love to see patchbay as well as just patchwork, any one working on that? @ehmry? |
Can we close this PR? |
@veprbl Patchwork has landed in nixpkgs regardless of this PR. 🎆 |
Motivation for this change
Template:
standardnotes
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)