elho
  • Joined on Jun 16, 2019
Loading Heatmap…

elho commented on issue witten/borgmatic#195

Use sub-commands instead of options for actions

`create --list` was just an example to make clear how to split and how having actions as positional arguments without dashes is unambiguous even if there is actions with the same names as options.

8 months ago

elho commented on issue witten/borgmatic#193

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

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

8 months ago

elho commented on issue witten/borgmatic#193

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

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).

8 months ago

elho commented on issue witten/borgmatic#191

Configuration option to control colored output

Strong personal taste, I guess. :smile: The higher contrast of black and white messages is more helpful to me to grasp them, than some color indicating the type of message, when it starting with e.g. "Error:" is pretty clear already. :wink:

8 months ago

elho commented on issue witten/borgmatic#192

Separate before/after backup run and individual archive hooks

No, by "archive" I do mean borg archive, its the exact opposite situation. :wink:

8 months ago

elho commented on issue witten/borgmatic#195

Use sub-commands instead of options for actions

Spontaneous, slightly hackish idea: Do a `argparse` parser with the common options and do `add_argument('subcmds', metavar='COMMAND', nargs='+')` then you split the`subcmds` whenever there's an item not starting with a dash, ie. `['create', '--list', '--stats', 'check', 'list']` becomes `['create', '--list', '--stats'], ['check'], ['list']` and you then invoke the according separate `argparse` parsers (that otherwise would be sub-parsers) on each of the split lists.

8 months ago

elho commented on issue witten/borgmatic#195

Use sub-commands instead of options for actions

Personally, I'd work around the `argparse` limitation 1 by making a `backup` sub-command that invokes the 3 others.

8 months ago

elho opened issue witten/borgmatic#196

Poor error reporting for bad YAML includes

8 months ago

elho opened issue witten/borgmatic#195

Use sub-commands instead of options for actions

8 months ago

elho opened issue witten/borgmatic#194

Finer grained verbosity levels

8 months ago

elho opened issue witten/borgmatic#193

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

8 months ago

elho opened issue witten/borgmatic#192

Separate before/after backup run and individual archive hooks

8 months ago

elho opened issue witten/borgmatic#191

Configuration option to control colored output

8 months ago