Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
https://bugzilla.redhat.com/show_bug.cgi?id=435090
Summary: Review Request: Cython - This is a development version of Pyrex, a language for writing Python extension modules. Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: ndbecker2@gmail.com QAContact: extras-qa@fedoraproject.org CC: fedora-package-review@redhat.com,notting@redhat.com
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
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: Cython - This is a development version of Pyrex, a language for writing Python extension modules.
https://bugzilla.redhat.com/show_bug.cgi?id=435090
jamatos@fc.up.pt changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED
------- Additional Comments From jamatos@fc.up.pt 2008-04-02 08:15 EST ------- 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. :-)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: Cython - This is a development version of Pyrex, a language for writing Python extension modules.
https://bugzilla.redhat.com/show_bug.cgi?id=435090
------- Additional Comments From ndbecker2@gmail.com 2008-04-06 19:35 EST ------- OK, I copied to http://nbecker.fedorapeople.org/
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: Cython - This is a development version of Pyrex, a language for writing Python extension modules.
https://bugzilla.redhat.com/show_bug.cgi?id=435090
jamatos@fc.up.pt changed:
What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|nobody@fedoraproject.org |jamatos@fc.up.pt
------- Additional Comments From jamatos@fc.up.pt 2008-04-14 07:06 EST ------- Taking the review.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: Cython - This is a development version of Pyrex, a language for writing Python extension modules.
https://bugzilla.redhat.com/show_bug.cgi?id=435090
------- Additional Comments From jamatos@fc.up.pt 2008-04-14 07:52 EST ------- 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.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: Cython - This is a development version of Pyrex, a language for writing Python extension modules.
https://bugzilla.redhat.com/show_bug.cgi?id=435090
------- Additional Comments From ndbecker2@gmail.com 2008-04-14 08:20 EST ------- Please see http://nbecker.fedorapeople.org/Cython-0.9.6.13.1-2.fc8.src.rpm
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: Cython - This is a development version of Pyrex, a language for writing Python extension modules.
https://bugzilla.redhat.com/show_bug.cgi?id=435090
------- Additional Comments From jamatos@fc.up.pt 2008-04-14 10:07 EST ------- Created an attachment (id=302341) --> (https://bugzilla.redhat.com/attachment.cgi?id=302341&action=view) 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+).
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: Cython - This is a development version of Pyrex, a language for writing Python extension modules.
https://bugzilla.redhat.com/show_bug.cgi?id=435090
------- Additional Comments From jamatos@fc.up.pt 2008-04-14 10:21 EST ------- 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. :-)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: Cython - This is a development version of Pyrex, a language for writing Python extension modules.
https://bugzilla.redhat.com/show_bug.cgi?id=435090
------- Additional Comments From ndbecker2@gmail.com 2008-04-14 11:08 EST ------- Great! Please see: http://nbecker.fedorapeople.org/Cython-0.9.6.13.1-3.fc8.src.rpm (same as your patch)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: Cython - This is a development version of Pyrex, a language for writing Python extension modules.
https://bugzilla.redhat.com/show_bug.cgi?id=435090
jamatos@fc.up.pt changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-review+
------- Additional Comments From jamatos@fc.up.pt 2008-04-14 11:37 EST -------
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
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: Cython - This is a development version of Pyrex, a language for writing Python extension modules.
https://bugzilla.redhat.com/show_bug.cgi?id=435090
ndbecker2@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-cvs?
------- Additional Comments From ndbecker2@gmail.com 2008-04-14 12:53 EST ------- 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
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: Cython - This is a development version of Pyrex, a language for writing Python extension modules.
https://bugzilla.redhat.com/show_bug.cgi?id=435090
kevin@tummy.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs? |fedora-cvs+
------- Additional Comments From kevin@tummy.com 2008-04-14 15:41 EST ------- cvs done.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: Cython - This is a development version of Pyrex, a language for writing Python extension modules.
https://bugzilla.redhat.com/show_bug.cgi?id=435090
ndbecker2@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |NEXTRELEASE
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: Cython - This is a development version of Pyrex, a language for writing Python extension modules.
https://bugzilla.redhat.com/show_bug.cgi?id=435090
jamatos@fc.up.pt changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |michel.sylvan@gmail.com
------- Additional Comments From jamatos@fc.up.pt 2008-06-08 14:34 EST ------- *** Bug 450455 has been marked as a duplicate of this bug. ***
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=435090
Steve Traylen steve.traylen@cern.ch changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |steve.traylen@cern.ch Flag|fedora-cvs+ |fedora-cvs?
--- Comment #13 from Steve Traylen steve.traylen@cern.ch 2011-04-08 12:57:41 EDT --- 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.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=435090
--- Comment #14 from Steve Traylen steve.traylen@cern.ch 2011-04-08 12:58:34 EDT --- To add a hyperlink Bug 612776
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=435090
--- Comment #15 from Neal Becker ndbecker2@gmail.com 2011-04-08 13:06:48 EDT --- Fine with me. I don't have resources to maintain them though - it's all yours.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=435090
--- Comment #16 from Jason Tibbitts tibbs@math.uh.edu 2011-04-08 15:41:09 EDT --- Git done (by process-git-requests).
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=435090
--- Comment #17 from Fedora Update System updates@fedoraproject.org 2011-04-08 17:25:26 EDT --- 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
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=435090
--- Comment #18 from Fedora Update System updates@fedoraproject.org 2011-04-08 17:25:39 EDT --- 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
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=435090
--- Comment #19 from Fedora Update System updates@fedoraproject.org 2011-04-24 17:58:24 EDT --- Cython-0.14.1-3.el5 has been pushed to the Fedora EPEL 5 stable repository.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=435090
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Fixed In Version| |Cython-0.14.1-3.el5 Resolution|NEXTRELEASE |ERRATA
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=435090
--- Comment #20 from Fedora Update System updates@fedoraproject.org 2011-04-24 17:59:01 EDT --- Cython-0.14.1-3.el6 has been pushed to the Fedora EPEL 6 stable repository.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=435090
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Fixed In Version|Cython-0.14.1-3.el5 |Cython-0.14.1-3.el6
package-review@lists.fedoraproject.org