Bug 226250 - Merge Review: perl-DateManip
Merge Review: perl-DateManip
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
All Linux
medium Severity medium
: ---
: ---
Assigned To: Steven Pritchard
Fedora Package Reviews List
Depends On:
  Show dependency treegraph
Reported: 2007-01-31 15:34 EST by Nobody's working on this, feel free to take it
Modified: 2007-11-30 17:11 EST (History)
2 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2007-06-27 22:58:49 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
steve: fedora‑review+

Attachments (Terms of Use)
spec patch (1.77 KB, patch)
2007-03-20 13:58 EDT, Steven Pritchard
no flags Details | Diff

  None (edit)
Description Nobody's working on this, feel free to take it 2007-01-31 15:34:40 EST
Fedora Merge Review: perl-DateManip

Initial Owner: rnorwood@redhat.com
Comment 1 Steven Pritchard 2007-03-20 13:56:15 EDT

* 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.)
Comment 2 Steven Pritchard 2007-03-20 13:58:04 EDT
Created attachment 150514 [details]
spec patch
Comment 3 Robin Norwood 2007-03-20 15:03:39 EDT
applied patch to the spec file, built 5.44-3.
Comment 4 Steven Pritchard 2007-06-27 18:54:23 EDT
Can we close this?
Comment 5 Robin Norwood 2007-06-27 22:58:49 EDT
oh, good plan.

/me closes.

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