[Bug 478614] Review Request: python-transitfeed - Google Transit Feed Specification library and tools
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=478614 --- Comment #10 from Fedora Update System upda...@fedoraproject.org 2009-01-21 16:37:14 EDT --- python-transitfeed-1.1.9-4.fc10 has been pushed to the Fedora 10 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 478614] Review Request: python-transitfeed - Google Transit Feed Specification library and tools
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=478614 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 478614] Review Request: python-transitfeed - Google Transit Feed Specification library and tools
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=478614 --- Comment #11 from Fedora Update System upda...@fedoraproject.org 2009-01-21 16:39:28 EDT --- python-transitfeed-1.1.9-4.fc9 has been pushed to the Fedora 9 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 478614] Review Request: python-transitfeed - Google Transit Feed Specification library and tools
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=478614 --- Comment #8 from Fedora Update System upda...@fedoraproject.org 2009-01-16 22:10:46 EDT --- python-transitfeed-1.1.9-4.fc9 has been submitted as an update for Fedora 9. http://admin.fedoraproject.org/updates/python-transitfeed-1.1.9-4.fc9 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 478614] Review Request: python-transitfeed - Google Transit Feed Specification library and tools
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=478614 --- Comment #9 from Fedora Update System upda...@fedoraproject.org 2009-01-16 22:10:49 EDT --- python-transitfeed-1.1.9-4.fc10 has been submitted as an update for Fedora 10. http://admin.fedoraproject.org/updates/python-transitfeed-1.1.9-4.fc10 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 478614] Review Request: python-transitfeed - Google Transit Feed Specification library and tools
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=478614 Kevin Fenzi ke...@tummy.com changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #6 from Kevin Fenzi ke...@tummy.com 2009-01-04 15:21:28 EDT --- cvs done. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 478614] Review Request: python-transitfeed - Google Transit Feed Specification library and tools
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=478614 --- Comment #7 from Ian Weller ianwel...@gmail.com 2009-01-04 15:27:01 EDT --- Thanks, Kevin. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 478614] Review Request: python-transitfeed - Google Transit Feed Specification library and tools
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=478614 --- Comment #3 from Ian Weller ianwel...@gmail.com 2009-01-04 00:22:55 EDT --- Spec: http://ianweller.fedorapeople.org/SRPMS/python-transitfeed/1.1.9-3/python-transitfeed.spec SRPM: http://ianweller.fedorapeople.org/SRPMS/python-transitfeed/1.1.9-3/python-transitfeed-1.1.9-3.fc10.src.rpm * Sat Jan 03 2009 Ian Weller ianwel...@gmail.com 1.1.9-3 - Add a gfts- prefix to everything in /usr/bin to get rid of naming conflicts -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 478614] Review Request: python-transitfeed - Google Transit Feed Specification library and tools
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=478614 Jon Stanley jonstan...@gmail.com changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #4 from Jon Stanley jonstan...@gmail.com 2009-01-04 01:23:44 EDT --- Looks good to me. APPROVED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 478614] Review Request: python-transitfeed - Google Transit Feed Specification library and tools
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=478614 Ian Weller ianwel...@gmail.com changed: What|Removed |Added Flag||fedora-cvs? --- Comment #5 from Ian Weller ianwel...@gmail.com 2009-01-04 01:30:47 EDT --- Thanks, Jon. New Package CVS Request === Package Name: python-transitfeed Short Description: Google Transit Feed Specification library and tools Owners: ianweller Branches: F-9 F-10 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 478614] Review Request: python-transitfeed - Google Transit Feed Specification library and tools
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=478614 Jon Stanley jonstan...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||jonstan...@gmail.com AssignedTo|nob...@fedoraproject.org|jonstan...@gmail.com -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 478614] Review Request: python-transitfeed - Google Transit Feed Specification library and tools
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=478614 Jon Stanley jonstan...@gmail.com changed: What|Removed |Added Flag||fedora-review? --- Comment #1 from Jon Stanley jonstan...@gmail.com 2009-01-02 02:00:03 EDT --- First glance - Builds fine in mock, however /usr/bin/merge is actually owned by RCS: [jstan...@rugrat result]$ yum whatprovides /usr/bin/merge Loaded plugins: refresh-packagekit rcs-5.7-33.fc10.x86_64 : Revision Control System (RCS) file version management tools Repo: fedora Matched from: Filename: /usr/bin/merge Expect a full review soon. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 478614] Review Request: python-transitfeed - Google Transit Feed Specification library and tools
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=478614
--- Comment #2 from Jon Stanley jonstan...@gmail.com 2009-01-02 02:26:00 EDT
---
OK - Package meets naming and packaging guidelines
OK - Spec file matches base package name.
OK - Spec has consistant macro usage.
See Notes - Meets Packaging Guidelines.
OK - License
OK - License field in spec matches
OK - License file included in package
OK - Spec in American English
OK - Spec is legible.
OK - Sources match upstream md5sum:
$ md5sum transitfeed-1.1.9.tar.gz
/data/home/jstanley/Download/transitfeed-1.1.9.tar.gz
709ab37eaaa3b56e7ff5f874091aa390 transitfeed-1.1.9.tar.gz
709ab37eaaa3b56e7ff5f874091aa390
/data/home/jstanley/Download/transitfeed-1.1.9.tar.gz
N/A - Package needs ExcludeArch
OK - BuildRequires correct
N/A - Spec handles locales/find_lang
N/A - Package is relocatable and has a reason to be.
OK - Package has %defattr and permissions on files is good.
OK - Package has a correct %clean section.
OK - Package has correct buildroot
%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
OK - Package is code or permissible content.
N/A - Doc subpackage needed/used.
OK - Packages %doc files don't affect runtime.
N/A - Headers/static libs in -devel subpackage.
N/A - Spec has needed ldconfig in post and postun
N/A - .pc files in -devel subpackage/requires pkgconfig
N/A - .so files in -devel subpackage.
N/A - -devel package Requires: %{name} = %{version}-%{release}
N/A - .la files are removed.
N/A - Package is a GUI app and has a .desktop file
OK - Package compiles and builds on at least one arch.
OK - Package has no duplicate files in %files.
OK - Package doesn't own any directories other packages own.
OK - Package owns all the directories it creates.
OK - No rpmlint output.
OK - final provides and requires are sane:
(include output of for i in *rpm; do echo $i; rpm -qp --provides $i; echo =;
rpm -qp --requires $i; echo; done
manually indented after checking each line. I also remove the rpmlib junk and
anything provided by glibc.)
python-transitfeed-1.1.9-2.fc10.noarch.rpm
python-transitfeed = 1.1.9-2.fc10
---
/usr/bin/python
python(abi) = 2.5
pytz
SHOULD Items:
OK - Should build in mock.
OK - Should build on all supported archs
Didn't test - Should function as described.
OK - Should have sane scriptlets.
N/A - Should have subpackages require base package with fully versioned depend.
OK - Should have dist tag
OK - Should package latest version
N/A - check for outstanding bugs on package. (For core merge reviews)
Issues:
1. As noted previously, /usr/bin/merge is a file conflict with RCS. That's a
blocker :(
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
