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



--- Comment #2 from Marcel Haerry <[email protected]> ---
(In reply to Christopher Meng from comment #1)
> 1. Remove
> 
> %defattr(-, root, root)

done. Will push new spec file once I solved the other issues.

> 2. Filter out the private library.
> 
> https://fedoraproject.org/wiki/Packaging:AutoProvidesAndRequiresFiltering

Sorry, but I don't yet understand which private library you mean, as I'm not
sure which of the Requires/Provides should be seen as private.

> 3. BuildRequires:  python-devel
> 
> should be
> 
> BuildRequires:  python2-devel

changed.

> 4. If possible, merge the %check section into setup.py.

Will ask the maintainer to integrate once I will notify him, that the library
now got included into Fedora.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=gEZdeKDshs&a=cc_unsubscribe
_______________________________________________
package-review mailing list
[email protected]
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to