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



--- Comment #11 from Miro HronĨok <[email protected]> ---
> %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 -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to