Skip to content

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

Closed
hroncok wants to merge 1 commit intolincolnloop:mainfrom
hroncok:project
Closed

Use standardized [project] table in pyproject.toml#399
hroncok wants to merge 1 commit intolincolnloop:mainfrom
hroncok:project

Conversation

@hroncok
Copy link
Copy Markdown

@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
@bartTC
Copy link
Copy Markdown
Member

bartTC commented Jul 22, 2025

Thank you! Tested it with uv, works great.

Merged in 9c6d254.

-Requires-Python: >=3.9,<4.0
+Requires-Python: ~=3.9

This doesn't make sense to me. I didn't take this over. It would mean, it requires any Python 3.9 version, but not above. Or do I understand incorrectly?

@bartTC bartTC closed this Jul 22, 2025
@hroncok
Copy link
Copy Markdown
Author

hroncok commented Jul 22, 2025

This doesn't make sense to me...

It's the same thing in different notation.

@hroncok
Copy link
Copy Markdown
Author

hroncok commented Jul 22, 2025

~=3.9 means it requires any Python 3 version at least 3.9

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