Bug 435090 - Review Request: Cython - This is a development version of Pyrex, a language for writing Python extension modules.
Summary: Review Request: Cython - This is a development version of Pyrex, a language f...
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: José Matos
QA Contact: Fedora Extras Quality Assurance
: 450455 (view as bug list)
Depends On:
TreeView+ depends on / blocked
Reported: 2008-02-27 10:41 UTC by Neal Becker
Modified: 2011-04-24 21:59 UTC (History)
5 users (show)

Fixed In Version: Cython-0.14.1-3.el6
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2008-04-15 12:27:16 UTC
jamatos: fedora-review+
tibbs: fedora-cvs+

Attachments (Terms of Use)
Spec file patch with some fixes (1.33 KB, patch)
2008-04-14 14:07 UTC, José Matos
no flags Details | Diff

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-
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- 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
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

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> -
+- 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:
(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- 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)


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
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.


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.

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.

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.

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