#120 Create borgmatic.d/ during default installation

Open
opened 1 month ago by patrick_stronn · 3 comments

I just realized we could do this, but if it were created during installation, I’d found it long ago!

Why not creating it by default?

I just realized we could do this, but if it were created during installation, I'd found it long ago! Why not creating it by default?
witten commented 1 month ago
Owner

Probably no good reason.. My guess is that it’s historical.. generate-borgmatic-config was written before the borgmatic.d feature existed.

Good idea!

Probably no good reason.. My guess is that it's historical.. `generate-borgmatic-config` was written before the `borgmatic.d` feature existed. Good idea!
witten added the
design finalized
label 1 week ago
nicoulaj commented 1 week ago

Just my two cents, what is the purpose of having both /etc/borgmatic and /etc/borgmatic.d ? Wouldn’t it be simpler to completely remove /etc/borgmatic and just keep /etc/borgmatic.d ?

Just my two cents, what is the purpose of having both `/etc/borgmatic` and `/etc/borgmatic.d` ? Wouldn't it be simpler to completely remove `/etc/borgmatic` and just keep `/etc/borgmatic.d` ?
witten commented 1 week ago
Owner

While you’re entirely correct that /etc/borgmatic.d/ could handle both cases, /etc/borgmatic/ is more for the “common case” of just having a single config file. Slightly nicer user experience, I guess.

Also, the code that supports both /etc/borgmatic/ and /etc/borgmatic.d/ has to be there anyway, since the user can provide multiple custom config file paths on the command-line with the --config flag.

While you're entirely correct that `/etc/borgmatic.d/` could handle both cases, `/etc/borgmatic/` is more for the "common case" of just having a single config file. Slightly nicer user experience, I guess. Also, the code that supports both `/etc/borgmatic/` and `/etc/borgmatic.d/` has to be there anyway, since the user can provide multiple custom config file paths on the command-line with the `--config` flag.
Sign in to join this conversation.
No Milestone
No Assignees
3 Participants
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
Cancel
Save
There is no content yet.