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



--- Comment #11 from Denis Fateyev <de...@fateyev.com> ---
Looks good. Only small corrections:
- BRs `python2-devel` and `pytest` can be also wrapped into `0%{?with_python2}`
conditions (there's no py2 devel packages requirement if no py2 build
provided);
- The same idea to `python2-%{pypi_name}` package declaration (it won't be
built anyway). But this is optional since won't affect anything;
- `rm -fr %{buildroot}%{python2_sitelib}/tests` should go into the conditions
one line above.

-- 
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
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to