[Bug 1203700] Review Request: python-cligj - Click params for GeoJSON CLI

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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



--- Comment #2 from Miro Hrončok <mhroncok@xxxxxxxxxx> ---
Spec URL:
https://raw.githubusercontent.com/hroncok/SPECS/master/python-cligj.spec
SRPM URL:
https://churchyard.fedorapeople.org/SRPMS/python-cligj-0.1.0-1.fc21.src.rpm

(In reply to Julien Enselme from comment #1)
> Hi,
> 
> some preliminary remarks :
> 
> - build fails in mock due to: ImportError: No module named setuptools.
> Please add setuptools as BR
Done.

> - upstream provides the license text (sadly not in the tarballs), please add
> it to your source:
> https://raw.githubusercontent.com/mapbox/cligj/master/LICENSE
Done, with url containing version/tag

> - Please add a / where needed in the %files section. See
> https://fedoraproject.org/wiki/Packaging:Python#Files_to_include
I've read that and I don't understand why would I need to add /, could you
please explain it to me or qute the exact sentence form the guidelines you are
referring to?

Thanks for the hints.

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





[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]