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 dependencies are outdated #27435

Closed
rasebo opened this issue Sep 20, 2023 · 4 comments
Closed

setup.py dependencies are outdated #27435

rasebo opened this issue Sep 20, 2023 · 4 comments
Assignees
Labels
Azure CLI Team The command of the issue is owned by Azure CLI team customer-reported Issues that are reported by GitHub users external to the Azure organization. Packaging question The issue doesn't require a change to the product in order to be resolved. Most issues start as that
Milestone

Comments

@rasebo
Copy link

rasebo commented Sep 20, 2023

Describe the bug

The dependencies in src/azure-cli/setup.py are outdated and not consistent with those defined everywhere else.

The python package that is built (and gets published on pypi.org) as a result is advertising the wrong requirements, generating the same error that was supposed to be fixed with #20041 (and possibly other issues as well).

Related command

n/a

Errors

error in PyGithub setup command: use_2to3 is invalid.

Issue script & Debug output

n/a

Expected behavior

DEPENDENCIES should contain the same dependencies as the requirements.py3.*.txt files from the same directory.

Environment Summary

n/a

Additional context

No response

@rasebo rasebo added the bug This issue requires a change to an existing behavior in the product in order to be resolved. label Sep 20, 2023
@microsoft-github-policy-service microsoft-github-policy-service bot added the customer-reported Issues that are reported by GitHub users external to the Azure organization. label Sep 20, 2023
@yonzhan
Copy link
Collaborator

yonzhan commented Sep 20, 2023

Thank you for opening this issue, we will look into it.

@yonzhan yonzhan added Packaging Azure CLI Team The command of the issue is owned by Azure CLI team question The issue doesn't require a change to the product in order to be resolved. Most issues start as that and removed bug This issue requires a change to an existing behavior in the product in order to be resolved. labels Sep 20, 2023
@yonzhan yonzhan added this to the Backlog milestone Sep 20, 2023
@rasebo rasebo changed the title setup.py dependencies and classifiers are outdated setup.py dependencies are outdated Sep 20, 2023
@bebound
Copy link
Contributor

bebound commented Sep 21, 2023

May I know the python and pip version you are using?
PyGithub~=1.38 means PyGithub>=1.38 and PyGithub==1.*, running pip install PyGithub~=1.38 should install PyGithub-1.59.1 without any error.

Ref: https://peps.python.org/pep-0440/#compatible-release

@rasebo
Copy link
Author

rasebo commented Sep 21, 2023

Right, sorry for that. You are right, it shouldn't be an issue.

I noticed that we did recently change the Docker image we're using from python:3.10.4-slim to python:3.11.4-slim. The current one is running Python 3.11.4 and pip 23.1.2.

While unexpected, looks like Python 3.11 doesn't seem to be supported yet by azure-cli: #24494. The classifiers in setup.py do explicitly state that.

@rasebo
Copy link
Author

rasebo commented Sep 21, 2023

Closing, discovered it was caused by the pip resolver going all the way down to 1.38 due to other dependencies inherited by packages in our requirements.txt. Adding PyGithub~=1.44.1 to the file stops it from going all the way down to the incompatible version.

@rasebo rasebo closed this as completed Sep 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Azure CLI Team The command of the issue is owned by Azure CLI team customer-reported Issues that are reported by GitHub users external to the Azure organization. Packaging question The issue doesn't require a change to the product in order to be resolved. Most issues start as that
Projects
None yet
Development

No branches or pull requests

3 participants