Bug 950907 - Review Request: python-jedi - Python auto-completion
Summary: Review Request: python-jedi - Python auto-completion
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: 19
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Robert Kuska
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-04-11 07:47 UTC by Petr Hracek
Modified: 2016-04-18 10:16 UTC (History)
3 users (show)

Fixed In Version: python-jedi-0.5b5-3.fc19
Clone Of:
Environment:
Last Closed: 2013-04-30 04:32:20 UTC
Type: ---
Embargoed:
rkuska: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Petr Hracek 2013-04-11 07:47:40 UTC
Spec URL: http://phracek.fedorapeople.org/python-jedi/python-jedi.spec
SRPM URL: http://phracek.fedorapeople.org/python-jedi/python-jedi-0.5b5-1.fc18.src.rpm
Description:
Jedi is an autocompletion tool for Python. It understands almost all of the basic Python syntax elements including many builtins.

Fedora Account System Username:phracek
[stone@kiasportyw jedi-master]$ rpmlint -v python-jedi.spec 
python-jedi.spec: I: checking-url http://pypi.python.org/packages/source/j/jedi/jedi-0.5b5.tar.gz (timeout 10 seconds)
0 packages and 1 specfiles checked; 0 errors, 0 warnings.
[stone@kiasportyw jedi-master]$

Comment 1 Robert Kuska 2013-04-11 08:03:21 UTC
I am taking this.

Comment 2 Robert Kuska 2013-04-11 08:58:02 UTC
- Url shouldn't be link to source code, use [1].
- Fix python-devel to python2-devel [2].
- Rpmlint returns 'python-jedi.src: W: invalid-license LGPL', LGPLv3 is the right
one [3], and few typos, use rpmlint on src.rpm.
- I see that there are tests on github [4], did you run them before packaging?
Also add please comment about tests run into spec file. 




[1] https://jedi.readthedocs.org/en/latest/index.html
[2] https://fedoraproject.org/wiki/Packaging:Python#BuildRequires
[3] https://fedoraproject.org/wiki/Licensing:Main?rd=Licensing#Good_Licenses
[4] https://github.com/davidhalter/jedi/tree/master/test

Comment 3 Petr Hracek 2013-04-11 12:13:13 UTC
Hi,

URL has been updated as well as BuildRequires section.
rpmlint has been run on src.rpm. This was not done before.

Tests comments are mentioned in changelog.

SPEC: http://phracek.fedorapeople.org/python-jedi/python-jedi.spec
SRPM: http://phracek.fedorapeople.org/python-jedi/python-jedi-0.5b5-2.fc18.src.rpm

Comment 4 Robert Kuska 2013-04-12 07:07:32 UTC
- For first, I am sorry for misleading comment about tests. Proper way to do this is checkout the source code, make archive and add it as another Source into spec. Check this [1] for an inspiration.
- I can't approve broken package, according the info you provided, some tests fail.

[1] http://pkgs.fedoraproject.org/cgit/rubygem-activesupport.git/tree/rubygem-activesupport.spec

Comment 5 Petr Hracek 2013-04-17 06:11:49 UTC
On the base of upstream discussion (https://github.com/davidhalter/jedi/issues/200) Test Suite if available only in dev branch.

SPEC: http://phracek.fedorapeople.org/python-jedi/python-jedi.spec
SPRM: http://phracek.fedorapeople.org/python-jedi/python-jedi-0.5b5-3.fc18.src.rpm

Comment 6 Robert Kuska 2013-04-22 06:10:20 UTC
Looks good. Approved.

Comment 7 Petr Hracek 2013-04-23 12:14:29 UTC
New Package SCM Request
=======================
Package Name: python-jedi
Short Description: Python auto-completion
Owners: phracek
Branches: f19

Comment 8 Gwyn Ciesla 2013-04-23 12:27:33 UTC
Use your FAS account name, not email address.

Comment 9 Petr Hracek 2013-04-23 13:09:25 UTC
New Package SCM Request
=======================
Package Name: python-jedi
Short Description: Python auto-completion
Owners: phracek
Branches: f19

Comment 10 Gwyn Ciesla 2013-04-23 13:31:25 UTC
Git done (by process-git-requests).

Comment 11 Fedora Update System 2013-04-23 14:47:12 UTC
python-jedi-0.5b5-3.fc19 has been submitted as an update for Fedora 19.
https://admin.fedoraproject.org/updates/python-jedi-0.5b5-3.fc19

Comment 12 Petr Hracek 2013-04-23 14:49:25 UTC
Package has been imported
scm-commit (http://lists.fedoraproject.org/pipermail/scm-commits/Week-of-Mon-20130422/1004863.html)

Comment 13 Fedora Update System 2013-04-23 16:46:52 UTC
python-jedi-0.5b5-3.fc19 has been pushed to the Fedora 19 testing repository.

Comment 14 Fedora Update System 2013-04-30 04:32:22 UTC
python-jedi-0.5b5-3.fc19 has been pushed to the Fedora 19 stable repository.


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