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

Avoid undocumented pypa/wheel API in dist_info #3905

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

abravalheri
Copy link
Contributor

@abravalheri abravalheri commented Apr 24, 2023

Summary of changes

Avoid pypa/wheel API in dist_info

  • Instead of using bdist_wheel to convert egg-info directory into dist-info, just do the conversion directly.
    This relies on the fact that PKG-INFO and METADATA files are compatible after the last PRs.

This is done because there is an understanding that pypa/wheel is not meant to be used public yet and will change in the future.


This is part of a series of PRs:

The motivation for this series of PRs is the following:

  • Logic for generating .egg-info and .dist-info directories is intertwined and implicit
    (See Remove dependency on wheel #1386).
  • Setuptools uses pypa/wheel API which is not stable yet and is very likely to change in the future.
  • pypa/wheel maintainers previously described that the long term vision is to transfer bdist_wheel
    directly to setuptools (See pypa/wheel#262, pypa/wheel#472, pypa/wheel#472).

Pull Request Checklist

Base automatically changed from dev/core_metadata to main August 29, 2023 17:44
@abravalheri abravalheri marked this pull request as draft March 5, 2024 12:08
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

1 participant