Bug 1031527 - Review Request: datefudge - fakes the system date/time
Summary: Review Request: datefudge - fakes the system date/time
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Christopher Meng
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-11-18 08:16 UTC by Nikos Mavrogiannopoulos
Modified: 2013-12-14 03:04 UTC (History)
3 users (show)

Fixed In Version: datefudge-1.18-2.fc20
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-12-01 09:31:17 UTC
Type: ---
i: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Nikos Mavrogiannopoulos 2013-11-18 08:16:19 UTC
Spec URL: http://people.redhat.com/nmavrogi/fedora/datefudge.spec
SRPM URL: http://people.redhat.com/nmavrogi/fedora/datefudge-1.18-1.fc20.src.rpm

Description: This program (and preload library) fakes the system date so that programs think the wall clock is ... different. The faking is not complete; time-stamp on files are not affected in any way. This package is useful if you want to test the date handling of your programs without changing the system clock. This is particularly useful when checking the handling of programs
of certificate expiration/activation etc.

Fedora Account System Username: nmav

Comment 1 Christopher Meng 2013-11-18 08:30:44 UTC
First, do you want to support EL5?

Comment 2 Nikos Mavrogiannopoulos 2013-11-18 08:56:22 UTC
Hello,
 I really don't know. Is there a reason for not supporting it?

Comment 3 Christopher Meng 2013-11-18 11:35:23 UTC
(In reply to Nikos Mavrogiannopoulos from comment #2)
> Hello,
>  I really don't know. Is there a reason for not supporting it?

I don't know, but if you don't want to support EL5, you need to remove some lines as modern RPM doesn't need these lines.

I don't know which os you are using, but I don't think we need to care about EL5.(Just my opinion)

Waiting for your final answer.

Comment 4 Nikos Mavrogiannopoulos 2013-11-18 11:50:55 UTC
I have no particular reason to support EL5, so indeed I should drop anything unneeded.

Comment 5 Christopher Meng 2013-11-18 12:23:13 UTC
Ok.

So:

1. Remove these:

BuildRoot:	%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)

rm -rf %{buildroot} in %install

Whole %clean

%defattr(-,root,root,-)

2. URL better use PTS page:

http://packages.qa.debian.org/d/datefudge.html

3. Source0 better use CDN:

http://cdn.debian.net/debian/pool/main/d/datefudge/datefudge_1.18.tar.gz

with macro replaced, too.

4. I checked your patch.

INSTALL_PROGRAM := "install" should also be patched to 

INSTALL_PROGRAM := "install -p"

5. $(CFLAGS) should be inserted with %{optflags}:

http://fedoraproject.org/wiki/Packaging:Guidelines#Compiler_flags

You can define cflags like make CFLAGS="%{optflags}"

And $(LDFLAGS) also should be defined with %{__global_ldflags}

Comment 6 Nikos Mavrogiannopoulos 2013-11-18 13:04:26 UTC
Thanks. It's now updated based on your comments. I've uploaded a -2 version.

Comment 7 Christopher Meng 2013-11-18 13:26:26 UTC
Please paste the new URL.

Comment 8 Nikos Mavrogiannopoulos 2013-11-18 13:31:07 UTC
(In reply to Christopher Meng from comment #7)
> Please paste the new URL.

Spec URL: http://people.redhat.com/nmavrogi/fedora/datefudge.spec
SRPM URL: http://people.redhat.com/nmavrogi/fedora/datefudge-1.18-2.fc20.src.rpm

Comment 9 Christopher Meng 2013-11-19 09:58:36 UTC
datefudge.i686: E: incorrect-fsf-address /usr/share/doc/datefudge/COPYING

Please notify upstream about this.

-------------

PACKAGE APPROVED.

Comment 10 Nikos Mavrogiannopoulos 2013-11-19 10:32:12 UTC
New Package SCM Request
=======================
Package Name: datefudge
Short Description: fakes the system date/time
Owners: nmav
Branches: f19 f20

Comment 11 Nikos Mavrogiannopoulos 2013-11-19 10:34:54 UTC
(In reply to Christopher Meng from comment #9)
> datefudge.i686: E: incorrect-fsf-address /usr/share/doc/datefudge/COPYING
> 
> Please notify upstream about this.
> 
> -------------
> 
> PACKAGE APPROVED.

Notified, thanks.

Comment 12 Gwyn Ciesla 2013-11-19 13:11:57 UTC
Git done (by process-git-requests).

Comment 13 Fedora Update System 2013-11-19 13:45:47 UTC
datefudge-1.18-2.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/datefudge-1.18-2.fc20

Comment 14 Fedora Update System 2013-11-19 13:46:29 UTC
datefudge-1.18-2.fc19 has been submitted as an update for Fedora 19.
https://admin.fedoraproject.org/updates/datefudge-1.18-2.fc19

Comment 15 Fedora Update System 2013-11-19 21:54:03 UTC
datefudge-1.18-2.fc20 has been pushed to the Fedora 20 testing repository.

Comment 16 Fedora Update System 2013-12-01 09:31:17 UTC
datefudge-1.18-2.fc19 has been pushed to the Fedora 19 stable repository.

Comment 17 Fedora Update System 2013-12-14 03:04:02 UTC
datefudge-1.18-2.fc20 has been pushed to the Fedora 20 stable repository.


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