Skip to content

fix: don't treat bad sdist metadata as fatal error#788

Merged
mergify[bot] merged 1 commit intopython-wheel-build:mainfrom
tiran:sdist-metadata-validation
Sep 25, 2025
Merged

fix: don't treat bad sdist metadata as fatal error#788
mergify[bot] merged 1 commit intopython-wheel-build:mainfrom
tiran:sdist-metadata-validation

Conversation

@tiran
Copy link
Copy Markdown
Collaborator

@tiran tiran commented Sep 25, 2025

default_get_install_dependencies_of_sdist no longer treats bad sdist metadata as a fatal error. The function now logs a non-fatal error message to inform the user of potential issues.

Fromager stub metadata is no longer validated at all. It likely contains a wrong version number.

@tiran tiran requested a review from a team as a code owner September 25, 2025 05:41
@mergify mergify Bot added the ci label Sep 25, 2025
`default_get_install_dependencies_of_sdist` no longer treats bad sdist
metadata as a fatal error. The function now logs a non-fatal error
message to inform the user of potential issues.

Fromager stub metadata is no longer validated at all. It likely contains
a wrong version number.

Signed-off-by: Christian Heimes <cheimes@redhat.com>
@tiran tiran force-pushed the sdist-metadata-validation branch from ddba77d to d2a5c8b Compare September 25, 2025 07:42
Copy link
Copy Markdown
Contributor

@EmilienM EmilienM left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Contributor

@rd4398 rd4398 left a comment

Choose a reason for hiding this comment

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

Looks good!

@mergify mergify Bot merged commit b3a547f into python-wheel-build:main Sep 25, 2025
112 checks passed
@tiran tiran deleted the sdist-metadata-validation branch October 6, 2025 08:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants