https://bugzilla.redhat.com/show_bug.cgi?id=2121902 Troy Curtis <troy@xxxxxxxxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |troy@xxxxxxxxxxxxxxxx --- Comment #3 from Troy Curtis <troy@xxxxxxxxxxxxxxxx> --- - Should fix the shebang in pyinstrument-doc/examples/django_example/manage.py - The doc sub-package can be marked noarch - Including Sphinx html documentation requires a bit more effort, possibly to the point of being unmanagable while complying to the current bundled library guidelines. See recent package review [1] and a packaging doc pull request [2]. - Looks like vendor/keypath.py is BSD 2-Clause, and also likely should be handled according to the Bundled library guidelines [3] - vendor/appdirs.py is MIT, and also likely should be handled according to the Bundled library guidelines [3] 1: https://bugzilla.redhat.com/show_bug.cgi?id=2173018#c5 2: https://pagure.io/packaging-committee/pull-request/1244 3: https://docs.fedoraproject.org/en-US/fesco/Bundled_Software_policy/ -- You are receiving this mail because: You are always notified about changes to this product and component You are on the CC list for the bug. https://bugzilla.redhat.com/show_bug.cgi?id=2121902 _______________________________________________ package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx 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/package-review@xxxxxxxxxxxxxxxxxxxxxxx Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
