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



--- Comment #8 from Fabio Valentini <[email protected]> ---
Looks good to me, with one exception:
I just noticed the included tree-sitter C library code bundles parts of icu,
so you'll need to add "Provides: bundled(libicu) = version" to the -devel
package, as well.

The ICU license file is already included, but you'll need to fix the license
tag to incorporate the license of the bundled ICU sources, which seems to be a
version of the "Unicode" license:
https://fedoraproject.org/wiki/Licensing/Unicode (so it should probably be
"License: MIT and Unicode").


-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
https://bugzilla.redhat.com/show_bug.cgi?id=2028895
_______________________________________________
package-review mailing list -- [email protected]
To unsubscribe send an email to [email protected]
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/[email protected]
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure

Reply via email to