[Bug 1156619] Review Request: python-tzlocal - tzinfo object for the local timezone
https://bugzilla.redhat.com/show_bug.cgi?id=1156619 Fedora Update System changed: What|Removed |Added Fixed In Version|python-tzlocal-1.1.2-4.el7 |python-tzlocal-1.1.2-4.el6 --- Comment #28 from Fedora Update System --- python-tzlocal-1.1.2-4.el6 has been pushed to the Fedora EPEL 6 stable repository. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1156619] Review Request: python-tzlocal - tzinfo object for the local timezone
https://bugzilla.redhat.com/show_bug.cgi?id=1156619 Fedora Update System changed: What|Removed |Added Fixed In Version|python-tzlocal-1.1.2-3.fc21 |python-tzlocal-1.1.2-4.el7 --- Comment #27 from Fedora Update System --- python-tzlocal-1.1.2-4.el7 has been pushed to the Fedora EPEL 7 stable repository. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1156619] Review Request: python-tzlocal - tzinfo object for the local timezone
https://bugzilla.redhat.com/show_bug.cgi?id=1156619 --- Comment #26 from Fedora Update System --- python-tzlocal-1.1.2-4.el6 has been submitted as an update for Fedora EPEL 6. https://admin.fedoraproject.org/updates/python-tzlocal-1.1.2-4.el6 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1156619] Review Request: python-tzlocal - tzinfo object for the local timezone
https://bugzilla.redhat.com/show_bug.cgi?id=1156619 --- Comment #25 from Fedora Update System --- python-tzlocal-1.1.2-4.el7 has been submitted as an update for Fedora EPEL 7. https://admin.fedoraproject.org/updates/python-tzlocal-1.1.2-4.el7 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1156619] Review Request: python-tzlocal - tzinfo object for the local timezone
https://bugzilla.redhat.com/show_bug.cgi?id=1156619 --- Comment #24 from Jon Ciesla --- Git done (by process-git-requests). -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1156619] Review Request: python-tzlocal - tzinfo object for the local timezone
https://bugzilla.redhat.com/show_bug.cgi?id=1156619 Jon Ciesla changed: What|Removed |Added Flags|fedora-cvs? |fedora-cvs+ -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1156619] Review Request: python-tzlocal - tzinfo object for the local timezone
https://bugzilla.redhat.com/show_bug.cgi?id=1156619 Piotr Popieluch changed: What|Removed |Added Flags|fedora-cvs+ |fedora-cvs? -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1156619] Review Request: python-tzlocal - tzinfo object for the local timezone
https://bugzilla.redhat.com/show_bug.cgi?id=1156619 --- Comment #23 from Piotr Popieluch --- Package Change Request == Package Name: python-tzlocal New Branches: el6 epel7 Owners: piotrp -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1156619] Review Request: python-tzlocal - tzinfo object for the local timezone
https://bugzilla.redhat.com/show_bug.cgi?id=1156619 Fedora Update System changed: What|Removed |Added Fixed In Version|python-tzlocal-1.1.2-3.fc19 |python-tzlocal-1.1.2-3.fc21 --- Comment #22 from Fedora Update System --- python-tzlocal-1.1.2-3.fc21 has been pushed to the Fedora 21 stable repository. If problems still persist, please make note of it in this bug report. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1156619] Review Request: python-tzlocal - tzinfo object for the local timezone
https://bugzilla.redhat.com/show_bug.cgi?id=1156619 Fedora Update System changed: What|Removed |Added Fixed In Version|python-tzlocal-1.1.2-3.fc20 |python-tzlocal-1.1.2-3.fc19 --- Comment #21 from Fedora Update System --- python-tzlocal-1.1.2-3.fc19 has been pushed to the Fedora 19 stable repository. If problems still persist, please make note of it in this bug report. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1156619] Review Request: python-tzlocal - tzinfo object for the local timezone
https://bugzilla.redhat.com/show_bug.cgi?id=1156619 Fedora Update System changed: What|Removed |Added Fixed In Version||python-tzlocal-1.1.2-3.fc20 Resolution|NEXTRELEASE |ERRATA --- Comment #20 from Fedora Update System --- python-tzlocal-1.1.2-3.fc20 has been pushed to the Fedora 20 stable repository. If problems still persist, please make note of it in this bug report. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1156619] Review Request: python-tzlocal - tzinfo object for the local timezone
https://bugzilla.redhat.com/show_bug.cgi?id=1156619 --- Comment #19 from Fedora Update System --- python-tzlocal-1.1.2-3.fc19 has been submitted as an update for Fedora 19. https://admin.fedoraproject.org/updates/python-tzlocal-1.1.2-3.fc19 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1156619] Review Request: python-tzlocal - tzinfo object for the local timezone
https://bugzilla.redhat.com/show_bug.cgi?id=1156619 --- Comment #18 from Fedora Update System --- python-tzlocal-1.1.2-3.fc21 has been submitted as an update for Fedora 21. https://admin.fedoraproject.org/updates/python-tzlocal-1.1.2-3.fc21 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1156619] Review Request: python-tzlocal - tzinfo object for the local timezone
https://bugzilla.redhat.com/show_bug.cgi?id=1156619 --- Comment #17 from Fedora Update System --- python-tzlocal-1.1.2-3.fc20 has been submitted as an update for Fedora 20. https://admin.fedoraproject.org/updates/python-tzlocal-1.1.2-3.fc20 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1156619] Review Request: python-tzlocal - tzinfo object for the local timezone
https://bugzilla.redhat.com/show_bug.cgi?id=1156619 Piotr Popieluch changed: What|Removed |Added Status|MODIFIED|CLOSED Resolution|--- |NEXTRELEASE Last Closed||2014-10-27 17:00:42 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1156619] Review Request: python-tzlocal - tzinfo object for the local timezone
https://bugzilla.redhat.com/show_bug.cgi?id=1156619 --- Comment #16 from Fedora Update System --- python-tzlocal-1.1.2-2.fc19 has been submitted as an update for Fedora 19. https://admin.fedoraproject.org/updates/python-tzlocal-1.1.2-2.fc19 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1156619] Review Request: python-tzlocal - tzinfo object for the local timezone
https://bugzilla.redhat.com/show_bug.cgi?id=1156619 --- Comment #15 from Fedora Update System --- python-tzlocal-1.1.2-2.fc20 has been submitted as an update for Fedora 20. https://admin.fedoraproject.org/updates/python-tzlocal-1.1.2-2.fc20 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1156619] Review Request: python-tzlocal - tzinfo object for the local timezone
https://bugzilla.redhat.com/show_bug.cgi?id=1156619 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1156619] Review Request: python-tzlocal - tzinfo object for the local timezone
https://bugzilla.redhat.com/show_bug.cgi?id=1156619 --- Comment #14 from Fedora Update System --- python-tzlocal-1.1.2-2.fc21 has been submitted as an update for Fedora 21. https://admin.fedoraproject.org/updates/python-tzlocal-1.1.2-2.fc21 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1156619] Review Request: python-tzlocal - tzinfo object for the local timezone
https://bugzilla.redhat.com/show_bug.cgi?id=1156619 Jon Ciesla changed: What|Removed |Added Flags|fedora-cvs? |fedora-cvs+ -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1156619] Review Request: python-tzlocal - tzinfo object for the local timezone
https://bugzilla.redhat.com/show_bug.cgi?id=1156619 Piotr Popieluch changed: What|Removed |Added Flags||fedora-cvs? --- Comment #12 from Piotr Popieluch --- New Package SCM Request === Package Name: python-tzlocal Short Description: A Python module that tries to figure out what your local timezone is Upstream URL: https://github.com/regebro/tzlocal Owners: piotrp Branches: f19 f20 f21 InitialCC: piotrp -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1156619] Review Request: python-tzlocal - tzinfo object for the local timezone
https://bugzilla.redhat.com/show_bug.cgi?id=1156619 --- Comment #11 from Piotr Popieluch --- missed that one.. fixed, thanks. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1156619] Review Request: python-tzlocal - tzinfo object for the local timezone
https://bugzilla.redhat.com/show_bug.cgi?id=1156619 --- Comment #10 from Parag AN(पराग) --- and yes have the same summary for python-tzlocal and python3-tzlocal otherwise rpmlint will complain warning :-) -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1156619] Review Request: python-tzlocal - tzinfo object for the local timezone
https://bugzilla.redhat.com/show_bug.cgi?id=1156619 Parag AN(पराग) changed: What|Removed |Added Flags|fedora-review? |fedora-review+ --- Comment #9 from Parag AN(पराग) --- APPROVED this package. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1156619] Review Request: python-tzlocal - tzinfo object for the local timezone
https://bugzilla.redhat.com/show_bug.cgi?id=1156619 --- Comment #8 from Piotr Popieluch --- Updated SPEC + SRPM with new summary/description as suggested: Spec URL: http://vps533.directvps.nl/python-tzlocal.spec SRPM URL: http://vps533.directvps.nl/python-tzlocal-1.1.2-2.fc20.src.rpm -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1156619] Review Request: python-tzlocal - tzinfo object for the local timezone
https://bugzilla.redhat.com/show_bug.cgi?id=1156619 Parag AN(पराग) changed: What|Removed |Added Blocks|177841 (FE-NEEDSPONSOR) | Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=177841 [Bug 177841] Tracker: Review requests from new Fedora packagers who need a sponsor -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1156619] Review Request: python-tzlocal - tzinfo object for the local timezone
https://bugzilla.redhat.com/show_bug.cgi?id=1156619 --- Comment #7 from Parag AN(पराग) --- Please provide updated SPEC and SRPM links here so I can approve this package. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1156619] Review Request: python-tzlocal - tzinfo object for the local timezone
https://bugzilla.redhat.com/show_bug.cgi?id=1156619 --- Comment #6 from Parag AN(पराग) --- This looks good now :) Review: + is OK - is Needs Work + Package built successful in mock (f22 x86_64) - rpmlint on generated rpms gave output python-tzlocal.src: W: spelling-error Summary(en_US) tzinfo -> tinfoil python-tzlocal.src: W: spelling-error %description -l en_US tzinfo -> tinfoil python3-tzlocal.noarch: W: spelling-error Summary(en_US) tzinfo -> tinfoil python3-tzlocal.noarch: W: summary-not-capitalized C tzinfo object for the local timezone python3-tzlocal.noarch: W: spelling-error %description -l en_US tzinfo -> tinfoil python-tzlocal.noarch: W: spelling-error Summary(en_US) tzinfo -> tinfoil python-tzlocal.noarch: W: spelling-error %description -l en_US tzinfo -> tinfoil 3 packages and 0 specfiles checked; 0 errors, 7 warnings. + Source verified with upstream as sha256sum srpm tarball: 4d9ddb8d5eab086e3a7c504c6e994ffa85df43e40da4d6be776218be051c677a upstream tarball: 4d9ddb8d5eab086e3a7c504c6e994ffa85df43e40da4d6be776218be051c677a + License is CCO and included in LICENSE.txt file + follow python packaging guidelines + rest also looks as per packaging guidelines. Suggestion: 1) Summary for main package python-tzlocal should be "A Python module that tries to figure out what your local timezone is". This is a nice summary as can be seen on github page 2) %description should contain some more information like This Python module returns a tzinfo object with the local timezone information. It requires pytz, and returns pytz tzinfo objects. This module attempts to fix a glaring hole in pytz, that there is no way to get the local timezone information, unless you know the zoneinfo name. 3) use above for python3-tzlocal %description Fix above and for this if you want you don't need to bump release number and package will be ready to be approved. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1156619] Review Request: python-tzlocal - tzinfo object for the local timezone
https://bugzilla.redhat.com/show_bug.cgi?id=1156619 --- Comment #5 from Piotr Popieluch --- bumped release number: Spec URL: http://vps533.directvps.nl/python-tzlocal.spec SRPM URL: http://vps533.directvps.nl/python-tzlocal-1.1.2-2.fc20.src.rpm -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1156619] Review Request: python-tzlocal - tzinfo object for the local timezone
https://bugzilla.redhat.com/show_bug.cgi?id=1156619 --- Comment #4 from Parag AN(पराग) --- Don't forget to add changes information in a new Changelog entry. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1156619] Review Request: python-tzlocal - tzinfo object for the local timezone
https://bugzilla.redhat.com/show_bug.cgi?id=1156619 --- Comment #3 from Parag AN(पराग) --- I can't see any updated package. Please be habitual with bumping release number. Generating new package and posting links of new SPEC and SRPM. We generally review SRPM which includes SPEC and not just SPEC. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1156619] Review Request: python-tzlocal - tzinfo object for the local timezone
https://bugzilla.redhat.com/show_bug.cgi?id=1156619 --- Comment #2 from Piotr Popieluch --- Thank you for commenting. I have added the rm commands in %install so that unit tests are not installed. I could not find a policy which mandates this but thought that it would be clean to not install those files as users will not use them. I got the idea from the python-setuptools.spec . If you think they need to be installed I will delete the rm lines. I've added comments to the spec to explain this. I have updated the spec and srpm: - deleted group tag - added license to python3 module - rewritten summary - wrapped description - added rm -rf *.egg-info to %prep - added comments to the rm commands in %install section -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1156619] Review Request: python-tzlocal - tzinfo object for the local timezone
https://bugzilla.redhat.com/show_bug.cgi?id=1156619 Parag AN(पराग) changed: What|Removed |Added Status|NEW |ASSIGNED CC||panem...@gmail.com Assignee|nob...@fedoraproject.org|panem...@gmail.com Flags||fedora-review? --- Comment #1 from Parag AN(पराग) --- 1) rpmlint on all generated rpms gave following common messages W: summary-not-capitalized C tzinfo object for the local timezone E: description-line-too-long C This Python module returns a tzinfo object with the local timezone information under Unix and Win-32. => you may want to add some word or re-word the summary also limit the line to 80 characters in description 2) I see some removals are done in %install section. Can you add comments to explain why you need to execute rm commands there? 3) you should in %prep rm -rf *.egg-info -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1156619] Review Request: python-tzlocal - tzinfo object for the local timezone
https://bugzilla.redhat.com/show_bug.cgi?id=1156619 Piotr Popieluch changed: What|Removed |Added Blocks||177841 (FE-NEEDSPONSOR) Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=177841 [Bug 177841] Tracker: Review requests from new Fedora packagers who need a sponsor -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
