Skip to content

Use standardized [project] table in pyproject.toml #399

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

hroncok
Copy link

@hroncok hroncok commented May 11, 2025

This allows us to swap the build backend more easily in RHEL, where we don't have poetry-code.

This requires poetry-core >= 2 because older poetry did not support this.

Also, replace the value of the license text with a SPDX identifier as documented in https://python-poetry.org/docs/pyproject/#license
Otherwise, poetry-core injects a Other/Proprietary License classifier. (Note that poetry-core does not yet support PEP 639 fully).

This change creates s slight change in the metadata, comments inline.

--- old/qrcode-8.2-py3-none-any/qrcode-8.2.dist-info/METADATA
+++ new/qrcode-8.2-py3-none-any/qrcode-8.2.dist-info/METADATA
@@ -2,16 +2,15 @@
 Name: qrcode
 Version: 8.2
 Summary: QR Code image generator
-License: BSD
+License: BSD-3-Clause

This was intentional.

 Keywords: qr,denso-wave,IEC18004
 Author: Lincoln Loop
 Author-email: info@lincolnloop.com
-Requires-Python: >=3.9,<4.0
+Requires-Python: ~=3.9

This is identical in reality.

 Classifier: Development Status :: 5 - Production/Stable
-Classifier: Intended Audience :: Developers

This only changes order :/

 Classifier: License :: OSI Approved :: BSD License
-Classifier: License :: Other/Proprietary License

This is intentional.

 Classifier: Operating System :: OS Independent
+Classifier: Intended Audience :: Developers

See above, order only.

 Classifier: Programming Language :: Python
 Classifier: Programming Language :: Python :: 3
 Classifier: Programming Language :: Python :: 3.9
@@ -26,8 +25,10 @@
 Provides-Extra: pil
 Provides-Extra: png
 Requires-Dist: colorama ; sys_platform == "win32"
-Requires-Dist: pillow (>=9.1.0) ; extra == "pil" or extra == "all"
-Requires-Dist: pypng ; extra == "png" or extra == "all"
+Requires-Dist: pillow (>=9.1.0) ; extra == "all"
+Requires-Dist: pillow (>=9.1.0) ; extra == "pil"
+Requires-Dist: pypng ; extra == "all"
+Requires-Dist: pypng ; extra == "png"

This is identical.

 Project-URL: Homepage, https://github.com/lincolnloop/python-qrcode
 Description-Content-Type: text/x-rst

This allows us to swap the build backend more easily in RHEL, where we don't have poetry-code.

This requires poetry-core >= 2 because older poetry did not support this.

Also, replace the value of the license text with a SPDX identifier as documented in
https://python-poetry.org/docs/pyproject/#license
Otherwise, poetry-core injects a `Other/Proprietary License` classifier.
(Note that poetry-core does not yet support PEP 639 fully).

This change creates s slight change in the metadata, comments inline.

    --- old/qrcode-8.2-py3-none-any/qrcode-8.2.dist-info/METADATA
    +++ new/qrcode-8.2-py3-none-any/qrcode-8.2.dist-info/METADATA
    @@ -2,16 +2,15 @@
     Name: qrcode
     Version: 8.2
     Summary: QR Code image generator
    -License: BSD
    +License: BSD-3-Clause

This was intentional.

     Keywords: qr,denso-wave,IEC18004
     Author: Lincoln Loop
     Author-email: info@lincolnloop.com
    -Requires-Python: >=3.9,<4.0
    +Requires-Python: ~=3.9

This is identical in reality.

     Classifier: Development Status :: 5 - Production/Stable
    -Classifier: Intended Audience :: Developers

This only changes order :/

     Classifier: License :: OSI Approved :: BSD License
    -Classifier: License :: Other/Proprietary License

This is intentional.

     Classifier: Operating System :: OS Independent
    +Classifier: Intended Audience :: Developers

See above, order only.

     Classifier: Programming Language :: Python
     Classifier: Programming Language :: Python :: 3
     Classifier: Programming Language :: Python :: 3.9
    @@ -26,8 +25,10 @@
     Provides-Extra: pil
     Provides-Extra: png
     Requires-Dist: colorama ; sys_platform == "win32"
    -Requires-Dist: pillow (>=9.1.0) ; extra == "pil" or extra == "all"
    -Requires-Dist: pypng ; extra == "png" or extra == "all"
    +Requires-Dist: pillow (>=9.1.0) ; extra == "all"
    +Requires-Dist: pillow (>=9.1.0) ; extra == "pil"
    +Requires-Dist: pypng ; extra == "all"
    +Requires-Dist: pypng ; extra == "png"

This is identical.

     Project-URL: Homepage, https://github.com/lincolnloop/python-qrcode
     Description-Content-Type: text/x-rst
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.

1 participant