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



--- Comment #4 from Igor Gnatenko <ignate...@redhat.com> ---
Looks good apart from few things:

- a lot of python- prefixed packages, must be using python2- prefix if
available (six, sqlalchemy, pyopenssl are definitely available in that variant)
- add `-p` flag for installing files to preserve timestamp
- BuildRequires:    git, do you really need git for building? if yes, isn't
git-core sufficient?

-- 
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
_______________________________________________
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

Reply via email to