Bug 668098
| Summary: | Review Request: rubygem-tzinfo - Ruby library that uses the standard tz (Olson) database to provide daylight savings aware transformations between times in different time zones. | ||
|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Minnikhanov <minnikhanov> |
| Component: | Package Review | Assignee: | Mo Morsi <mmorsi> |
| Status: | CLOSED RAWHIDE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
| Severity: | medium | Docs Contact: | |
| Priority: | medium | ||
| Version: | rawhide | CC: | fedora-package-review, mmorsi, notting |
| Target Milestone: | --- | Flags: | mmorsi:
fedora-review+
j: fedora-cvs+ |
| Target Release: | --- | ||
| Hardware: | All | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Fixed In Version: | Doc Type: | Bug Fix | |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2011-01-26 17:47:49 UTC | Type: | --- |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | --- | Target Upstream Version: | |
| Embargoed: | |||
|
Description
Minnikhanov
2011-01-07 21:50:49 UTC
Update to upstream release # 0.3.24 January 15, 2011 Spec URL: http://dl.dropbox.com/u/14118661/rubygem-tzinfo.spec SRPM URL: http://dl.dropbox.com/u/14118661/rubygem-tzinfo-0.3.24-1.fc14.src.rpm koji scratch build successful: http://koji.fedoraproject.org/koji/taskinfo?taskID=2729534 Looks mostly good. The only comment I would say is if you could include a %check section and run 'rake test' in it. You will have to add a BuildRequies(check) for rubygem(rake) rpmlint looks good, koji looks good To run the test suite, simply add the following to the spec
%check
pushd %{buildroot}%{geminstdir}
rake test
You will also need to add "BuildRequires: rubygem(rake)"
Up to you though, this is not required for approval, and I've verified the test suite works against this rpm in any case.
The package looks to be Fedora compliant overall minus one nit, please remove the "rm -rf %{buildroot}" in the %install section.
APPROVED
Spec URL: http://dl.dropbox.com/u/14118661/rubygem-tzinfo.spec SRPM URL: http://dl.dropbox.com/u/14118661/rubygem-tzinfo-0.3.24-2.fc14.src.rpm koji scratch build successful: http://koji.fedoraproject.org/koji/taskinfo?taskID=2729534 Test pass. (In reply to comment #3) > To run the test suite, simply add the following to the spec > > %check > pushd %{buildroot}%{geminstdir} > rake test > > You will also need to add "BuildRequires: rubygem(rake)" + Fixed. > The package looks to be Fedora compliant overall minus one nit, please remove > the "rm -rf %{buildroot}" in the %install section. + Fixed. > > APPROVED I am very thankful for collaboration. New Package SCM Request ======================= Package Name: rubygem-tzinfo Short Description: Ruby library that uses the standard tz (Olson) database to provide daylight savings aware transformations between times in different time zones. Owners: minn Branches: InitialCC: Spec URL: http://dl.dropbox.com/u/14118661/rubygem-tzinfo.spec SRPM URL: http://dl.dropbox.com/u/14118661/rubygem-tzinfo-0.3.24-2.fc14.src.rpm koji scratch build successful: http://koji.fedoraproject.org/koji/taskinfo?taskID=2742209 Test pass. Comment 3 fixed. Git done (by process-git-requests). See that you push to rawhide and didn't request any other Fedora branches. Closing this out. Appreciate it. New Package SCM Request ======================= Package Name: rubygem-tzinfo Short Description: Ruby library that uses the standard tz (Olson) database to provide daylight savings aware transformations between times in different time zones. Owners: minn Branches: f15 InitialCC: (In reply to comment #9) > New Package SCM Request > ======================= > Package Name: rubygem-tzinfo > Short Description: Ruby library that uses the standard tz (Olson) database to > provide daylight savings aware transformations between times in different time > zones. > Owners: minn > Branches: f15 > InitialCC: Sorry. Not need. |