Bug 435090

Summary: Review Request: Cython - This is a development version of Pyrex, a language for writing Python extension modules.
Product: [Fedora] Fedora Reporter: Neal Becker <ndbecker2>
Component: Package ReviewAssignee: José Matos <jamatos>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: fedora-package-review, michel, notting, rdieter, steve.traylen
Target Milestone: ---Flags: jamatos: fedora-review+
j: fedora-cvs+
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Cython-0.14.1-3.el6 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2008-04-15 12:27:16 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
Spec file patch with some fixes none

Description Neal Becker 2008-02-27 10:41:28 UTC
Spec URL: http://nbecker.dyndns.org:8080/RPM/Cython.spec
SRPM URL: http://nbecker.dyndns.org:8080/RPM/Cython-0.9.6.12-1.fc8.src.rpm
Description: This is a development version of Pyrex, a language
for writing Python extension modules.

rpmlint - 2 small warnings
mock - pass

Comment 1 José Matos 2008-04-02 12:15:01 UTC
Neal I would like to review the package but I can't access your site because
it is in a port blocked by the firewall here. Could you please place elsewhere
with fedorapeople.org being the logical candidate. :-)

Comment 2 Neal Becker 2008-04-06 23:35:16 UTC
OK, I copied to http://nbecker.fedorapeople.org/

Comment 3 José Matos 2008-04-14 11:06:24 UTC
Taking the review.

Comment 4 José Matos 2008-04-14 11:52:34 UTC
Needs work:
* Source Cython-0.9.6.13.1.tar.gz is different from upstream
  (wiki: QAChecklist item 2) using md5sum I get:
b84818d4cfcec68e6dfb71187e9f0b4b  upstream
3b7bde3b0869b3624cd80fe5cde4e2ed  srpm

* rpmlint of source RPM:
Cython.src: W: mixed-use-of-spaces-and-tabs (spaces: line 22, tab: line 1)
Cython.src: W: invalid-license PSFL

* rpmlint of Cython:
Cython.noarch: W: file-not-utf8
/usr/share/doc/Cython-0.9.6.13.1/CHANGES_pyrex.txt
Cython.noarch: W: invalid-license PSFL

The license should be Python. The mixed use of spaces and tabs is easy to fix
and the other warning is just that. :-)

* %files
According to http://fedoraproject.org/wiki/Packaging/Python

/!\ You shouldn't rely on INSTALLED_FILES, as that will not list directories,
which will need to be specified in the %files section as well. Using globs in
the %files section is safer.

* %description
You refer Doc/About.html but that file is not packaged.

Comment 5 Neal Becker 2008-04-14 12:20:58 UTC
Please see
http://nbecker.fedorapeople.org/Cython-0.9.6.13.1-2.fc8.src.rpm

Comment 6 José Matos 2008-04-14 14:07:15 UTC
Created attachment 302341 [details]
Spec file patch with some fixes

+* Mon Apr 14 2008 José Matos <jamatos[AT]fc.up.pt> - 0.9.6.13.1-3
+- Remove remaining --record.
+- Add more documentation (Doc and Tools).
+- Add correct entry for egg-info (F9+).

Comment 7 José Matos 2008-04-14 14:21:18 UTC
In this I thought it was easier to send a patch. :-)

There was a spurious --record left in %install, I have rename the lib dir
location following the usual name (it is easier to follow).

I have added more documentation, IMHO there is no reason not to include the
whole Doc directory. For Tools it would be nice to have the cython-mode.el as
a sub-package but clearly it is not required. :-)

Regarding the egg-info I see that you still using F-8. :-)

Comment 8 Neal Becker 2008-04-14 15:08:44 UTC
Great!  Please see:
http://nbecker.fedorapeople.org/Cython-0.9.6.13.1-3.fc8.src.rpm
(same as your patch)

Comment 9 José Matos 2008-04-14 15:37:29 UTC
Review for release 3.fc9:
* RPM name is OK
* Source Cython-0.9.6.13.1.tar.gz is the same as upstream
* This is the latest version
* Builds fine in mock
* rpmlint looks OK
* File list looks OK
* License is OK

(31 checks have been run)

APPROVED

Comment 10 Neal Becker 2008-04-14 16:53:37 UTC
New Package CVS Request
=======================
Package Name: Cython
Short Description: A language for writing Python extension modules
Owners: nbecker
Branches: F-7 F-8
InitialCC: 
Cvsextras Commits: yes

Comment 11 Kevin Fenzi 2008-04-14 19:41:50 UTC
cvs done.

Comment 12 José Matos 2008-06-08 18:34:08 UTC
*** Bug 450455 has been marked as a duplicate of this bug. ***

Comment 13 Steve Traylen 2011-04-08 16:57:41 UTC
Package Change Request
======================
Package Name: Cython
New Branches: el5 el6
Owners: stevetraylen


Following the comments in #612776

I'd like to create EPEL branches.

Steve.

Comment 14 Steve Traylen 2011-04-08 16:58:34 UTC
To add a hyperlink Bug 612776

Comment 15 Neal Becker 2011-04-08 17:06:48 UTC
Fine with me.  I don't have resources to maintain them though - it's all yours.

Comment 16 Jason Tibbitts 2011-04-08 19:41:09 UTC
Git done (by process-git-requests).

Comment 17 Fedora Update System 2011-04-08 21:25:26 UTC
Cython-0.14.1-3.el5 has been submitted as an update for Fedora EPEL 5.
https://admin.fedoraproject.org/updates/Cython-0.14.1-3.el5

Comment 18 Fedora Update System 2011-04-08 21:25:39 UTC
Cython-0.14.1-3.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/Cython-0.14.1-3.el6

Comment 19 Fedora Update System 2011-04-24 21:58:24 UTC
Cython-0.14.1-3.el5 has been pushed to the Fedora EPEL 5 stable repository.

Comment 20 Fedora Update System 2011-04-24 21:59:01 UTC
Cython-0.14.1-3.el6 has been pushed to the Fedora EPEL 6 stable repository.