Spec URL: http://anishpatil.fedorapeople.org/zanata-python-client.spec SRPM URL: http://anishpatil.fedorapeople.org/zanata-python-client-1.3.14-1.fc20.src.rpm Description: Python Client for Zanata Server Fedora Account System Username:anishpatil
Review: + mock build is successful for f22. + rpmlint output on rpms gave zanata-python-client.noarch: E: explicit-lib-dependency python-hashlib zanata-python-client.noarch: E: incorrect-fsf-address /usr/bin/flies zanata-python-client.noarch: W: no-manual-page-for-binary flies zanata-python-client.noarch: W: no-manual-page-for-binary zanata zanata-python-client.src:179: W: macro-in-%changelog %doc 2 packages and 0 specfiles checked; 2 errors, 3 warnings. + Source verified with upstream as (sha256sum) srpm tarball :9cbc8cacc3f86327e2a2aa902cd8215df3964dc23b395043338d75460aebd77b upstream tarball :9cbc8cacc3f86327e2a2aa902cd8215df3964dc23b395043338d75460aebd77b + License "MIT" is valid and is included in its own file as %doc + rest looks following packaging guidelines. Suggestions: 1) Ask upstream to fix the fsf address in /usr/bin/flies 2) every '%' character in changelog should be written as '%%' 3) you don't need sitelib macro for current Fedora releases %{!?python_sitelib: %global python_sitelib %(%{__python} -c "from distutils.sysconfig import get_python_lib; print get_python_lib()")} drop above. 4) Drop Group tag 5) Instead of following %if 0%{?fedora} < 13 BuildRequires: python-devel %endif just add BuildRequires: python2-devel we want this BR: for python packages 6) Change %{__python} to %{__python2} 7) drop %clean 8) change %{python_sitelib} to %{python2_sitelib} 9) drop defattr line Check more on http://fedoraproject.org/wiki/Packaging:Python
ugh! pasting mistake. I see license for this package is LGPLv2+ Drop COPYING file in %doc as it contains GPLv3+ text which is not related to this package. Good to drop it from upstream also.
Hi Parag Thank you for your review comments, I have opened a bug for correct fsf file on upstream Please find new SRPM and Spec file on Spec URL: http://anishpatil.fedorapeople.org/zanata-python-client.spec SRPM URL: http://anishpatil.fedorapeople.org/zanata-python-client-1.3.14-2.fc20.src.rpm
Few more fixes needed 1) In above link spec I can still see COPYING packaged, remove it 2) change %{__python} setup.py install --skip-build --root $RPM_BUILD_ROOT to %{__python2} setup.py install --skip-build --root $RPM_BUILD_ROOT 3)change python2 setup.py build to %{__python2} setup.py build All this is given on http://fedoraproject.org/wiki/Packaging:Python#Example_spec_file
Hi Parag! Thank you for your review comments Please find new spec and SRPM file on Spec URL: http://anishpatil.fedorapeople.org/zanata-python-client.spec SRPM URL: http://anishpatil.fedorapeople.org/zanata-python-client-1.3.14-3.fc20.src.rpm
Looks good now. APPROVED.
Thanks Parag for review New Package SCM Request ======================= Package Name: zanata-python-client Short Description: Python Client for Zanata Server Upstream URL: http://zanata.org Owners: anishpatil Branches: f20 f21 epel7 InitialCC: i18n-team
Git done (by process-git-requests).
zanata-python-client-1.3.14-3.fc20 has been submitted as an update for Fedora 20. https://admin.fedoraproject.org/updates/zanata-python-client-1.3.14-3.fc20
Package Change Request ====================== Package Name: zanata-python-client New Branches: f21 Owners: anishpatil
zanata-python-client-1.3.14-3.fc20 has been pushed to the Fedora 20 testing repository.
zanata-python-client-1.3.14-3.fc20 has been pushed to the Fedora 20 stable repository.
Package Change Request ====================== Package Name: zanata-python-client New Branches: el6 epel7 Owners: anishpatil InitialCC: i18n-team
Git done (by process-git-requests). el6 exists.