Bug 1051840 - Review Request: python-libqutrub - Arabic verb conjuagtion library
Summary: Review Request: python-libqutrub - Arabic verb conjuagtion library
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Parag AN(पराग)
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2014-01-12 00:56 UTC by Mosaab Alzoubi
Modified: 2014-11-14 20:23 UTC (History)
3 users (show)

Fixed In Version: python-libqutrub-1.0-3.el7
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2014-11-14 20:23:43 UTC
Type: ---
Embargoed:
panemade: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Mosaab Alzoubi 2014-01-12 00:56:38 UTC
Spec URL: http://ojuba.org/oji/SPECS/python-libqutrub.spec
SRPM URL: http://ojuba.org/oji/SRPMS/python-libqutrub-1.0-1.fc20.src.rpm
Description:
Arabic language verb conjuagtion library to be used by Python.
Fedora Account System Username: moceap

Comment 1 Volker Fröhlich 2014-02-11 07:57:05 UTC
I think the license is GPL, not GPLv1. At least I can't find any evidence of a version statement.

Comment 2 Mosaab Alzoubi 2014-02-11 11:10:56 UTC
I call the programmar Taha Zarroqi and he said he mean GPL1 :)

Comment 3 Parag AN(पराग) 2014-10-21 06:32:03 UTC
Review:

+ package built fine in mock rawhide(x86_64)

+ rpmlint on generated rpms gave output
python-libqutrub.noarch: W: spelling-error Summary(en_US) conjuagtion -> conjugation, conjuration, continuation
python-libqutrub.noarch: I: enchant-dictionary-not-found ar
python-libqutrub.noarch: W: spelling-error %description -l en_US conjuagtion -> conjugation, conjuration, continuation
python-libqutrub.noarch: W: no-documentation
python-libqutrub.src: W: spelling-error Summary(en_US) conjuagtion -> conjugation, conjuration, continuation
python-libqutrub.src: W: spelling-error %description -l en_US conjuagtion -> conjugation, conjuration, continuation
python-libqutrub.src: W: strange-permission LibQutrub-1.0.zip 0600L
2 packages and 0 specfiles checked; 0 errors, 6 warnings.

+ Source verified with upstream as (sha256sum)
srpm tarball: c10d6e1d101026920060d2b8d921debcdfaaf795bd3c7241278e357415e942c1
upstream tarball : c10d6e1d101026920060d2b8d921debcdfaaf795bd3c7241278e357415e942c1

+ License is just "GPL+" as per PKG-INFO  

suggestions:
1) Email to upstream author and get clarification on what is the license for this python module. When you get reply you can include that reply as a LICENSE file as Source1: in spec and add it to %doc

2) I failed to understand the meaning of following line
#Site of files just for including:

instead of this you can include like "Remove the python shebang"

3) you need to follow https://fedoraproject.org/wiki/Packaging:Python guidelines

Add in %prep after setup macro line
rm -rf *.egg-info

this is to make sure you will not install directly egg info file that upstream if providing and instead you are installing egg info that gets created in %build section

4) As per python guidelines your %install section should use
%{__python2} setup.py install --skip-build --root=%{buildroot}


5) your %files section should look like this
%files -n python-%{module_name}
%doc PKG-INFO
%{python2_sitelib}/%{modname}/
%{python2_sitelib}/%{distname}-%{version}-py2.*.egg-info


6) License tag can be just "GPL+" as there is no mention of any GPL version information but good to ask upstream. 

Note: We need some reference that will show what is the license for this package.


7) Group tag is optional so you can remove it for fedora releases.

Comment 5 Parag AN(पराग) 2014-10-24 04:17:54 UTC
1) fix the rpmlint messages
python-libqutrub.noarch: W: spelling-error Summary(en_US) conjuagtion -> conjugation, conjuration, continuation
python-libqutrub.noarch: I: enchant-dictionary-not-found ar
python-libqutrub.noarch: W: spelling-error %description -l en_US conjuagtion -> conjugation, conjuration, continuation
python-libqutrub.noarch: W: wrong-file-end-of-line-encoding /usr/share/doc/python-libqutrub/PKG-INFO
python-libqutrub.src: W: spelling-error Summary(en_US) conjuagtion -> conjugation, conjuration, continuation
python-libqutrub.src: W: spelling-error %description -l en_US conjuagtion -> conjugation, conjuration, continuation
python-libqutrub.src:13: W: mixed-use-of-spaces-and-tabs (spaces: line 13, tab: line 4)
2 packages and 0 specfiles checked; 0 errors, 6 warnings.


=> fix the wrong-file-end-of-line-encoding and mixed-use-of-spaces-and-tabs warnings.

2) When you add license text as GPLv1 and use tag GPL+ then its contradiction. Because GPL+ mean technically licensed under *any* version of the GPL.

If upstream is saying its GPLv1 then ask upstream to add it in upstream code and release new version or as I said either keep license as "GPL+" and drop Source1 or if upstream can clarify it as a GPLv1 then attach its email reply or any other static reference on the internet (say like you got a reply on upstream project website) that you can include in spec file.

Comment 6 Mosaab Alzoubi 2014-10-24 20:09:27 UTC
I called him not by email, and he mean GPLv1.
All Fixed:

Spec: http://ojuba.org/test/python-libqutrub.spec
Srpm: http://ojuba.org/test/python-libqutrub-1.0-3.oj35.src.rpm

Comment 7 Parag AN(पराग) 2014-10-25 01:25:13 UTC
just a single rpmlint warning remains
python-libqutrub.src:13: W: mixed-use-of-spaces-and-tabs (spaces: line 13, tab: line 4)

you can fix this when you import this package.

APPROVED.

Comment 8 Mosaab Alzoubi 2014-10-25 01:27:43 UTC
Ok, Thank You.

New Package SCM Request
=======================
Package Name: python-libqutrub
Short Description: Arabic verb conjuagtion library
Owners: moceap
Branches: f19 f20 f21 epel7

Comment 9 Gwyn Ciesla 2014-10-27 12:03:48 UTC
Git done (by process-git-requests).

Comment 10 Fedora Update System 2014-10-28 18:42:26 UTC
python-libqutrub-1.0-3.fc19 has been submitted as an update for Fedora 19.
https://admin.fedoraproject.org/updates/python-libqutrub-1.0-3.fc19

Comment 11 Fedora Update System 2014-10-28 18:42:26 UTC
python-libqutrub-1.0-3.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/python-libqutrub-1.0-3.fc20

Comment 12 Fedora Update System 2014-10-28 18:42:36 UTC
python-libqutrub-1.0-3.fc21 has been submitted as an update for Fedora 21.
https://admin.fedoraproject.org/updates/python-libqutrub-1.0-3.fc21

Comment 13 Fedora Update System 2014-10-28 18:42:46 UTC
python-libqutrub-1.0-3.el7 has been submitted as an update for Fedora EPEL 7.
https://admin.fedoraproject.org/updates/python-libqutrub-1.0-3.el7

Comment 14 Fedora Update System 2014-10-31 01:27:50 UTC
python-libqutrub-1.0-3.el7 has been pushed to the Fedora EPEL 7 testing repository.

Comment 15 Fedora Update System 2014-11-10 05:59:13 UTC
python-libqutrub-1.0-3.fc21 has been pushed to the Fedora 21 stable repository.

Comment 16 Fedora Update System 2014-11-10 05:59:21 UTC
python-libqutrub-1.0-3.fc20 has been pushed to the Fedora 20 stable repository.

Comment 17 Fedora Update System 2014-11-10 06:07:47 UTC
python-libqutrub-1.0-3.fc19 has been pushed to the Fedora 19 stable repository.

Comment 18 Fedora Update System 2014-11-14 20:23:43 UTC
python-libqutrub-1.0-3.el7 has been pushed to the Fedora EPEL 7 stable repository.


Note You need to log in before you can comment on or make changes to this bug.