Bug 823333
| Summary: | Review Request: rubygem-mixlib-cli - a ruby gem mixin for CLI support, including option parsing | ||
|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Jonas Courteau <rpms> |
| Component: | Package Review | Assignee: | Josef Stribny <jstribny> |
| Status: | CLOSED CURRENTRELEASE | QA Contact: | |
| Severity: | unspecified | Docs Contact: | |
| Priority: | unspecified | ||
| Version: | rawhide | CC: | hhorak, jdunn, jstribny, notting, package-review, vondruch |
| Target Milestone: | --- | Flags: | jstribny:
fedora-review+
gwync: fedora-cvs+ |
| Target Release: | --- | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| Whiteboard: | |||
| Fixed In Version: | Doc Type: | Bug Fix | |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2013-01-23 16:05:10 UTC | Type: | Bug |
| 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: | 823344, 823352 | ||
|
Description
Jonas Courteau
2012-05-20 23:51:59 UTC
* Remove Requires: ruby for other Ruby interpretations to be used with in f19
* Rakefile, NOTICE and README.rdoc are not require during runtime
- Please consider moving them into the -doc subpackage
* Please consider moving specs into the -doc subpackage rather than exclude them
* I would suggest you to exclude %{gem_cache} as it's not needed
* Tests print:
........You must supply -r!
Usage: /usr/local/bin/rspec (options)
-r
......
- Tests shouldn't print STDOUT like this. It could be fixed by something suggested on stackoverflow [1].
[1] http://stackoverflow.com/questions/1496019/suppresing-output-to-console-with-ruby
@Julian C. Dunn: Adding you to CC. Could you please take over this bug?
(In reply to comment #1) > * Remove Requires: ruby for other Ruby interpretations to be used with in f19 > > * Rakefile, NOTICE and README.rdoc are not require during runtime > - Please consider moving them into the -doc subpackage > > * Please consider moving specs into the -doc subpackage rather than exclude > them > > * I would suggest you to exclude %{gem_cache} as it's not needed All of the above have been corrected. > * Tests print: > > ........You must supply -r! > Usage: /usr/local/bin/rspec (options) > -r > ...... > > - Tests shouldn't print STDOUT like this. It could be fixed by something > suggested on stackoverflow [1]. I added a patch for this to the SRPM, though I don't know if it'll be accepted upstream -- it's a matter of taste and the fix is kind of dirty. But it will silence output to STDOUT on the tests. Please have a look: http://jdunn.fedorapeople.org/rubygem-mixlib-cli/rubygem-mixlib-cli.spec http://jdunn.fedorapeople.org/rubygem-mixlib-cli/rubygem-mixlib-cli-1.2.2-2.fc19.src.rpm * %{gem_instdir}/NOTICE and %{gem_instdir}/README.rdoc should be marked as %doc
Otherwise the package is fine. Rpmlint gave me no errors and it builds in Koji -> APPROVED. Please mark the above mentioned files as %doc before pushing. Thanks.
(In reply to comment #2) > I added a patch for this to the SRPM, though I don't know if it'll be > accepted upstream -- it's a matter of taste and the fix is kind of dirty. > But it will silence output to STDOUT on the tests. Julien, could you please always try to comment what is the patch about? Its purpose, its source etc. It is good to know if you submit the patch upstream, what is the status or if you taken the patch from upstream, when it can be dropped etc. (In reply to comment #4) > (In reply to comment #2) > > I added a patch for this to the SRPM, though I don't know if it'll be > > accepted upstream -- it's a matter of taste and the fix is kind of dirty. > > But it will silence output to STDOUT on the tests. > > Julien, could you please always try to comment what is the patch about? Its > purpose, its source etc. It is good to know if you submit the patch > upstream, what is the status or if you taken the patch from upstream, when > it can be dropped etc. Sure, no problem. Thanks for the advice. Package Change Request ====================== Package Name: rubygem-mixlib-cli New Branches: f16 f17 f18 Owners: jdunn InitialCC: Unretire package. Git done (by process-git-requests). rubygem-mixlib-cli-1.2.2-3.fc18 has been submitted as an update for Fedora 18. https://admin.fedoraproject.org/updates/rubygem-mixlib-cli-1.2.2-3.fc18 rubygem-mixlib-cli-1.2.2-3.fc17 has been submitted as an update for Fedora 17. https://admin.fedoraproject.org/updates/rubygem-mixlib-cli-1.2.2-3.fc17 rubygem-mixlib-cli-1.2.2-3.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/rubygem-mixlib-cli-1.2.2-3.fc16 rubygem-mixlib-cli-1.2.2-3.fc18 has been pushed to the Fedora 18 testing repository. rubygem-mixlib-cli-1.2.2-3.fc17 has been pushed to the Fedora 17 stable repository. rubygem-mixlib-cli-1.2.2-3.fc16 has been pushed to the Fedora 16 stable repository. rubygem-mixlib-cli-1.2.2-3.fc18 has been pushed to the Fedora 18 stable repository. |