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

Port _SixMetaPathImporter to Python 3.10 #343

Merged
merged 2 commits into from
Jan 3, 2021
Merged

Port _SixMetaPathImporter to Python 3.10 #343

merged 2 commits into from
Jan 3, 2021

Conversation

vstinner
Copy link
Contributor

Fixes #341.

@vstinner
Copy link
Contributor Author

@frenzymadness
Copy link

The failure in CI is caused by the typing module which is not compatible with Python 3.3 but all other tests look good. I can also confirm that this patch works well with Python 3.10 and fixes all tests previously failing due to the warning. We are already testing this patch in Fedora and I can confirm that more than a hundred packages are okay with it with Python 3.9.

@vstinner
Copy link
Contributor Author

cc @jaraco @warsaw

@jaraco
Copy link
Contributor

jaraco commented Dec 14, 2020

LGTM, but I don't have a lot of context around it.

@brettcannon
Copy link
Contributor

Because this is such a weird loader the patch LGTM as a way to make it function appropriately.

@frenzymadness
Copy link

I can confirm that more than 5 hundred RPM packages build fine with six with this patch.

@vstinner
Copy link
Contributor Author

vstinner commented Jan 3, 2021

@benjaminp: Thanks for the update. Tests still pass ;-)

@benjaminp benjaminp merged commit e51553a into benjaminp:master Jan 3, 2021
@vstinner
Copy link
Contributor Author

vstinner commented Jan 3, 2021

Thanks for the merge :-)

@vstinner vstinner deleted the importer branch January 3, 2021 17:11
benjaminp added a commit that referenced this pull request Apr 4, 2021
taratatach added a commit to cozy/cozy.github.io that referenced this pull request Mar 12, 2024
  `mkdocs` v1.2.0 makes use of `ghp_import` v2.1.0 which in turn makes
  use of `dateutil` which makes use of the `six.moves` module.
  The way this module is generated in `six` v1.11.0 is not compatible
  with Python 3.10+.

  This gets fixed in v1.16.0 (see
  benjaminp/six#343).
taratatach added a commit to cozy/cozy.github.io that referenced this pull request Mar 13, 2024
  `mkdocs` v1.2.0 makes use of `ghp_import` v2.1.0 which in turn makes
  use of `dateutil` which makes use of the `six.moves` module.
  The way this module is generated in `six` v1.11.0 is not compatible
  with Python 3.10+.

  This gets fixed in v1.16.0 (see
  benjaminp/six#343).
taratatach added a commit to cozy/cozy.github.io that referenced this pull request Mar 13, 2024
  `mkdocs` v1.2.0 makes use of `ghp_import` v2.1.0 which in turn makes
  use of `dateutil` which makes use of the `six.moves` module.
  The way this module is generated in `six` v1.11.0 is not compatible
  with Python 3.10+.

  This gets fixed in v1.16.0 (see
  benjaminp/six#343).
taratatach added a commit to cozy/cozy.github.io that referenced this pull request Mar 13, 2024
  `mkdocs` v1.2.0 makes use of `ghp_import` v2.1.0 which in turn makes
  use of `dateutil` which makes use of the `six.moves` module.
  The way this module is generated in `six` v1.11.0 is not compatible
  with Python 3.10+.

  This gets fixed in v1.16.0 (see
  benjaminp/six#343).
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.

Implement create_module() and exec_module() for _SixMetaPathImporter
5 participants