Skip to content

Add LICENSE file to wheels#69

Merged
ddelange merged 2 commits intooconnor663:masterfrom
justeph:justeph/fix-licensing
May 19, 2025
Merged

Add LICENSE file to wheels#69
ddelange merged 2 commits intooconnor663:masterfrom
justeph:justeph/fix-licensing

Conversation

@justeph
Copy link
Copy Markdown
Contributor

@justeph justeph commented May 19, 2025

The wheels distributed on pypi don't include the License.

Add a project section with the minimum required name and version fields. This allows maturin to add the license file into the wheel.

Add a project section with the minimum required name and version fields.
This allows maturin to add the license file into the wheel.

Signed-off-by: Julien Stephan <jstephan@baylibre.com>
Comment thread pyproject.toml Outdated
Copy link
Copy Markdown
Collaborator

@ddelange ddelange left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

artifacts are looking good, the LICENSE is now included in the wheel and the METADATA file contains both the License-File: LICENSE directive as well as the License: CC0-1.0 OR Apache-2.0 SPDX identifier 👍 thanks @justeph!

@ddelange ddelange changed the title fix licensing Add LICENSE file to wheels May 19, 2025
@ddelange ddelange merged commit 780c9f9 into oconnor663:master May 19, 2025
135 checks passed
@ddelange
Copy link
Copy Markdown
Collaborator

https://pypi.org/project/blake3/1.0.5/

@justeph
Copy link
Copy Markdown
Contributor Author

justeph commented May 19, 2025

excellent thank you!

messense pushed a commit to PyO3/maturin that referenced this pull request Oct 14, 2025
> I didn't understand why maturin
[fails](https://github.com/oconnor663/blake3-py/actions/runs/18435038379/job/52527366829#step:10:66)
if I try to make name dynamic too.

docs for one of the issues I mentioned in
#2769

> Also License will be missing from the wheel even if [project] is not
defined at all, which is also undocumented, see
oconnor663/blake3-py#69.

this one looks like an actual bug in maturin to me, but I might be
mistaken.
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