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

setup.py: relax pyparsing version restriction #69

Merged
merged 1 commit into from
Apr 14, 2023

Conversation

arkamar
Copy link
Contributor

@arkamar arkamar commented Mar 2, 2023

The need to restrict pyparsing to <3.0 seems to be unnecessary anymore. It is from the time when pyparsing-3 was pre-release candidate which was causing build failures. Original commit [1] locked pyparsing on specific version which was later relaxed to <3.0 [2]. pyparsing is currently in version 3.0.9 and it works fine with configshell-fb.

[1] ec7ca0e ("setup.py: lets stick to pyparsing v2.4.7")
[2] f1fd5a9 ("setup.py: specify a version range for pyparsing")

The need to restrict pyparsing to <3.0 seems to be unnecessary anymore.
It is from the time when pyparsing-3 was pre-release candidate which was
causing build failures. Original commit [1] locked pyparsing on specific
version which was later relaxed to <3.0 [2]. pyparsing is currently in
version 3.0.9 and it works fine with configshell-fb.

[1] ec7ca0e ("setup.py: lets stick to pyparsing v2.4.7")
[2] f1fd5a9 ("setup.py: specify a version range for pyparsing")
arkamar added a commit to arkamar/gentoo that referenced this pull request Mar 2, 2023
arkamar added a commit to arkamar/gentoo that referenced this pull request Mar 2, 2023
gentoo-bot pushed a commit to gentoo/gentoo that referenced this pull request Mar 2, 2023
Bug: https://bugs.gentoo.org/896764
Upstream-PR: open-iscsi/configshell-fb#69
Signed-off-by: Petr Vaněk <arkamar@atlas.cz>
Closes: #29893
Signed-off-by: Michał Górny <mgorny@gentoo.org>
@maurizio-lombardi maurizio-lombardi merged commit 195307c into open-iscsi:master Apr 14, 2023
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.

None yet

2 participants