Skip to content

Conversation

@zacchiro
Copy link
Contributor

Rationale: by specifying the license as a file (like before), the full text of the license is included in the package metadata, and shown in various places, e.g., in the output of pip show, which is quite cumbersome. With this change, we use the alternative way of specifying license (namely: using a standard SPDX identifier), which makes license metadata concise and, incidentally, also machine readable.

Rationale: by specifying the license as a file (like before), the full text of the license is included in the package metadata, and shown in various places, e.g., in the output of `pip show`, which is quite cumbersome. With this change, we use the alternative way of specifying license (namely: using a standard SPDX identifier), which makes license metadata concise and, incidentally, also machine readable.
@dnicolodi
Copy link
Collaborator

Thanks @zacchiro. Doing this is on my todo list. However, this assumes PEP 729 support in the build backend, thus it must come with a bump in the required setuptools version. Also, having the license specified with a classifier is deprecated when specifying the license with a SPDX identifier in the license filed, and setuptools makes it an hard metadata validation error, thus the classifier should be removed.

@dnicolodi
Copy link
Collaborator

The docs / build aqnd checks / build CI failures are related to this PR. The others are not. It seems that beancount git master changed something in the serialization of amounts.

@dnicolodi
Copy link
Collaborator

I switched to using the SPDX license identifier in #259 alongside some fixes to make the CI happy. Thanks @zacchiro.

@dnicolodi dnicolodi closed this Jul 28, 2025
@zacchiro zacchiro deleted the patch-1 branch July 29, 2025 06:37
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