Bug 1001984
| Summary: | Review Request: rubygem-text - Collection of text algorithms | ||
|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Mamoru TASAKA <mtasaka> |
| Component: | Package Review | Assignee: | Miroslav Suchý <msuchy> |
| Status: | CLOSED NEXTRELEASE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
| Severity: | medium | Docs Contact: | |
| Priority: | medium | ||
| Version: | rawhide | CC: | msuchy, mtasaka, notting |
| Target Milestone: | --- | Flags: | msuchy:
fedora-review+
gwync: 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: | 2013-09-26 15:11:42 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
Mamoru TASAKA
2013-08-28 09:17:10 UTC
Taking *I* would include in -doc:
%{gem_instdir}/Rakefile
%{gem_instdir}/test/
Can you please include
https://raw.github.com/threedaymonk/text/master/COPYING.txt
as SOURCE1?
Otherwise it looks good.
Thank you for initial comments: (In reply to Miroslav Suchý from comment #2) > *I* would include in -doc: > %{gem_instdir}/Rakefile > %{gem_instdir}/test/ - For now I decided not to include this. > Can you please include > https://raw.github.com/threedaymonk/text/master/COPYING.txt > as SOURCE1? - Added this (note that in Source0 I renamed this so as not to make files' names duplicate when multiple srpm are expanded) > Otherwise it looks good. http://mtasaka.fedorapeople.org/Review_request/rubygem-text/rubygem-text.spec http://mtasaka.fedorapeople.org/Review_request/rubygem-text/rubygem-text-1.2.3-2.fc.src.rpm * Fri Aug 30 2013 Mamoru TASAKA <mtasaka> - 1.2.3-2 - Include license text APPROVED Thank you! New Package SCM Request ======================= Package Name: rubygem-text Short Description: Collection of text algorithms Owners: mtasaka Branches: f18 f19 f20 InitialCC: Git done (by process-git-requests). Ping? Can we close this bug now? I see that you submited it to current Fedora: https://admin.fedoraproject.org/updates/FEDORA-2013-16543/rubygem-text-1.2.3-2.fc20 It is good habit to put in that form number of this BZ and bodhi will then automatically switch this BZ to ON_QA and Close it as the package hit stable. Also please add you package to: https://fedoraproject.org/wiki/Upstream_release_monitoring#Q_-_W so you are notified when new version is released. Closing. Thank you for review. I am using my own check script for upstream monitoring. (By the way, adding bugzilla number to bodhi adds the comment which tells that if any issues are found on the submitted packages, the issue should be reported on _this_ ticket, which is not proper. The issue should be reported should be reported against newly created component.) |