Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Maximum supported msgpack requirement -- still valid today? #8144

Closed
he32 opened this issue Mar 12, 2024 · 3 comments
Closed

Maximum supported msgpack requirement -- still valid today? #8144

he32 opened this issue Mar 12, 2024 · 3 comments
Labels

Comments

@he32
Copy link

he32 commented Mar 12, 2024

Have you checked borgbackup docs, FAQ, and open GitHub issues?

Yes

Is this a BUG / ISSUE report or a QUESTION?

Question

System information. For client/server mode post info for both machines.

Server: Linux
Client: NetBSD/i386 9.3 w/python 3.11

Your borg version (borg -V).

borg 1.2.7

Full borg commandline that lead to the problem (leave away excludes and passwords)

I run borg via borgmatic, with
borgmatic --verbosity 1 --syslog-verbosity -2 --list --stats

Describe the problem you're observing.

I'm installing borg via NetBSD's packaging system, pkgsrc.
In the most recent version, this gets me py311-msgpack-1.0.8.
This causes borg to refuse to run with:

backupserver: Error running actions for repository
You do not have a supported version of the msgpack python package installed. Terminating.
This should never happen as specific, supported versions are required by our setup.py.

Yes, I know, it says in the following "do not contact borgbackup support about this". But...

I have read both #3753 and 39761eb

So... borg insists that msgpack version 1.0.7 is the maximum supported version. The issue referenced above appears to mostly be about the era of python 3.4 / 3.5, and msgpack versions 0.5.*. Is this still relevant to the current versions of msgpack? Leaving such a requirement that another package "must not be newer than this particular version" will tend to break normal package upgrade procedures. And the "allow 1.0.7" commit was a little short on details of why that is now OK, but still left the "must not be newer than this particular version (1.0.7)" requirement in place.

I tried duplicating the "allow msgpack 1.0.7" to also allow 1.0.8, and the resulting borg client appears to work nicely. Yes, I know, I have voided my warranty...

@ThomasWaldmann
Copy link
Member

It is the responsibility of the dist package maintainer(s) to not break the requirements of packages, so you need to contact the netbsd packagers of borg and msgpack about this.

@ThomasWaldmann
Copy link
Member

The reason why it is that strict is because there have been broken msgpack releases in the past. So I want to first test new releases before they are "officially accepted" by borg.

Specifically about msgpack 1.0.8: it seems to work ok in my first tests, so I already added a commit allowing it in next borg release. Platform testing of that is still to do, but that is part of the usual release procedure, so anything strange would be noticed then.

@ThomasWaldmann
Copy link
Member

ThomasWaldmann commented Mar 30, 2024

borg 1.2.8 was released, accepting msgpack 1.0.8.

Upcoming borg 1.4.0b2 will also accept msgpack 1.0.8.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants