Bug 1123546

Summary: Review Request: perl-DateTime-Incomplete - Representing partial dates and times
Product: [Fedora] Fedora Reporter: David Dick <ddick>
Component: Package ReviewAssignee: Petr Šabata <psabata>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: package-review, psabata
Target Milestone: ---Flags: psabata: fedora-review+
gwync: fedora-cvs+
Target Release: ---   
Hardware: All   
OS: Linux   
Fixed In Version: perl-DateTime-Incomplete-0.07-3.fc20 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2014-08-16 00:33:35 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
Bug Depends On:    
Bug Blocks: 1123583    

Description David Dick 2014-07-26 05:32:42 UTC
Spec URL: https://ddick.fedorapeople.org/packages/perl-DateTime-Incomplete.spec
SRPM URL: https://ddick.fedorapeople.org/packages/perl-DateTime-Incomplete-0.07-1.fc20.src.rpm
Description: Representing partial dates and times
Fedora Account System Username: ddick

Comment 2 Petr Šabata 2014-07-28 12:36:15 UTC
The standalone and SRPM spec files differ.

Comment 3 David Dick 2014-07-28 20:52:47 UTC

Comment 4 David Dick 2014-07-30 08:32:34 UTC
just a ping on this one?

Comment 5 Petr Šabata 2014-07-30 13:46:28 UTC
Missing BR: utf8 (t/13strftime.t)

The autodetected DateTime::Set version requirement will always overrule yours as RPM compares versions a bit differently (0.0901 > 0.15).  Given that the minimum required version is extremely old and DateTime::Set uses four-digit versions in Rawhide but only two-digit version in stable branches, to avoid any trouble I'd filter out the specific version dependency completely and require generic, unversioned DateTime::Set instead.

Comment 6 David Dick 2014-07-30 20:44:07 UTC
Fixed and uploaded.

Comment 7 Petr Šabata 2014-07-31 11:21:08 UTC
Good enough for Rawhide.  Approving.

If you want this in f20 or el*, filter out the autodetected `perl(DateTime::Set) >= 0.0901' dependency as mentioned in comment #5.

Comment 8 David Dick 2014-08-01 10:44:45 UTC
Thanks Petr!  I've updated the dependencies

New Package SCM Request
Package Name: perl-DateTime-Incomplete
Short Description: Representing partial dates and times
Upstream URL: http://search.cpan.org/dist/DateTime-Incomplete/
Owners: ddick
Branches: f20, f21, epel7, el6
InitialCC: perl-sig


Comment 9 Gwyn Ciesla 2014-08-01 12:14:50 UTC
Git done (by process-git-requests).

Comment 10 Fedora Update System 2014-08-01 23:17:00 UTC
perl-DateTime-Incomplete-0.07-1.fc20 has been submitted as an update for Fedora 20.

Comment 11 Fedora Update System 2014-08-01 23:23:12 UTC
perl-DateTime-Incomplete-0.07-1.el6 has been submitted as an update for Fedora EPEL 6.

Comment 12 Fedora Update System 2014-08-02 16:21:51 UTC
perl-DateTime-Incomplete-0.07-1.el6 has been pushed to the Fedora EPEL 6 testing repository.

Comment 13 Fedora Update System 2014-08-04 20:37:47 UTC
perl-DateTime-Incomplete-0.07-2.fc20 has been submitted as an update for Fedora 20.

Comment 14 Fedora Update System 2014-08-04 21:51:51 UTC
perl-DateTime-Incomplete-0.07-3.fc20 has been submitted as an update for Fedora 20.

Comment 15 Fedora Update System 2014-08-16 00:33:35 UTC
perl-DateTime-Incomplete-0.07-3.fc20 has been pushed to the Fedora 20 stable repository.  If problems still persist, please make note of it in this bug report.