unsnap: Add at 2023.03.11#3208
Conversation
|
Is the patch for the README really necessary, as it doesn't seem to be included in the final package? |
|
Works for me per 17ce51d |
48a8876 to
a267606
Compare
|
I have included additional patches to improve the user experience. In particular,
No, but it is part of the patchset I've rebased from an open PR. I'm hoping to upstream it at some point. |
|
I've made sure that it works with the wrapper script from #3211. Should be good to go now! |
ermo
left a comment
There was a problem hiding this comment.
I was confused when just running sudo unsnap, because there is no help output and the script didn't give me a message at the end telling me how to actually effectuate the changes?
Suggestion: Consider making sudo unsnap print out a message telling the user to:
- check out the help page
- use
sudo unsnap autoto actually remove snaps and snapd
**Summary** Initial inclusion of `unsnap` at the last upstream commit. It includes a patchset that adds support for `eopkg` and our snap wrapper.
|
@ermo: I added a message with clarification. Let me know what you think! |
Summary
Initial inclusion of
unsnapat the last upstream commit. It includes a patchset that adds support foreopkg.Resolves #323
Test Plan
Run
sudo unsnap autoto automatically migrate snaps to flatpak.Checklist