https://bugzilla.redhat.com/show_bug.cgi?id=1575142 Robert-André Mauchin <zebob.m@xxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |zebob.m@xxxxxxxxx --- Comment #2 from Robert-André Mauchin <zebob.m@xxxxxxxxx> --- - Apache 2.0 is not a valid License shorthand, it should be "ASL 2.0". See the full list here: https://fedoraproject.org/wiki/Licensing:Main?rd=Licensing#SoftwareLicenses - Not needed: %clean rm -rf %{buildroot} - Not needed in %install: rm -rf %{buildroot} - For Fedora >= 28, you can use %?python_enable_dependency_generator instead of specify the Requires manually. See https://fedoraproject.org/wiki/Changes/EnablingPythonGenerators. If you plan to use it on EPEL or Fedora < 28, disregard this for now. - You could use %{py2_dist and {py3_dist to specify your BR and RR, See https://fedoraproject.org/wiki/Packaging:Python#Requires_and_BuildRequires_with_standardized_names -- 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