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

smmap 5.0.0 support #76

Closed
DmitryKuzmenko opened this issue Oct 20, 2021 · 4 comments
Closed

smmap 5.0.0 support #76

DmitryKuzmenko opened this issue Oct 20, 2021 · 4 comments
Labels

Comments

@DmitryKuzmenko
Copy link

DmitryKuzmenko commented Oct 20, 2021

Is it possible to bump the smmap version to 5.0.0?
It is the current version in ArchLinux repository and this is breaking packages depending on gitdb like git-up.
I tried to bump the smmap version. Tests passed with smmap-5.0.0 on ArchLinux.

@Byron
Copy link
Member

Byron commented Oct 20, 2021

Sorry for the hassle. It looks like pypi already has 5.0, but due to me not having pushed the changes these were not present here.

Now master has the version change, does that help?

@Byron
Copy link
Member

Byron commented Oct 20, 2021

Oh, we are in gitdb, never mind 😅. This means you would also need a gitdb version bump, right?

Also it's strange that a major version bump is able to break anything, as it currently is not used by anybody. The requirements here say smmap>=3.0.1,<5, so it shouldn't pick it up.

Maybe you can help shed some light on what happens there.

@Byron Byron added the feedback label Oct 20, 2021
@DmitryKuzmenko
Copy link
Author

DmitryKuzmenko commented Oct 22, 2021

@Byron This is my point. Requirements of gitdb is locking smmap version as smmap>=3.0.1,<5. Meanwhile smmap-5.0.0 is released and included to the current Arch packages. That is making gitdb failing on my system. I ran the gitdb unit tests with smmap-5.0.0 and tests passed. What I'm asking is to update the requirements.txt here as smmap>=3.0.1,<=5 or whatever else supporting 5.0 release.

Byron added a commit that referenced this issue Oct 23, 2021
@Byron
Copy link
Member

Byron commented Oct 23, 2021

I am happy to do that! This also means the bug was introduced in the arch package manager which seems to pair incompatible crates while not trying to rebuild all dependencies with the new version, potentially causing widespread breakage, not only with GitPython.

v4.0.8 was released, avoiding to major bump this one to avoid similar issues to arise on arch. By now I feel there is no way to do it right.

I am closing this issue as new version constraints were set to include smmap 5.0, and also drop support for py3.4 and 3.5 to match smmap.

@Byron Byron closed this as completed Oct 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

No branches or pull requests

2 participants