Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=863793 --- Comment #14 from Miro Hrončok <mhroncok@xxxxxxxxxx> --- (In reply to comment #13) > - Please provide access to the application in /usr/bi. (If it's worth > providing a .desktop file, it's worth putting in /usr/bin as well. ;-) Actually, it's not worth providing a .desktop file, but unfortunately, packaging guidelines say so. But OK, a shell script to /usr/bin shouldn't be a problem. > - This package installs documentation in %{python_sitelib}, instead of using > %doc. Please use %doc, unless there is a clear, documented, reason not to. There is a button in the application, that opens the documentation. I could either symlink the documentation form doc to python_sitelib, or patch the app to search it in doc, but in both cases, it would brake the rule "If a package includes something as %doc, it must not affect the runtime of the application", right? Will clear the rest issues. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=MEMaAFKS99&a=cc_unsubscribe _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review