-
Notifications
You must be signed in to change notification settings - Fork 5
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
versioningit use makes packaging impossible #25
Comments
Oh, that's a shame. I'm not 100% clear on why we added it or what it does for us but I'll look into removing it. Cc @sarnold who added it in case he has an opinion about this. |
@mgorny I read the issue you referenced but it doesn't seem to have anything to do with GitHub archives. Can you elaborate a bit more? Thanks! |
Perhaps the bug is more focused on the "usual" solution that the problem itself. Versioningit doesn't work with GitHub archives:
This is normal because there's no way it can detect the version from that. Other packages such as |
Switch from the versioningit package to more commonly used setuptools-scm. It serves the same purpose but it is more mature and it supports building from GitHub archives. Fixes PennyDreadfulMTG#25
Thanks! |
The new 0.6.4 release started using versioningit. Unfortunately, this makes packaging hard to impossible for Linux distributions since versioningit refuses to support using GitHub archives (see jwodder/versioningit#45), and this is the only source we can use that includes the test suite.
Could you please consider switching to a more portable alternative such as setuptools-scm?
The text was updated successfully, but these errors were encountered: