Skip to content

feat: add exclude-if-xattr option#1718

Merged
aawsome merged 3 commits intorustic-rs:mainfrom
hacker:feat/exclude-if-xattr
Mar 31, 2026
Merged

feat: add exclude-if-xattr option#1718
aawsome merged 3 commits intorustic-rs:mainfrom
hacker:feat/exclude-if-xattr

Conversation

@hacker
Copy link
Copy Markdown
Contributor

@hacker hacker commented Mar 28, 2026

This would be a rustic counterpart of rustic-rs/rustic_core#491

@hacker
Copy link
Copy Markdown
Contributor Author

hacker commented Mar 28, 2026

I'm not entirely sure what your workflow is, as this PR requires its counterpart in rustic_core for tests to pass and so does the rustic_core PR — tests in rustic would fail if core is merged without this PR.

At any rate, just explaining why I made this PR draft. (and still don't know if it was a good idea:)).

@hacker
Copy link
Copy Markdown
Contributor Author

hacker commented Mar 30, 2026

Now that you've merged the core counterpart, I'll make this one ready for review, but let me know if you want me to update .lock or your bot will take care of that.

@hacker hacker marked this pull request as ready for review March 30, 2026 12:08
@aawsome
Copy link
Copy Markdown
Member

aawsome commented Mar 30, 2026

Now that you've merged the core counterpart, I'll make this one ready for review, but let me know if you want me to update .lock or your bot will take care of that.

Can you please also update the Cargo.lock file - else the unit tests will fail.
(There might be some recent security issues in the pipeline, too. You can either also try to fix them or I will make a separate PR for that. Could be also that these will need dependency updates within rustic_core to get fixed, though)

@hacker
Copy link
Copy Markdown
Contributor Author

hacker commented Mar 30, 2026

Hey, I did update core, but I'm still confused with the whole dependency system and what depends on what, but I do suspect that security issues are mostly with core… Anyway, I don't trust myself to take care of them yet ;-)

Copy link
Copy Markdown
Member

@aawsome aawsome left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thanks @hacker

@aawsome aawsome merged commit 3c84607 into rustic-rs:main Mar 31, 2026
29 of 30 checks passed
github-merge-queue bot pushed a commit that referenced this pull request Apr 5, 2026
## 🤖 New release

* `rustic-rs`: 0.11.1 -> 0.11.2 (✓ API compatible changes)

<details><summary><i><b>Changelog</b></i></summary><p>

<blockquote>

##
[0.11.2](v0.11.1...v0.11.2)
- 2026-04-05

### Added

- add exclude-if-xattr option
([#1718](#1718))

### Fixed

- fix some typos
([#1710](#1710))
- Add keep-{within-}hourly to config/full.toml
([#1704](#1704))

### Other

- update dependencies
([#1723](#1723))
- *(deps)* unpin and update self_update
([#1699](#1699))
- move to axum for webdav command
([#1720](#1720))
- update dependencies
([#1721](#1721))
- *(deps)* bump quinn-proto from 0.11.13 to 0.11.14
([#1703](#1703))
- Update core ([#1698](#1698))
</blockquote>


</p></details>

---
This PR was generated with
[release-plz](https://github.com/release-plz/release-plz/).

Co-authored-by: rustic-release-plz[bot] <182542030+rustic-release-plz[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants