https://bugzilla.redhat.com/show_bug.cgi?id=1279579
Zbigniew Jędrzejewski-Szmek zbyszek@in.waw.pl changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED CC| |zbyszek@in.waw.pl Assignee|nobody@fedoraproject.org |zbyszek@in.waw.pl Flags| |fedora-review?
--- Comment #1 from Zbigniew Jędrzejewski-Szmek zbyszek@in.waw.pl --- The name should really be lowercase, it's a bother to remember complex capitalization. https://fedoraproject.org/wiki/Packaging:NamingGuidelines#General_Naming says "Package names should be in lower case and use dashes in preference to underscores."
You *could* build the documentation, would be nice to have a pdf with the help.
No need to rm -rf build/ in %prep: %autosetup will recreate the whole source directory anyway. This applies also to a few other packages that I reviewed; I forgot to mention it before.
#!/usr/bin/env python3 → #!/usr/bin/python3.
- license is OK (Apache) - license file is present, %license is used - latest version - no scriptlets needed or present - package builds and installs
Build fails in tests, I'll kick of a scratch build.
Note to self: check explicit Requires: python3.