Borg patterns is broken #977

Closed
opened 2025-01-23 16:49:12 +00:00 by pasha132 · 3 comments
Contributor

What I'm trying to do and why

Hello!

After update to 1.9.6 it is unable to use borg patterns. Whatever you write in appropriate sections will be prepended with 'fn' prefix.

Possible solution is in PR #976

Steps to reproduce

Write something like in config:

exclude_patterns:
    - 'sh:**/directory'

And execute borgmatic in dry-run, for example. You will see 'fn:sh:**/directory' in logs.

Actual behavior

No response

Expected behavior

Exact pattern templates should not be changed.

Other notes / implementation ideas

No response

borgmatic version

1.9.6

borgmatic installation method

Gentoo portage

Borg version

borg 1.4.0

Python version

Python 3.12.8

Database version (if applicable)

No response

Operating system and version

No response

### What I'm trying to do and why Hello! After update to 1.9.6 it is unable to use borg patterns. Whatever you write in appropriate sections will be prepended with 'fn' prefix. Possible solution is in PR #976 ### Steps to reproduce Write something like in config: ``` exclude_patterns: - 'sh:**/directory' ``` And execute borgmatic in dry-run, for example. You will see `'fn:sh:**/directory'` in logs. ### Actual behavior _No response_ ### Expected behavior Exact pattern templates should not be changed. ### Other notes / implementation ideas _No response_ ### borgmatic version 1.9.6 ### borgmatic installation method Gentoo portage ### Borg version borg 1.4.0 ### Python version Python 3.12.8 ### Database version (if applicable) _No response_ ### Operating system and version _No response_
Owner

Thank you so much for finding and fixing this! I think I just totally spaced on the fact that exclude_patterns can actually be patterns and aren't just plain ol' paths. Anyway, my comments are on the PR.

Thank you so much for finding and fixing this! I think I just totally spaced on the fact that `exclude_patterns` can actually be patterns and aren't just plain ol' paths. Anyway, my comments are on the PR.
witten added the bug label 2025-01-23 18:35:36 +00:00
Owner

PR merged! I'll put out a release with this shortly.

PR merged! I'll put out a release with this shortly.
witten referenced this issue from a commit 2025-01-23 20:13:59 +00:00
Owner

Released in borgmatic 1.9.7!

Released in borgmatic 1.9.7!
Sign in to join this conversation.
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: borgmatic-collective/borgmatic#977