https://bugzilla.redhat.com/show_bug.cgi?id=1303819 --- Comment #11 from Denis Fateyev <denis@xxxxxxxxxxx> --- 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@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review