Bug 979749
Summary: | Review Request: rubygem-time-lord - Managing concepts of time and space in Ruby | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Rick Elrod <relrod> |
Component: | Package Review | Assignee: | Josef Stribny <jstribny> |
Status: | CLOSED WONTFIX | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | hhorak, jstribny, ktdreyer |
Target Milestone: | --- | Flags: | jstribny:
fedora-review?
|
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2014-05-12 03:22:10 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: | 979752 |
Description
Rick Elrod
2013-06-30 08:31:11 UTC
Taking. 1, "# I filed a bug on this upstream, but this works around it for now." In such cases. please always include the link to the upstream bug. 2, The package should BuildRequires: ruby(release) for >= f19 and BuildRequires: ruby(abi) for < f19 3, Please change BuildRequires of rubygem(timecop), rubygem(yard), rubygem(pry), ... to the new syntax: rubygem-timecop, rubygem-yard, rubygem-pry, ... 4, Don't use rake for running the test suite, ideally use MiniTest. See the guidelines [1] 5, Get rid of Bundler dependency as it's not needed and it's problematic to use in packages. 6, 'pry', 'coveralls', 'timecop' are listed as development dependencies... are they really needed for tests? I think if you delete those lines from the test suite, it will work without them. Less dependencies -> better. 7, Introduce -doc subpackage according to guidelines [1], separate files, mark %doc files, exclude gem cache and keep the rakefile: For main package: %files %dir %{gem_instdir} %{gem_libdir} %exclude %{gem_cache} %{gem_spec} %doc %{gem_instdir}/COPYRIGHT For -doc subpackage: %files doc %doc %{gem_instdir}/README.md %doc %{gem_docdir} %{gem_instdir}/Rakefile [1] https://fedoraproject.org/wiki/Packaging:Ruby?rd=Packaging/Ruby (In reply to Josef Stribny from comment #2) > 1, "# I filed a bug on this upstream, but this works around it for now." > In such cases. please always include the link to the upstream bug. For reference, the URL was https://github.com/krainboltgreene/time-lord/issues/14 > 2, The package should BuildRequires: ruby(release) for >= f19 and > BuildRequires: ruby(abi) for < f19 Also, now that F18 is EOL, you can remove this and just use ruby(release). In addition to the above comments, you can remove the text about kramdown being absent from Fedora (it's present now), and change the URLs to HTTPS. Bumping out of my bug queue for now as I don't need this anymore. |