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



--- Comment #4 from Steve Traylen <[email protected]> ---
- Please add new line between the two entry of the changelog.

Done, I was sure both used to be permitted, but I can't see that now.

- Please use {__python2} instead of {__python}

Done, Indeed missed that from the new guidelines.

- Please update your spec or SRPMS so they match

ack.

- Since the tests are not available in 0.5, I guess that you should remove the
%check  macro

I'd rather leave it as a reminder as something to do with next release.

- Since all currently maintained versions of Fedora have python3, there is no
need for the with_python3 variable.

I plan EPEL7 and 6 builds also where is is not available.

Currently I'm trying to fix the duplicate files but that is rather tricky it
seems, will post again when done.

-- 
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]
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to