Skip to content

Conversation

@Austcool-Walker
Copy link

Added StarPlayrX snapshot purge command to help completely erase pesky snapshots fully on the newer betas including 11.0.1

@barrykn
Copy link
Owner

barrykn commented Nov 1, 2020

This is absolutely not the purpose of zap-snapshots.sh.

To avoid confusion, I think I'll rename zap-snapshots.sh to delete-old-snapshots.sh. Then I can integrate this as delete-all-snapshots.sh.

I'll deal with this later this weekend.

@Austcool-Walker
Copy link
Author

sounds good! 😄

@barrykn barrykn changed the base branch from main to dev-v0.5.0 November 1, 2020 03:57
@barrykn
Copy link
Owner

barrykn commented Nov 1, 2020

On second thought, what I might do instead is leave zap-snapshots.sh with its current name and current default behavior, but add a --all/-a command line option to make it do all snapshots like StarPlayrX's. I'll think about it more and then resolve it, one way or the other, later tonight (or possibly tomorrow, but probably tonight).

@barrykn
Copy link
Owner

barrykn commented Nov 2, 2020

Unfortunately, since my last comment some stuff in my life has started going crazy, so I'll have to release v0.5.0 without this and merge this into v0.5.1 when things settle down, maybe toward the end of the week or the weekend. Sorry.

@barrykn
Copy link
Owner

barrykn commented Nov 12, 2020

I'm trying to get v0.5.1 out before Big Sur itself, or at least not too long afterward, so I've run out of time to merge this for v0.5.1. I'll merge this for v0.5.2 or v0.5.3. Sorry.

@barrykn
Copy link
Owner

barrykn commented Nov 15, 2020

After seeing the latest changes, I'm a bit confused about what this code is trying to do (or, to be more clear, what advantages it has over my existing code). For the moment, I'll err on the side of not merging this in any form, while I work on higher priority fixes. (But this is not a final decision, so I'll leave the pull request open.)

@barrykn barrykn changed the base branch from dev-v0.5.0 to main November 15, 2020 02:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants