Skip to content

Conversation

@DimitriPapadopoulos
Copy link
Contributor

Keep the redundant Trove classifier for now, until tools such as pip-licenses support PEP 639.

Keep the redundant Trove classifier for now, until tools such as
pip-licenses support PEP 639.
Fixes this build error:
      flit_core.config.ConfigError: License classifiers are deprecated in favor of the license expression.
                                    Remove the 'License :: OSI Approved :: MIT License' classifier
@eli-schwartz
Copy link
Contributor

Keep the redundant Trove classifier for now, until tools such as pip-licenses support PEP 639.

Since in the end you removed it, maybe that's a sign that this PR is too soon?

I'd also recommend submitting a bug report to the upstream build backend asking for this error message to be deleted, since it appears the error message is in fact a bad thing that breaks the ecosystem.

@DimitriPapadopoulos
Copy link
Contributor Author

Too soon? I don't know. It's almost exclusively an issue with pip-licenses, which hardly appears to be maintained: it is a tool designed to dump a license list but doesn't support PEP 639 - created more than 5 years ago.

@DimitriPapadopoulos
Copy link
Contributor Author

DimitriPapadopoulos commented Jul 31, 2025

With that said, the error message could indeed be turned into a warning:

@eli-schwartz

This comment was marked as off-topic.

@DimitriPapadopoulos

This comment was marked as off-topic.

@DimitriPapadopoulos
Copy link
Contributor Author

Upstream issue: pypa/flit#753.

@eli-schwartz

This comment was marked as off-topic.

@Secrus
Copy link
Member

Secrus commented Jul 31, 2025

Gentlebeings, this is neither the PR nor the repo for such discussion. Please, keep the discussion on topic.

@Secrus Secrus merged commit 350740a into pypa:main Aug 2, 2025
21 checks passed
@DimitriPapadopoulos DimitriPapadopoulos deleted the PEP639 branch August 2, 2025 16:47
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.

3 participants