https://bugzilla.redhat.com/show_bug.cgi?id=1179350 --- Comment #11 from Miro Hrončok <mhroncok@xxxxxxxxxx> --- > %if 0%{?with_python3} > BuildRequires: rpm-python3 > %endif Does it not need rpm-python for Python 2? You can define %license on platforms without it on the top instead of if/else do %license/%doc. %{!?_licensedir:%global license %%doc} >From my Fedora POV, I'd rather see the %py3_install macros etc., but I see why that's not possible. Otherwise this looks good from the first glance, will do a mock build on rawhide and epels and a formal review tmrw. -- 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 To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx