-
-
Notifications
You must be signed in to change notification settings - Fork 216
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
latest version of git breaks setuptools_scm with python3.9 #1038
Comments
Thanks for the note We need a bug fix there |
Merged
I confirm the issue |
This was referenced May 12, 2024
Closed
bpkroth
added a commit
to bpkroth/MLOS
that referenced
this issue
May 20, 2024
motus
added a commit
to microsoft/MLOS
that referenced
this issue
May 21, 2024
A new git version outputs a different date format that older versions of python's datetime module don't recognize. pypa/setuptools-scm#1038 setuptools-scm cut a new version to address that, but it's not available in conda main channel yet, which is required, since without that the conda pip phase can't execute in a single transaction For now, we install things via conda-forge and adjust the channel priority order so that the full set of dependencies could be resolved. That of course brought in additional changes (e.g., `python=3.12` by default, new `pylint`, `pycodestyle`, etc.), so this now also includes some additional linting changes. However, longer term, we need to switch to a pyproject.toml file to fix this properly. There we can specify prereqs for even loading the setup.py as well as fix some other config complexities, though that is a broader change. --------- Co-authored-by: Sergiy Matusevych <sergiym@microsoft.com>
webknjaz
added a commit
to webknjaz/pylibssh
that referenced
this issue
Jun 4, 2024
This is necessary to let it deal with the datetime output format that versions of Git 2.45.0 and above now produce. Refs: * pypa/setuptools-scm#1038 * pypa/setuptools-scm#1039
13 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
When trying to pip install a package with python 3.9 we get the following exception:
I think I've tracked the issue down to the version of git being used, and the way it formats the date string with
log --format=%cI
. With the new version:Note that the output uses the 'Z' suffix, which is standard ISO format but was not supported in older versions (e.g. 3.9) of the datetime module.
The previous version of git outputs the ISO datetime string in a format that older datetime can handle:
So things are "fixed" with later versions of python that can handle regular ISO formats. Maybe a restriction on supported git versions with python 3.9 could be expressed somehow?
The text was updated successfully, but these errors were encountered: