[Bug 226250] Merge Review: perl-DateManip

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Merge Review: perl-DateManip


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


steve@xxxxxxxxx changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|fedora-review?              |fedora-review+




------- Additional Comments From steve@xxxxxxxxx  2007-03-20 13:56 EST -------
APPROVED

* Source matches CPAN.
* License is correct.
* Package name is correct.
* Summary, Description, etc. all look good (except the URL - see below).
* Builds fine in mock.

The URL is wrong, but that's the only problem I found.  There are a few other
issues, like a broken changelog, and a tab on one line that makes rpmlint
complain.  It wouldn't hurt my feelings if you applied the attached patch.  (I
guessed at the proper changelog version rather than looking through CVS.)

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]