Bug 480860 - Review Request: timespan - A tool that performs date-based time calculations
Summary: Review Request: timespan - A tool that performs date-based time calculations
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Lubomir Rintel
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2009-01-20 22:54 UTC by Fabian Affolter
Modified: 2009-02-20 11:51 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2009-02-20 11:51:31 UTC
Type: ---
Embargoed:
lkundrak: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Fabian Affolter 2009-01-20 22:54:05 UTC
Spec URL: http://fab.fedorapeople.org/packages/SRPMS/timespan.spec
SRPM URL: http://fab.fedorapeople.org/packages/SRPMS/timespan-2.1-1.fc9.src.rpm

Project URL: https://sourceforge.net/projects/timespan/

Description:
timespan performs date-based time calculations. It can calculate days,
hours, minutes, or seconds between 1 Jan 1970 and the specified date
or between two specified dates. 

Koji scratch build:
http://koji.fedoraproject.org/koji/taskinfo?taskID=1070012

rpmlint output:
[fab@laptop024 i386]$ rpmlint timespan*
2 packages and 0 specfiles checked; 0 errors, 0 warnings.

[fab@laptop024 SRPMS]$ rpmlint timespan-2.1-1.fc9.src.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

Comment 1 manuel wolfshant 2009-01-27 01:21:52 UTC
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

Comment 2 Fabian Affolter 2009-01-27 12:35:13 UTC
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.

Comment 3 manuel wolfshant 2009-01-27 12:53:19 UTC
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...

Comment 4 Petr Sklenar 2009-01-28 15:45:55 UTC
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.

Comment 5 Fabian Affolter 2009-01-29 13:39:28 UTC
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

Comment 6 Lubomir Rintel 2009-01-29 18:03:33 UTC
(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

Comment 7 Fabian Affolter 2009-01-29 18:18:50 UTC
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:

Comment 8 Kevin Fenzi 2009-01-30 06:27:55 UTC
cvs done.

Comment 9 Fedora Update System 2009-02-05 02:12:28 UTC
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

Comment 10 Fedora Update System 2009-02-05 02:19:55 UTC
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


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