pyproject.toml: migrate to PEP 639 license expressions - #1956
Conversation
Make 'license' a proper SDPX license expression. Use 'license-files' for license files. Replace the also-deprecated "License ::" classifier. See https://setuptools.pypa.io/en/latest/userguide/license_migration.html Fixes: | SetuptoolsDeprecationWarning: `project.license` as a TOML table is deprecated | !! | | ******************************************************************************** | Please use a simple string containing a SPDX expression for `project.license`. You can also use `project.license-files`. (Both options available on setuptools>=77.0.0). | | By 2027-Feb-18, you need to update your project and remove deprecated calls | or your builds will no longer be supported. | | See https://packaging.python.org/en/latest/guides/writing-pyproject-toml/#license for details. | ******************************************************************************** | SetuptoolsDeprecationWarning: License classifiers are deprecated. | !! | | ******************************************************************************** | 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. | ******************************************************************************** Signed-off-by: Enrico Jörns <ejo@pengutronix.de>
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #1956 +/- ##
========================================
- Coverage 61.0% 61.0% -0.1%
========================================
Files 182 182
Lines 14906 14906
========================================
- Hits 9105 9103 -2
- Misses 5801 5803 +2
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. |
Bastian-Krause
left a comment
Bastian-Krause
left a comment
There was a problem hiding this comment.
Duplicate of #1687. Needs investigation if the situation changed since then, see the comments in that PR.
|
Looks like the current recommendation for our case is to use |
|
Looking at https://layers.openembedded.org/layerindex/recipe/37586/, this would likely break scarthgap. |
I don't think we should wait until that's EOL, but maybe a little bit longer so everybody had enough time to switch to wrynose? I think a certain someone will need meta-labgrid on scarthgap with a recent enough version of labgrid soon. Bumping setuptools there might cause fallout for other Python packages, so I guess I would need to revert this patch then if we merge it now. |
|
I thought license-files would be supported for every version of setup tools that supports project.toml? |
3 participants
Description
Make
licensea proper SDPX license expression.Use
license-filesfor license files.Replace the also-deprecated
License ::classifier.See https://setuptools.pypa.io/en/latest/userguide/license_migration.html
Fixes: