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

Ensure backwards-compatibility of bitstring lsb0 #2

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

micprog
Copy link
Member

@micprog micprog commented Jun 27, 2023

As some systems currently still have an older version of bitstring installed, this commit allows newer versions of dumpling to be backward-compatible with older bitstring versions.

@micprog micprog requested a review from meggiman June 27, 2023 12:04
@meggiman
Copy link
Contributor

Hi @micprog,
Thanks for the PR. In my opinion however, it would make much more sense to force the required bitstring version to be installed (by adding a version >= specified in setup.cfg) than adding patches to the source code. That way, installing or upgrading dumpling with pip would automatically force you to install the right version of bitstring. Or are there any reasons I might overlook to insist on a source code patch?

@micprog
Copy link
Member Author

micprog commented Jun 28, 2023

In principle, I definitely agree with your approach. I am not sure if python version support (specifically python 3.6) is guaranteed for the updated bitstring package, but enforcing an updated python version is also reasonable.

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