Bug 478660 - Review Request: dateshift - A date/time test tool
Summary: Review Request: dateshift - A date/time test tool
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Tom "spot" Callaway
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2009-01-02 23:42 UTC by Fabian Affolter
Modified: 2009-02-20 11:42 UTC (History)
3 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2009-02-20 11:42:10 UTC
Type: ---
Embargoed:
tcallawa: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Fabian Affolter 2009-01-02 23:42:59 UTC
Spec URL: http://fab.fedorapeople.org/packages/SRPMS/dateshift.spec
SRPM URL: http://fab.fedorapeople.org/packages/SRPMS/dateshift-1.1-1.fc9.src.rpm

Project URL: http://www.hornby.dsl.pipex.com/dateshift/

Description:
Dateshift is a tool for shifting date/time. In particular it allows 
a process or set of process to run with a different clock setting,
without having to change the system clock.

This is useful because changing the system clock requires root privilege
and is inconvenient on shared machines. 

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

rpmlint output:
[fab@laptop024 i386]$ rpmlint dateshift*
dateshift.i386: W: shared-lib-calls-exit /usr/lib/libdsintercept.so.0.0.0 exit
dateshift-devel.i386: W: no-documentation
3 packages and 0 specfiles checked; 0 errors, 2 warnings.

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

I reported to upstream about the rpmlint warning.  But so far I have no answer.

Comment 1 Tom "spot" Callaway 2009-01-21 22:57:30 UTC
Review
=======
Good:

- rpmlint checks return:
dateshift.x86_64: W: shared-lib-calls-exit /usr/lib64/libdsintercept.so.0.0.0 exit.5
dateshift-devel.x86_64: W: no-documentation

You should continue trying to get upstream to resolve this, but I won't block this package review on it.

- package meets naming guidelines
- package meets packaging guidelines
- spec file legible, in am. english
- source matches upstream (0bcc71038686a1afdd0713b01eb81e7652634fa6)
- license text in %doc
- package compiles on devel (x86_64)
- no missing BR
- no unnecessary BR
- no locales
- not relocatable
- owns all directories that it creates
- no duplicate files
- permissions ok
- %clean ok
- macro use consistent
- code, not content
- no need for -docs
- nothing in %doc affects runtime
- no need for .desktop file 

Bad:

- License tag (GPLv2) is wrong. License tag should be GPLv2+.

*****
Please fix the license tag before you commit to cvs, but this package is approved.

Comment 2 Fabian Affolter 2009-01-22 08:19:43 UTC
Thanks Tom for the review.

Comment 3 Fabian Affolter 2009-01-22 08:21:22 UTC
New Package CVS Request
=======================
Package Name: dateshift
Short Description: A date/time test tool
Owners: fab
Branches: F-9 F-10
InitialCC:

Comment 4 Kevin Fenzi 2009-01-23 23:32:43 UTC
cvs done.

Comment 5 Fedora Update System 2009-01-27 01:51:04 UTC
dateshift-1.1-2.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 dateshift'.  You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F10/FEDORA-2009-0996

Comment 6 Fedora Update System 2009-01-27 01:53:57 UTC
dateshift-1.1-2.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 dateshift'.  You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F9/FEDORA-2009-1024


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