ruamel.yaml dependency is too restrictive #783

Closed
opened 2023-11-07 14:44:12 +00:00 by kaechele · 3 comments
Contributor

What I'm trying to do and why

The ruamel.yaml dependency is currently pinned at >0.15.0,<0.18.0.
Fedora now started shipping ruamel.yaml 0.18.2, so we're getting broken dependencies.

Borgmatic seems to work and test fine with ruamel.yaml 0.18.5.

Steps to reproduce

No response

Actual behavior

No response

Expected behavior

No response

Other notes / implementation ideas

No response

borgmatic version

No response

borgmatic installation method

No response

Borg version

No response

Python version

No response

Database version (if applicable)

No response

Operating system and version

No response

### What I'm trying to do and why The ruamel.yaml dependency is currently pinned at >0.15.0,<0.18.0. Fedora now started shipping ruamel.yaml 0.18.2, so we're getting broken dependencies. Borgmatic seems to work and test fine with ruamel.yaml 0.18.5. ### Steps to reproduce _No response_ ### Actual behavior _No response_ ### Expected behavior _No response_ ### Other notes / implementation ideas _No response_ ### borgmatic version _No response_ ### borgmatic installation method _No response_ ### Borg version _No response_ ### Python version _No response_ ### Database version (if applicable) _No response_ ### Operating system and version _No response_
Owner

Thanks for bringing this to my attention! Newer versions of ruamel.yaml (0.18.x) actually break both config generation and end-to-end tests. So this is unfortunately not as straightforward as relaxing the version requirement. I'll see what can be done though.

Thanks for bringing this to my attention! Newer versions of ruamel.yaml (0.18.x) actually break both config generation and end-to-end tests. So this is unfortunately not as straightforward as relaxing the version requirement. I'll see what can be done though.
Owner

This has been implemented in main and will be part of the next release!

This has been implemented in main and will be part of the next release!
witten referenced this issue from a commit 2023-11-07 18:09:43 +00:00
Owner

Just released with borgmatic 1.8.5!

Just released with borgmatic 1.8.5!
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#783
No description provided.