Add license metadata to pyproject.toml

[johnthagen]

Closes #1622

[CLAassistant]

All committers have signed the CLA.

[johnthagen]

I'll probably pass on signing a CLA for something this tiny, but any maintainer is welcome to make this fix on their own.

[IvanIsCoding]

I remember #1494 explicitly removed this, there was a warning that field was deprecated. I will read more on the subject and see what other projects are doing

[IvanIsCoding]

Apparently I misunderstood https://peps.python.org/pep-0639/#deprecate-license-field. The PR is right, I just need to understand the warning that was being given before. I will merge this in a separate PR.

[jakelishman]

Ivan: maybe you're mixing up the differences between the field names in the wheel "core metadata" fields and how the information is read in from pyproject.toml (or equivalents) by Python build backends? That was my confusion when I read the PEP, at least - they've all got similar names, but there's nothing formally requiring there to be a 1:1 link between them.

[IvanIsCoding]

Ivan: maybe you're mixing up the differences between the field names in the wheel "core metadata" fields and how the information is read in from pyproject.toml (or equivalents) by Python build backends? That was my confusion when I read the PEP, at least - they've all got similar names, but there's nothing formally requiring there to be a 1:1 link between them.

Yeah I mixed things up, the summary of it is: maturin should generate the right metadata if we merge this PR. The fix is right. I will confirm with pip-licenses

[mergify]

Tick the box to add this pull request to the merge queue (same as @mergifyio queue).

• Queue this pull request