https://bugzilla.redhat.com/show_bug.cgi?id=1067003
Petr Pisar ppi...@redhat.com changed:
What|Removed |Added
Flags|fedora-review? |fedora-review+
---
https://bugzilla.redhat.com/show_bug.cgi?id=1067003
--- Comment #9 from Petr Pisar ppi...@redhat.com ---
The standalone spec file and file from source RPM differ. I will use the
standalone one for this review.
The URL and Source0 are usable. Ok.
The source archive is original (SHA-256:
https://bugzilla.redhat.com/show_bug.cgi?id=1067003
--- Comment #10 from Denis Fateyev de...@fateyev.com ---
Fixed the long description and adjusted license tag.
Updated version:
http://www.fateyev.com/RPMS/Fedora20/testing/perl-Time-ParseDate.spec
--
You are receiving this mail because:
You
https://bugzilla.redhat.com/show_bug.cgi?id=1067003
--- Comment #8 from Denis Fateyev de...@fateyev.com ---
Removed `%defattr`. Recent Koji scratch builds:
https://koji.fedoraproject.org/koji/taskinfo?taskID=7093794 (Rawhide)
https://koji.fedoraproject.org/koji/taskinfo?taskID=7093788 (EPEL 6)
https://bugzilla.redhat.com/show_bug.cgi?id=1067003
Paul Howarth p...@city-fan.org changed:
What|Removed |Added
CC||p...@city-fan.org
https://bugzilla.redhat.com/show_bug.cgi?id=1067003
--- Comment #4 from Denis Fateyev de...@fateyev.com ---
Note: this package (`perl-Time-ParseDate`) replaces obsoleted package
`perl-Time-modules`. The reviewer should consider
https://fedoraproject.org/wiki/Package_Renaming_Process
Finally,
https://bugzilla.redhat.com/show_bug.cgi?id=1067003
--- Comment #5 from Xavier Bachelot xav...@bachelot.org ---
The package looks good, there are only a couple mostly cosmetic issues :
- blank line at top of spec.
- it would be clearer to not mix Requires and BuildRequires. Move the only
https://bugzilla.redhat.com/show_bug.cgi?id=1067003
--- Comment #6 from Denis Fateyev de...@fateyev.com ---
Fixed 1 and 2. As for `%defattr`, I'm also planning to package for RHEL5, and
kept it since it doesn't break anything on newer versions.
https://bugzilla.redhat.com/show_bug.cgi?id=1067003
Xavier Bachelot xav...@bachelot.org changed:
What|Removed |Added
CC|
https://bugzilla.redhat.com/show_bug.cgi?id=1067003
Petr Pisar ppi...@redhat.com changed:
What|Removed |Added
Status|NEW |ASSIGNED
https://bugzilla.redhat.com/show_bug.cgi?id=1067003
--- Comment #2 from Petr Pisar ppi...@redhat.com ---
This package looks like a replacement for perl-Time-modules. See
http://search.cpan.org/~muir/Time-modules/ and
http://search.cpan.org/~muir/Time-ParseDate/
If this is so, you have to
https://bugzilla.redhat.com/show_bug.cgi?id=1067003
Denis Fateyev de...@fateyev.com changed:
What|Removed |Added
CC|
https://bugzilla.redhat.com/show_bug.cgi?id=1067003
--- Comment #1 from Denis Fateyev de...@fateyev.com ---
Description is a bit bogus (auto-imported from CPAN), here is a more
appropriate and shorten version:
This module recognizes various absolute and relative date/time formats. There
can be
13 matches
Mail list logo