[Bug 903829] New: Review Request: perl-Time-Interval - Convert seconds to human readable form - package-review

List overview All Threads
Download

newer

[Bug 903829] New: Review Request: perl-Time-Interval - Convert seconds to human readable form

older

[Bug 904328] New: Review Request:...

[Bug 903824] New: Review Request:...

Red Hat Bugzilla

24 Jan 2013 24 Jan '13

3:49 p.m.

Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=903829

Bug ID: 903829 Summary: Review Request: perl-Time-Interval - Convert seconds to human readable form Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: unspecified Reporter: long@rule.lv

Spec URL: http://unibackup.rule.lv/FedoraRPM/perl-Time-Interval.spec SRPM URL: http://unibackup.rule.lv/FedoraRPM/perl-Time-Interval-1.22-2.fc16.src.rpm Description: This is a rather simple perl module for dealing with time intervals. Among other things, this module can tell you the number of hours, minutes, and seconds elapsed between two dates. Fedora Account System Username: normunds

-- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=0f0D7v1kT4&a=cc_unsubscribe

Show replies by date

Red Hat Bugzilla

24 Jan 24 Jan

3:57 p.m.

New subject: [Bug 903829] Review Request: perl-Time-Interval - Convert seconds to human readable form

Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=903829

Normunds long@rule.lv changed:

What |Removed |Added ---------------------------------------------------------------------------- Blocks| |177841 (FE-NEEDSPONSOR)

--- Comment #1 from Normunds long@rule.lv --- Bug 903829, Bug 903826, Bug 903824 are my first Fedora packages, yet more to come. I checked them with both Mock and Koji for all Fedora releases (16, 17, 18, 19, rawhide).

-- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=AK5I1G1GFt&a=cc_unsubscribe

Red Hat Bugzilla

4:05 p.m.

New subject: [Bug 903829] Review Request: perl-Time-Interval - Convert seconds to human readable form

Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=903829

Normunds long@rule.lv changed:

What |Removed |Added ---------------------------------------------------------------------------- Alias| |perl-Time-Interval

-- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=ZG1bBH5eaC&a=cc_unsubscribe

Red Hat Bugzilla

26 Jan 26 Jan

4:21 a.m.

New subject: [Bug 903829] Review Request: perl-Time-Interval - Convert seconds to human readable form

Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=903829

--- Comment #2 from Normunds long@rule.lv --- All packages mentioned below were tested with rpmlint, mock (for i386 arch) and koji (16, 17, 18, 19, rawhide). These are my first packages for Fedora, so if you find something to improve in one of them, don't bother, I'll check other packages for reported problems.

Need sponsor.

bug 903824 perl-Convert-Age.spec bug 903826 perl-Net-Domain-TLD.spec bug 903829 perl-Time-Interval.spec bug 904328 perl-Config-ApacheFormat.spec bug 904329 perl-Data-Validate-Domain.spec bug 904330 perl-Data-Validate-IP.spec bug 904331 perl-Shell.spec

Thanks.

-- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=Ebq4vFmfCW&a=cc_unsubscribe

Red Hat Bugzilla

28 Jan 28 Jan

7:09 a.m.

New subject: [Bug 903829] Review Request: perl-Time-Interval - Convert seconds to human readable form

Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=903829

Petr Šabata psabata@redhat.com changed:

--- Comment #3 from Petr Šabata psabata@redhat.com --- Taking the review.

-- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=EcvdUAGJ7Q&a=cc_unsubscribe

Red Hat Bugzilla

7:23 a.m.

New subject: [Bug 903829] Review Request: perl-Time-Interval - Convert seconds to human readable form

Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=903829

--- Comment #4 from Petr Šabata psabata@redhat.com --- Issues:

Missing build-time dependencies: perl(Exporter) perl(Test)

Lines 30 and 31 are useless. None of the files has executable bits set. Line 32 is wrong. Modules aren't scripts and aren't supposed to contain shebangs. Line 43 may be removed. Done by rpmbuild.

Preferrably substitute command macros with simple calls.

The license is not mentioned anywhere in the upstream tarball, not even on the CPAN module webpage. Please, ask upstream for clarification.

-- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=8yoqHU33R9&a=cc_unsubscribe

Red Hat Bugzilla

4:01 p.m.

New subject: [Bug 903829] Review Request: perl-Time-Interval - Convert seconds to human readable form

Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=903829

--- Comment #5 from Normunds long@rule.lv --- Spec file and src package updated.

* Fixed dependencies. * Replaced macros with simple commands. * Removed useless lines * Sent query to Time-Interval maintainer. Interestingly, someone has asked this same question to him few years ago, although he looks active other Perl package maintainer, Time-Interval license still is not updated :)

New package overwritten in original location.

-- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=ZL3CYygvNE&a=cc_unsubscribe

Red Hat Bugzilla

29 Jan 29 Jan

8:54 a.m.

New subject: [Bug 903829] Review Request: perl-Time-Interval - Convert seconds to human readable form

Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=903829

Petr Šabata psabata@redhat.com changed:

What |Removed |Added ---------------------------------------------------------------------------- Blocks|177841 (FE-NEEDSPONSOR) |

--- Comment #6 from Petr Šabata psabata@redhat.com --- Removing FE-NEEDSPONSOR.

-- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=asXo8DJp0d&a=cc_unsubscribe

Red Hat Bugzilla

9:52 a.m.

New subject: [Bug 903829] Review Request: perl-Time-Interval - Convert seconds to human readable form

Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=903829

--- Comment #7 from Petr Šabata psabata@redhat.com --- Seems okay now (just the perl macro on the MODULE_COMPAT line...).

(In reply to comment #5)

...

Sent query to Time-Interval maintainer. Interestingly, someone has asked

this same question to him few years ago, although he looks active other Perl package maintainer, Time-Interval license still is not updated :)

I see https://rt.cpan.org/Public/Bug/Display.html?id=43024

I can't approve the review until the licensing issue is cleared up.

-- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=0jzyfkoJlG&a=cc_unsubscribe

Red Hat Bugzilla

31 Jan 31 Jan

4:04 p.m.

New subject: [Bug 903829] Review Request: perl-Time-Interval - Convert seconds to human readable form

Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=903829

--- Comment #8 from Normunds long@rule.lv --- Package updated, license is now included in upstream, other requested changes done.

Spec URL: http://unibackup.rule.lv/FedoraRPM/perl-Time-Interval.spec SRPM URL: http://unibackup.rule.lv/FedoraRPM/perl-Time-Interval-1.231-1.fc16.src.rpm

-- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=2XuyyGOL5o&a=cc_unsubscribe

Red Hat Bugzilla

1 Feb 1 Feb

3:22 a.m.

New subject: [Bug 903829] Review Request: perl-Time-Interval - Convert seconds to human readable form

Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=903829

--- Comment #9 from Petr Šabata psabata@redhat.com --- There's been a new release, 1.232, which removes the ._* files from the distribution.

Two more things: 1. Include the LICENSE file in your %doc 2. Your changelog entry is suddenly without a contact e-mail

-- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=b5LQOcCzEQ&a=cc_unsubscribe

Red Hat Bugzilla

3:54 a.m.

New subject: [Bug 903829] Review Request: perl-Time-Interval - Convert seconds to human readable form

Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=903829

--- Comment #10 from Normunds long@rule.lv --- Question: is contact e-mail mandatory? I decided that including e-mail in publicly available spec file would be perfect catch for spam bots.

Sorry for inconsistencies.

Thanks.

-- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=vp7bzrN7dT&a=cc_unsubscribe

Red Hat Bugzilla

5:14 a.m.

New subject: [Bug 903829] Review Request: perl-Time-Interval - Convert seconds to human readable form

Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=903829

--- Comment #11 from Petr Šabata psabata@redhat.com --- (In reply to comment #10)

...

Question: is contact e-mail mandatory? I decided that including e-mail in publicly available spec file would be perfect catch for spam bots.

Yes, it is. But don't worry too much, your e-mail is available in the publicly available git commits anyway :)

-- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=vabBlIPUUd&a=cc_unsubscribe

Red Hat Bugzilla

2 Feb 2 Feb

12:44 p.m.

New subject: [Bug 903829] Review Request: perl-Time-Interval - Convert seconds to human readable form

Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=903829

--- Comment #12 from Normunds long@rule.lv --- Big thanks for your support with getting response from upstream.

Updated to version 1.232, added obfuscated e-mail, added LICENSE to doc.

Spec URL: http://unibackup.rule.lv/FedoraRPM/perl-Time-Interval.spec SRPM URL: http://unibackup.rule.lv/FedoraRPM/perl-Time-Interval-1.232-1.fc16.src.rpm

Thanks.

-- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=GbQMaORngs&a=cc_unsubscribe

Red Hat Bugzilla

4 Feb 4 Feb

9:09 a.m.

New subject: [Bug 903829] Review Request: perl-Time-Interval - Convert seconds to human readable form

Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=903829

--- Comment #13 from Petr Šabata psabata@redhat.com --- Again, if you must use this format, enclose it with qw(< >) at least.

-- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=pJJGJmPckJ&a=cc_unsubscribe

Red Hat Bugzilla

10:24 a.m.

New subject: [Bug 903829] Review Request: perl-Time-Interval - Convert seconds to human readable form

Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=903829

--- Comment #14 from Normunds fedorapkg@rule.lv --- Spec file/srpm updated, added <> to e-mail.

-- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=W43ZPlKGH3&a=cc_unsubscribe

Red Hat Bugzilla

5 Feb 5 Feb

12:52 p.m.

New subject: [Bug 903829] Review Request: perl-Time-Interval - Convert seconds to human readable form

Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=903829

--- Comment #15 from Normunds fedorapkg@rule.lv --- Another reviewer suggested to use simpler e-mail obfuscation, so, to be consistent, I changed it for all packages.

-- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=rCeSSeTI4n&a=cc_unsubscribe

Red Hat Bugzilla

6 Feb 6 Feb

10:26 a.m.

New subject: [Bug 903829] Review Request: perl-Time-Interval - Convert seconds to human readable form

Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=903829

Petr Šabata psabata@redhat.com changed:

--- Comment #16 from Petr Šabata psabata@redhat.com --- Alright, approving.

-- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=9K95zgn4zT&a=cc_unsubscribe

Red Hat Bugzilla

1:41 p.m.

New subject: [Bug 903829] Review Request: perl-Time-Interval - Convert seconds to human readable form

Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=903829

Normunds fedorapkg@rule.lv changed:

What |Removed |Added ---------------------------------------------------------------------------- Flags| |fedora-cvs?

--- Comment #17 from Normunds fedorapkg@rule.lv --- Thanks for your help and support :)

-- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=UbrKblfYwj&a=cc_unsubscribe

Red Hat Bugzilla

1:46 p.m.

New subject: [Bug 903829] Review Request: perl-Time-Interval - Convert seconds to human readable form

Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=903829

--- Comment #18 from Normunds fedorapkg@rule.lv --- New Package SCM Request ======================= Package Name: perl-Time-Interval Short Description: Perl module that converts time intervals of days, hours, minutes, and seconds Owners: normunds psabata Branches: f16 f17 f18 InitialCC: perl-sig

-- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=5Egkft6OA8&a=cc_unsubscribe

Red Hat Bugzilla

2:06 p.m.

New subject: [Bug 903829] Review Request: perl-Time-Interval - Convert seconds to human readable form

Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=903829

--- Comment #19 from Jon Ciesla limburgher@gmail.com --- Git done (by process-git-requests).

-- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=oS8qCM8w3b&a=cc_unsubscribe

Red Hat Bugzilla

14 Feb 14 Feb

3:37 p.m.

New subject: [Bug 903829] Review Request: perl-Time-Interval - Convert seconds to human readable form

Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=903829

Fedora Update System updates@fedoraproject.org changed:

What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |MODIFIED

-- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=vIhvU75feB&a=cc_unsubscribe

Red Hat Bugzilla

3:37 p.m.

New subject: [Bug 903829] Review Request: perl-Time-Interval - Convert seconds to human readable form

Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=903829

--- Comment #20 from Fedora Update System updates@fedoraproject.org --- perl-Time-Interval-1.232-1.fc17 has been submitted as an update for Fedora 17. https://admin.fedoraproject.org/updates/perl-Time-Interval-1.232-1.fc17

-- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=H3zxxMACx5&a=cc_unsubscribe

Red Hat Bugzilla

4:14 p.m.

New subject: [Bug 903829] Review Request: perl-Time-Interval - Convert seconds to human readable form

Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=903829

--- Comment #21 from Fedora Update System updates@fedoraproject.org --- perl-Time-Interval-1.232-1.fc18 has been submitted as an update for Fedora 18. https://admin.fedoraproject.org/updates/perl-Time-Interval-1.232-1.fc18

-- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=AXayXdl6SR&a=cc_unsubscribe

Red Hat Bugzilla

25 Feb 25 Feb

8:29 p.m.

New subject: [Bug 903829] Review Request: perl-Time-Interval - Convert seconds to human readable form

Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=903829

--- Comment #22 from Fedora Update System updates@fedoraproject.org --- perl-Time-Interval-1.232-1.fc17 has been pushed to the Fedora 17 stable repository.

-- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=a8B4SyMfoC&a=cc_unsubscribe

Red Hat Bugzilla

8:40 p.m.

New subject: [Bug 903829] Review Request: perl-Time-Interval - Convert seconds to human readable form

Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=903829

--- Comment #23 from Fedora Update System updates@fedoraproject.org --- perl-Time-Interval-1.232-1.fc18 has been pushed to the Fedora 18 stable repository.

-- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=U5JlTVmWE1&a=cc_unsubscribe

Red Hat Bugzilla

3 Apr 3 Apr

11:17 a.m.

New subject: [Bug 903829] Review Request: perl-Time-Interval - Convert seconds to human readable form

https://bugzilla.redhat.com/show_bug.cgi?id=903829

Petr Šabata psabata@redhat.com changed:

-- 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

4128

Age (days ago)

4562

Last active (days ago)

package-review@lists.fedoraproject.org

26 comments

1 participants

tags (0)

participants (1)

Red Hat Bugzilla