Bug 254018
| Summary: | Review Request: rubygem-fastthread - optimized thread.rb replacement | ||
|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Scott Seago <sseago> |
| Component: | Package Review | Assignee: | David Lutterkort <lutter> |
| Status: | CLOSED RAWHIDE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
| Severity: | medium | Docs Contact: | |
| Priority: | medium | ||
| Version: | rawhide | CC: | fedora-package-review, hbrock, notting, vanmeeuwen+fedora |
| Target Milestone: | --- | Flags: | lutter:
fedora-review+
kevin: 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: | 2007-11-01 03:51:37 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: | |||
| Bug Depends On: | |||
| Bug Blocks: | 254015 | ||
|
Description
Scott Seago
2007-08-23 17:19:59 UTC
OK - Package name
OK - License tag is correct and licenses are approved
OK - Specfile name
OK - Specfile is legible
OK - No prebuilt binaries included
OK - BuildRoot value (one of the recommended values)
OK - PreReq not used
OK - Source md5sum matches upstream
OK - No hardcoded pathnames
OK - Package owns all the files it installs
OK - Package requires create needed unowned directories
OK - Package builds successfully on i386 and x86_64 (mock)
OK - BuildRequires sufficient
OK - File permissions set properly
OK - Macro usage is consistent
OK - rpmlint output. rpmlint complains about missing docs, but that's an
upstream problem
OK - Package is named rubygem-%{gemname}
OK - Source points to full URL of gem
OK - Package version identical with gem version
OK - Package Requires and BuildRequires rubygems
OK - Package provides rubygem(%{gemname}) = %version
OK - Package requires gem dependencies correctly
OK - %prep and %build are empty
OK - %gemdir defined properly, and gem installed into it
OK - Package owns its directories under %gemdir
OK - No arch specific content in %{gemdir}
OK - Defines ruby_sitearch from rbconfig
OK - arch specific content moved to %{ruby_sitearch}
BAD - License info isn't given explicitly, only inferred from the fact that
fastthread is part of mongrel and falls under mongrel licensing (not something
to fix for this review, but to point out to upstream at some point)
Package doesn't create a -debuginfo; but given the gyrations of building
rubygems, that can't be done sanely.
APPROVED
Please follow http://fedoraproject.org/wiki/CVSAdminProcedure and import
the package. Close this bug as RAWHIDE once it's been successfully imported
and built.
New Package CVS Request ======================= Package Name: rubygem-fastthread Short Description: Optimized replacement for thread.rb primitives Owners: sseago Branches: F-7, EL-5 InitialCC: Cvsextras Commits: No cvs done. Package Change Request ====================== Package Name: rubygem-fastthread New Branches: EL-4 Owners: sseago, kanarip Approved by sseago in private email cvs done. |