Skip to content

Conversation

@m4dcoder
Copy link
Contributor

@m4dcoder m4dcoder commented Jan 20, 2020

The make .st2client-install-check command is failing because latest more-itertools no longer supports python 2.7. To workaround, pin more-itertools to an older version. The zipp requirement is pinned to >=0.5,<=1.0.0 because the latest 2.0.0 version is registered as 0.0.0 and causing install check to fail.

Closes #4843

@pull-request-size pull-request-size bot added the size/XS PR that changes 0-9 lines. Quick fix/merge. label Jan 20, 2020
@m4dcoder m4dcoder requested a review from Kami January 20, 2020 20:05
@m4dcoder m4dcoder added this to the 3.2.0 milestone Jan 20, 2020
@m4dcoder m4dcoder requested a review from arm4b January 20, 2020 20:06
@arm4b
Copy link
Member

arm4b commented Jan 20, 2020

Related #4843

Looks like Travis is still not happy about some of st2client dependencies, according to https://travis-ci.org/StackStorm/st2/jobs/639637822

@m4dcoder
Copy link
Contributor Author

@armab This is a new and different error.

Copy link
Member

@arm4b arm4b left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+100, Thanks!

The make .st2client-install-check command is failing because latest more-itertools no longer supports python 2.7. To workaround, pin more-itertools to an older version.
When running st2client install check, the requirement zipp>=0.5 will install zipp 2.0.0 which will registered as 0.0.0 that fails the st2client install check. This is likely an upstream error. The zipp requirement is pinned to >=0.5,<=1.0.0 to workaround the issue.
Copy link
Member

@arm4b arm4b left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would be good to include Changelog record for that fix as well.

@m4dcoder m4dcoder merged commit ae60f0b into master Jan 21, 2020
@m4dcoder m4dcoder deleted the fix-more-itertools branch January 21, 2020 00:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/XS PR that changes 0-9 lines. Quick fix/merge.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants