Skip to content

Conversation

@matthewfeickert
Copy link
Contributor

********************************************************************************
Please consider removing the following classifiers in favor of a SPDX license expression:
License :: OSI Approved :: GNU Lesser General Public License v2 or later (LGPLv2+)
See https://packaging.python.org/en/latest/guides/writing-pyproject-toml/#license for details.
********************************************************************************
  • Additionally add long_description_content_type to pass 'twine check --strict'.

@matthewfeickert
Copy link
Contributor Author

@tacaswell This addresses some metadata issues for getting cibuildwheel passing.

* Use the recommended SPDX license expression string for the license identifier.
   - c.f. https://packaging.python.org/en/latest/guides/writing-pyproject-toml/#license
* Remove the trove classifier as well, given DeprecationWarning.

> ********************************************************************************
> Please consider removing the following classifiers in favor of a SPDX license expression:
> License :: OSI Approved :: GNU Lesser General Public License v2 or later (LGPLv2+)
> See https://packaging.python.org/en/latest/guides/writing-pyproject-toml/#license for details.
> ********************************************************************************

* Additionally add long_description_content_type to pass 'twine check --strict'.
author='Mikhail Korobov',
author_email='kmike84@gmail.com',
license=LICENSE,
license="LGPL-2.1-or-later",
Copy link
Contributor Author

@matthewfeickert matthewfeickert Oct 29, 2025

Choose a reason for hiding this comment

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

Given

datrie/COPYING

Lines 1 to 2 in c9525da

GNU LESSER GENERAL PUBLIC LICENSE
Version 2.1, February 1999

and the existing

'License :: OSI Approved :: GNU Lesser General Public License v2 or later (LGPLv2+)',

the SPDX identifier should be for v2.1+.

@tacaswell
Copy link
Contributor

This is mechanical "spell checking" of the license meta-data, but I would like to give @kmike and @superbobry a chance to weigh in before we merge this.

@matthewfeickert
Copy link
Contributor Author

This is mechanical "spell checking" of the license meta-data, but I would like to give @kmike and @superbobry a chance to weigh in before we merge this.

👋 Ping @kmike @superbobry

@matthewfeickert
Copy link
Contributor Author

@tacaswell Do you have alternative ways of contacting them?

@tacaswell tacaswell merged commit dc59c60 into pytries:master Nov 13, 2025
16 checks passed
@matthewfeickert matthewfeickert deleted the mnt/metadata branch November 13, 2025 18:46
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.

2 participants