Pass through all options (that make sense) to wrapped borg commands #193

Closed
opened 2019-06-16 17:54:19 +00:00 by elho · 4 comments

There are a number of optinons the borg commands that are wrapped support, that are not passed though.
Some may not make sense, as being handled by the config, but those that do, should be possible to specify.

Examples would be --short' and '--last' for the --listcommand.borg listsupports them, butborgmatic --list` does not.

There are a number of optinons the borg commands that are wrapped support, that are not passed though. Some may not make sense, as being handled by the config, but those that do, should be possible to specify. Examples would be `--short' and '--last' for the `--list` command. `borg list` supports them, but `borgmatic --list` does not.
Owner

I've generally been implementing these on an as-needed / as-issue-filed basis rather than trying to go for a comprehensive full-coverage sweep. (See witten/borgmatic#14 for more info.)

Do you have more in mind besides --list's --short and --last?

I've generally been implementing these on an as-needed / as-issue-filed basis rather than trying to go for a comprehensive full-coverage sweep. (See https://projects.torsion.org/witten/borgmatic/issues/14 for more info.) Do you have more in mind besides `--list`'s `--short` and `--last`?
Author

Some in mind, others I will only find out over time using borgmatic more, but each time there would be the turnaround time between ticket and release and yet more till one's distro ships that release or the extra effort of bulding a newer package yoruself (I did that now to try borgmatic, but in the long run, it should just be usable with a stable distro release).

AS for list, there's --prefix and --format, for create there may be --noatime, which I didn't make my mind up over yet.

Some in mind, others I will only find out over time using borgmatic more, but each time there would be the turnaround time between ticket and release and yet more till one's distro ships that release or the extra effort of bulding a newer package yoruself (I did that now to try borgmatic, but in the long run, it should just be usable with a stable distro release). AS for `list`, there's `--prefix` and `--format`, for `create` there may be `--noatime`, which I didn't make my mind up over yet.
Author

Correction: While --noatime is indeed missing, it rather is something that should be a config option.

Correction: While `--noatime` is indeed missing, it rather is something that should be a config option.
Owner

Done and released with borgmatic 1.3.11. I implemented pass-throughs for all list-specific flags, all info-specific flags, and I added a couple of new config file options like atime in the location section. There still may be the odd flag here or there that needs coverage (especially with create), so please feel free to file follow-up tickets.

I did do some investment in making it easier to add these in the future. And I should note that having real sub-commands did make this go more smoothly!

Done and released with borgmatic 1.3.11. I implemented pass-throughs for all list-specific flags, all info-specific flags, and I added a couple of new config file options like `atime` in the location section. There still may be the odd flag here or there that needs coverage (especially with `create`), so please feel free to file follow-up tickets. I did do some investment in making it easier to add these in the future. And I should note that having real sub-commands did make this go more smoothly!
Sign in to join this conversation.
No Milestone
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: borgmatic-collective/borgmatic#193
No description provided.