--repository option for all subcommands #564
Labels
No Label
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 New Issue
No description provided.
Delete Branch "%!s(<nil>)"
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?
--repository
option 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
--repository
flag 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 umount
only operates on a mounted path. Additionally, I think it will help discoverability if--repository
shows up in the help for individual actions.Here are the actions that need
--repository
support added as part of this ticket:rcreate
(formerlyinit
)prune
compact
create
check
Status-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.
2a36a2a312
suggests 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 list
which should get--repository
tooAlready ok
Looks like those already have
--repository
flags 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!