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



--- Comment #3 from Javier Peña <[email protected]> ---
Hi Chandan,

Thanks for the fixes. I still have some comments:

- The with_python3 snippet still needs to be changed (see comment 1)
- Please include python BRs within "%if 0%{?with_python3} ... %endif",
otherwise a RHEL 7 rebuild would fail.
- I have checked in a venv, and the tests do not get installed. So we can
remove the following line:

rm -r $(find %{_buildrootdir} -type d -name 'tests') || /bin/true

-- 
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 -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to