Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
Summary: Review Request: perl-DateTime-Format-XSD - Format DateTime according to xsd:dateTime
https://bugzilla.redhat.com/show_bug.cgi?id=729649
Summary: Review Request: perl-DateTime-Format-XSD - Format DateTime according to xsd:dateTime Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: steve.traylen@cern.ch QAContact: extras-qa@fedoraproject.org CC: notting@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: ---
Spec URL: http://cern.ch/straylen/rpms/perl-DateTime-Format-XSD/perl-DateTime-Format-X... SRPM URL: http://cern.ch/straylen/rpms/perl-DateTime-Format-XSD/perl-DateTime-Format-X...
Description: XML Schema defines a usage profile which is a subset of the ISO8601 profile. This profile defines that the following is the only possible representation for a dateTime, despite all other options ISO provides.
F16 Scratch: http://koji.fedoraproject.org/koji/taskinfo?taskID=3263868
rpmlint is clean.
(Note to self, this is not required for review but bug #729644 to be resolved before EPEL can be done.)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=729649
Iain Arnell iarnell@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED AssignedTo|nobody@fedoraproject.org |iarnell@gmail.com
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=729649
Iain Arnell iarnell@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-review?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=729649
--- Comment #1 from Iain Arnell iarnell@gmail.com 2011-08-11 23:20:17 EDT --- Koji (success) http://koji.fedoraproject.org/koji/taskinfo?taskID=3267170
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=729649
--- Comment #2 from Iain Arnell iarnell@gmail.com 2011-08-11 23:30:37 EDT --- A nice simple package. The only minor issue I have is that the description is incomplete. How about replacing "the following" with 'YYYY-MM-DD"T"HH:MI:SS(Z|[+-]zh:zm)'?
The explicit Requires perl(DateTime::Format::ISO8601) is unnecessary in Fedora (at least since F15), but I guess you like to keep the spec the same for EPEL, so not really a problem.
APPROVED.
koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=3267170
Spec looks sane, clean and consistent; license is correct (GPL+ or Artistic); make test passes cleanly.
Source tarballs match upstream (sha1sum): 503389db4e50f5a6d678236a7b40c1d67c3fda6a DateTime-Format-XSD-0.2.tar.gz 503389db4e50f5a6d678236a7b40c1d67c3fda6a DateTime-Format-XSD-0.2.tar.gz.srpm
Final provides / requires are sane:
======> perl-DateTime-Format-XSD-0.2-1.fc17.noarch.rpm <====== ====> rpmlint 1 packages and 0 specfiles checked; 0 errors, 0 warnings. ====> provides perl-DateTime-Format-XSD = 0.2-1.fc17 ====> requires perl(base) perl(DateTime::Format::ISO8601) perl(DateTime::Format::ISO8601) perl(:MODULE_COMPAT_5.14.1) perl(strict) perl(warnings) ====> obsoletes ====> conflicts
======> perl-DateTime-Format-XSD-0.2-1.fc17.src.rpm <====== ====> rpmlint 1 packages and 0 specfiles checked; 0 errors, 0 warnings. ====> provides ====> requires perl(DateTime::Format::ISO8601) perl(ExtUtils::MakeMaker) perl(Test::More) ====> obsoletes ====> conflicts
====> mock install INFO: mock.py version 1.1.12 starting... State Changed: init plugins INFO: selinux enabled State Changed: start Mock Version: 1.1.12 INFO: Mock Version: 1.1.12 State Changed: lock buildroot INFO: installing package(s): perl-DateTime-Format-XSD-0.2-1.fc17.noarch.rpm INFO: Ignored option -c (probably due to merging -yc != -y -c)
================================================================================ Package Arch Version Repository Size ================================================================================ Installing: perl-DateTime-Format-XSD noarch 0.2-1.fc17 /perl-DateTime-Format-XSD-0.2-1.fc17.noarch 4.5 k Installing for dependencies: perl-Class-Factory-Util noarch 1.7-10.fc16 fedora 17 k perl-Class-Load noarch 0.06-5.fc16 fedora 12 k perl-Class-Singleton noarch 1.4-10.fc16 fedora 17 k perl-DateTime x86_64 1:0.7000-3.fc16 fedora 2.0 M perl-DateTime-Format-Builder noarch 0.8000-7.fc16 fedora 83 k perl-DateTime-Format-ISO8601 noarch 0.07-7.fc16 fedora 25 k perl-DateTime-Format-Strptime noarch 1.5000-4.fc16 fedora 29 k perl-List-MoreUtils x86_64 0.32-3.fc16 fedora 55 k perl-Math-Round noarch 0.06-10.fc16 fedora 13 k perl-Params-Validate x86_64 1.00-3.fc16 fedora 64 k perl-parent noarch 1:0.225-185.fc17 fedora 30 k
Transaction Summary ================================================================================ Install 12 Package(s)
Total size: 2.3 M Total download size: 2.3 M Installed size: 10 M http://mirror.nl.leaseweb.net/fedora/linux/development/rawhide/x86_64/os/Pac...: [Errno -1] Package does not match intended download. Suggestion: run yum --enablerepo=fedora clean metadata Trying other mirror. ftp://alviss.et.tudelft.nl/pub/fedora/linux/development/rawhide/x86_64/os/Packages/perl-Class-Load-0.06-5.fc16.noarch.rpm: [Errno -1] Package does not match intended download. Suggestion: run yum --enablerepo=fedora clean metadata Trying other mirror. http://mirror.nl.as6453.net/fedora/linux/development/rawhide/x86_64/os/Packa...: [Errno -1] Package does not match intended download. Suggestion: run yum --enablerepo=fedora clean metadata Trying other mirror. http://ftp.nluug.nl/pub/os/Linux/distr/fedora/linux/development/rawhide/x86_...: [Errno -1] Package does not match intended download. Suggestion: run yum --enablerepo=fedora clean metadata Trying other mirror. http://mirrors.nl.eu.kernel.org/fedora/development/rawhide/x86_64/os/Package...: [Errno -1] Package does not match intended download. Suggestion: run yum --enablerepo=fedora clean metadata Trying other mirror. http://nl.mirror.eurid.eu/fedora/linux/development/rawhide/x86_64/os/Package...: [Errno -1] Package does not match intended download. Suggestion: run yum --enablerepo=fedora clean metadata Trying other mirror. http://mirror.switch.ch/ftp/mirror/fedora/linux/development/rawhide/x86_64/o...: [Errno -1] Package does not match intended download. Suggestion: run yum --enablerepo=fedora clean metadata Trying other mirror. http://fedora.tu-chemnitz.de/pub/linux/fedora/linux/development/rawhide/x86_...: [Errno -1] Package does not match intended download. Suggestion: run yum --enablerepo=fedora clean metadata Trying other mirror. http://ftp.heanet.ie/pub/fedora/linux/development/rawhide/x86_64/os/Packages...: [Errno -1] Package does not match intended download. Suggestion: run yum --enablerepo=fedora clean metadata Trying other mirror.
Installed: perl-DateTime-Format-XSD.noarch 0:0.2-1.fc17
Dependency Installed: perl-Class-Factory-Util.noarch 0:1.7-10.fc16 perl-Class-Load.noarch 0:0.06-5.fc16 perl-Class-Singleton.noarch 0:1.4-10.fc16 perl-DateTime.x86_64 1:0.7000-3.fc16 perl-DateTime-Format-Builder.noarch 0:0.8000-7.fc16 perl-DateTime-Format-ISO8601.noarch 0:0.07-7.fc16 perl-DateTime-Format-Strptime.noarch 0:1.5000-4.fc16 perl-List-MoreUtils.x86_64 0:0.32-3.fc16 perl-Math-Round.noarch 0:0.06-10.fc16 perl-Params-Validate.x86_64 0:1.00-3.fc16 perl-parent.noarch 1:0.225-185.fc17
State Changed: unlock buildroot State Changed: end
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=729649
Iain Arnell iarnell@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-review? |fedora-review+
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=729649
--- Comment #3 from Iain Arnell iarnell@gmail.com 2011-08-11 23:58:00 EDT --- Oops. I missed that the package doesn't automatically provide perl(DateTime::Format::XSD). You should add this explicitly to the spec:
Provides: perl(DateTime::Format::XSD) = %{version}
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=729649
--- Comment #4 from Steve Traylen steve.traylen@cern.ch 2011-08-12 04:41:17 EDT --- Hi Iain,
Thanks for the review, I'll add the extra provides before I import.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=729649
Steve Traylen steve.traylen@cern.ch changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-cvs?
--- Comment #5 from Steve Traylen steve.traylen@cern.ch 2011-08-12 04:42:38 EDT --- New Package SCM Request ======================= Package Name: perl-DateTime-Format-XSD Short Description: Format DateTime according to xsd:dateTime Owners: stevetraylen Branches: f15 f16 el5 el6 InitialCC: perl-sig
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=729649
--- Comment #6 from Jon Ciesla limb@jcomserv.net 2011-08-12 08:38:46 EDT --- Git done (by process-git-requests).
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=729649
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |MODIFIED
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=729649
--- Comment #7 from Fedora Update System updates@fedoraproject.org 2011-08-12 09:14:57 EDT --- perl-DateTime-Format-XSD-0.2-1.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/perl-DateTime-Format-XSD-0.2-1.fc15
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=729649
--- Comment #8 from Fedora Update System updates@fedoraproject.org 2011-08-12 09:15:04 EDT --- perl-DateTime-Format-XSD-0.2-1.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/perl-DateTime-Format-XSD-0.2-1.fc16
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=729649
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|MODIFIED |ON_QA
--- Comment #9 from Fedora Update System updates@fedoraproject.org 2011-08-12 16:35:47 EDT --- perl-DateTime-Format-XSD-0.2-1.fc16 has been pushed to the Fedora 16 testing repository.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=729649
--- Comment #10 from Fedora Update System updates@fedoraproject.org 2011-08-13 15:21:42 EDT --- perl-DateTime-Format-XSD-0.2-1.el6, perl-DateTime-Format-ISO8601-0.07-7.el6 has been submitted as an update for Fedora EPEL 6. https://admin.fedoraproject.org/updates/perl-DateTime-Format-XSD-0.2-1.el6,p...
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=729649
--- Comment #11 from Fedora Update System updates@fedoraproject.org 2011-08-13 15:23:34 EDT --- perl-DateTime-Format-XSD-0.2-1.el5, perl-DateTime-Format-ISO8601-0.07-7.el5 has been submitted as an update for Fedora EPEL 5. https://admin.fedoraproject.org/updates/perl-DateTime-Format-XSD-0.2-1.el5,p...
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=729649
--- Comment #12 from Fedora Update System updates@fedoraproject.org 2011-08-23 14:50:04 EDT --- perl-DateTime-Format-XSD-0.2-1.fc15 has been pushed to the Fedora 15 stable repository.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=729649
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ON_QA |CLOSED Fixed In Version| |perl-DateTime-Format-XSD-0. | |2-1.fc15 Resolution| |ERRATA Last Closed| |2011-08-23 14:50:10
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=729649
--- Comment #13 from Fedora Update System updates@fedoraproject.org 2011-08-23 16:37:27 EDT --- perl-DateTime-Format-XSD-0.2-1.fc16 has been pushed to the Fedora 16 stable repository.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=729649
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Fixed In Version|perl-DateTime-Format-XSD-0. |perl-DateTime-Format-XSD-0. |2-1.fc15 |2-1.fc16
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=729649
--- Comment #14 from Fedora Update System updates@fedoraproject.org 2011-08-31 18:53:19 EDT --- perl-DateTime-Format-XSD-0.2-1.el5, perl-DateTime-Format-ISO8601-0.07-7.el5 has been pushed to the Fedora EPEL 5 stable repository.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=729649
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Fixed In Version|perl-DateTime-Format-XSD-0. |perl-DateTime-Format-XSD-0. |2-1.fc16 |2-1.el5
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=729649
--- Comment #15 from Fedora Update System updates@fedoraproject.org 2011-08-31 18:57:51 EDT --- perl-DateTime-Format-XSD-0.2-1.el6, perl-DateTime-Format-ISO8601-0.07-7.el6 has been pushed to the Fedora EPEL 6 stable repository.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=729649
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Fixed In Version|perl-DateTime-Format-XSD-0. |perl-DateTime-Format-XSD-0. |2-1.el5 |2-1.el6
package-review@lists.fedoraproject.org