[Bug 480860] Review Request: timespan - A tool that performs date-based time calculations
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=480860 manuel wolfshant changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|nob...@fedoraproject.org|wo...@nobugconsulting.ro Flag||fedora-review? --- Comment #1 from manuel wolfshant 2009-01-26 20:21:52 EDT --- are you sure you want to submit this? the standard "date" command does everything this tool does, and much much more. info date -> examples for a quick grasp -- 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 480860] Review Request: timespan - A tool that performs date-based time calculations
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=480860 --- Comment #2 from Fabian Affolter 2009-01-27 07:35:13 EDT --- Usually I like the freedom to choose from several applications with similar functionality but your reason not to go on with this review is quite good. I don't want to waste your time. Let's drop this review request. At the other hand it's a simple package and a simple review. Maybe a person who is seeking a sponsor would like to review this. -- 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 480860] Review Request: timespan - A tool that performs date-based time calculations
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=480860 manuel wolfshant changed: What|Removed |Added Status|ASSIGNED|NEW CC||wo...@nobugconsulting.ro AssignedTo|wo...@nobugconsulting.ro|nob...@fedoraproject.org Flag|fedora-review? | --- Comment #3 from manuel wolfshant 2009-01-27 07:53:19 EDT --- I am sorry , I did not want to imply that there is a problem or something. It's just that I simply find it useless. I like your idea with leaving this review for someone seeking for sponsorship. Unassigning... -- 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 480860] Review Request: timespan - A tool that performs date-based time calculations
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=480860 Petr Sklenar changed: What|Removed |Added CC||pskle...@redhat.com --- Comment #4 from Petr Sklenar 2009-01-28 10:45:55 EDT --- Hello, This is my informal review. I cannot sponsor you as I'm not an approved packager. Your package and specfile seems OK for me: My review: + rpmlint output is clean. + The package is named according to the Package Naming Guidelines. + The spec file name matches the base package %{name}, in the format %{name}.spec. There is timespan.spec. + The package is licensed with a Fedora approved license and meets the Licensing Guidelines. + The License field in the package spec file matches the actual license. GPLv2+ + There is license in separate file in %doc : /usr/share/doc/timespan-2.1/COPYING + The spec file is written in American English. + The spec file for the package is legible. + The sources used to build the package match the upstream source, as provided in the spec URL (md5sum timespan-2.1.tar.gz c9d545eb5f617b29b3a634d7a0aec39a + The package successfully compiles and builds into binary rpms on at least one primary architecture. + All build dependencies are listed in BuildRequires, except for any that are listed in the exceptions section of the Packaging Guidelines. + The package owns all directories that it creates. + There isn't duplicate files in the %files listing. + Permissions on files are set properly. + The package has a %clean section, which contains rm -rf %{buildroot}. + The package consistently uses macros. + The package contains code, or permissible content. + Package doesn't own files or directories already owned by other packages. + At the beginning of %install, package runs rm -rf %{buildroot} (or $RPM_BUILD_ROOT). + All file names in rpm packages are valid UTF-8. architectures. + I did a limited test that the package functions as described, like '/usr/bin/timespan -l now' + Summary and description corresponds package's home. -- 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 480860] Review Request: timespan - A tool that performs date-based time calculations
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=480860 --- Comment #5 from Fabian Affolter 2009-01-29 08:39:28 EDT --- I don't need a sponsor but thanks for the review. You should mention this informal review to find a sponsor for yourself. For more details check out this page [1]. For RH employees is the 'seeking sponsor' procedure a bit different than for Fedora contributors. [1] https://fedoraproject.org/wiki/PackageMaintainers/Join#Get_a_Fedora_Account -- 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 480860] Review Request: timespan - A tool that performs date-based time calculations
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=480860 Lubomir Rintel changed: What|Removed |Added CC||lkund...@v3.sk AssignedTo|nob...@fedoraproject.org|lkund...@v3.sk Flag||fedora-review+ --- Comment #6 from Lubomir Rintel 2009-01-29 13:03:33 EDT --- (In reply to comment #5) > I don't need a sponsor but thanks for the review. You should mention this > informal review to find a sponsor for yourself. For more details check out > this page [1]. For RH employees is the 'seeking sponsor' procedure a bit > different than for Fedora contributors. > > [1] > https://fedoraproject.org/wiki/PackageMaintainers/Join#Get_a_Fedora_Account I think the sponsoring procedure is the same, just the CLA signing part is different. I think Peter's review was fairly exhaustive and package simple and correct as well. Thank you both! 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 480860] Review Request: timespan - A tool that performs date-based time calculations
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=480860 Fabian Affolter changed: What|Removed |Added Status|NEW |ASSIGNED Flag||fedora-cvs? --- Comment #7 from Fabian Affolter 2009-01-29 13:18:50 EDT --- New Package CVS Request === Package Name: timespan Short Description: A tool that performs date-based time calculations Owners: fab Branches: F-9 F-10 InitialCC: -- 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 480860] Review Request: timespan - A tool that performs date-based time calculations
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=480860 Kevin Fenzi changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #8 from Kevin Fenzi 2009-01-30 01:27:55 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 480860] Review Request: timespan - A tool that performs date-based time calculations
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=480860 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|ON_QA --- Comment #9 from Fedora Update System 2009-02-04 21:12:28 EDT --- timespan-2.1-1.fc10 has been pushed to the Fedora 10 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update timespan'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F10/FEDORA-2009-1251 -- 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 480860] Review Request: timespan - A tool that performs date-based time calculations
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=480860 --- Comment #10 from Fedora Update System 2009-02-04 21:19:55 EDT --- timespan-2.1-1.fc9 has been pushed to the Fedora 9 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing-newkey update timespan'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F9/FEDORA-2009-1328 -- 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 480860] Review Request: timespan - A tool that performs date-based time calculations
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=480860 Fabian Affolter changed: What|Removed |Added Status|ON_QA |CLOSED Resolution||NEXTRELEASE --- Comment #11 from Fabian Affolter 2009-02-20 06:51:31 EDT --- Was pushed to stable https://admin.fedoraproject.org/updates/F9/FEDORA-2009-1328 https://admin.fedoraproject.org/updates/F10/FEDORA-2009-1251 -- 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