Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

License for packaging#6

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
Manishearth merged 3 commits intounicode-rs:masterfromLecrisUT:fedora/licensing
Nov 3, 2025

Conversation

@LecrisUT
Copy link
Contributor

I am trying to package this project for Fedora1 and we've encountered ambiguous license issue. The suggestion was that it should be licensed under(MIT OR Apache-2.0) AND Unicode-DFS-2016 due toscripts/unicode.py

There could be another possibility to simply excludescripts/* from crate, but I am not sure if the generated code is under the same license Unicode license or not.

Footnotes

  1. https://bugzilla.redhat.com/show_bug.cgi?id=2279853

@LecrisUT
Copy link
ContributorAuthor

I was also pointed tothis discussion which seems to suggest that the generated files would have to have theUnicode-DFS-2016 license attached iiuc.

@musicinmybrain
Copy link

It istables.rs that I believe may actually be (additionally) subject to the license of the Unicode Character Database from which it is derived. The Python script – itcould be excluded from the crate simply because it’s not used by cargo, but I don’t see a reason why itneeds to be.

LecrisUT reacted with thumbs up emoji

@decathorpe
Copy link

As far as I can tell, the situation here is very similar to the unicode-ident crate:https://crates.io/crates/unicode-ident

See alsodtolnay/unicode-ident#9 whereAND Unicode-DFS-2016 was added topackage.license in Cargo.toml.

See alsohttps://github.com/unicode-rs/unicode-xid/pull/32/files for a similar request for a different unicode-rs crate.

@musicinmybrain
Copy link

It was pointed outdownstream that the license text here isUnicode-3.0, notUnicode-DFS-2016, and thatUnicode-3.0 should indeed be the correct license for Unicode version 17 data.

Copy link
ContributorAuthor

@LecrisUTLecrisUT left a comment

Choose a reason for hiding this comment

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

Thanks, what a (un?)fortunate chain of events.@Manishearth, would you consider this PR?

@ManishearthManishearth merged commit558c2bf intounicode-rs:masterNov 3, 2025
5 checks passed
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@ManishearthManishearthManishearth approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@LecrisUT@musicinmybrain@decathorpe@Manishearth

[8]ページ先頭

©2009-2025 Movatter.jp