--repository option for all subcommands #564
Labels
No labels
blocked
breaking
bug
data loss
design finalized
good first issue
new feature area
question / support
security
waiting for response
No milestone
No assignees
3 participants
Notifications
Due date
No due date set.
Dependencies
No dependencies set
Reference
borgmatic-collective/borgmatic#564
Loadingโฆ
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
--repositoryoption should be directly under the borgmatic command, and not under specific subcommands.It would allow to trigger any subcommand only on a specific repository.
I've decided to add the
--repositoryflag to individual borgmatic actions rather than as a global flag, because certain actions by their nature don't operate on a repository. For instance,borgmatic umountonly operates on a mounted path. Additionally, I think it will help discoverability if--repositoryshows up in the help for individual actions.Here are the actions that need
--repositorysupport added as part of this ticket:rcreate(formerlyinit)prunecompactcreatecheckStatus-check: Is rcreate already done?
I see it mentioned in the docs https://torsion.org/borgmatic/docs/reference/command-line/
Is it that it needs handling when there is no config file with the repository in it?
I've been reading through the code figuring out an implementation.
2a36a2a312suggests it's done for rcreate.I can model the rest on it. (WIP)
Yeah, it's done for
rcreate. This ticket would be adding it for all actions (that it makes sense on).Implemented by @nain in #652. Will be part of the next release!
thanks !
hey, I forgot to talk about
borgmatic listwhich should get--repositorytooAlready ok
Looks like those already have
--repositoryflags according to--help...? Some of them have gotten added piecemeal over time. But please let us know if we missed anything else.Yes, it does, tested and approved :)
Just released in borgmatic 1.7.9!