| Summary: | Review Request: python-urwidtrees - Tree Widget Container API for the urwid toolkit | ||
|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Tomas Tomecek <ttomecek> |
| Component: | Package Review | Assignee: | William Moreno <williamjmorenor> |
| Status: | CLOSED RAWHIDE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
| Severity: | medium | Docs Contact: | |
| Priority: | medium | ||
| Version: | rawhide | CC: | package-review, williamjmorenor |
| Target Milestone: | --- | Flags: | williamjmorenor:
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: | 2016-03-29 06:19:31 UTC | Type: | --- |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | --- | Target Upstream Version: | |
| Bug Depends On: | |||
| Bug Blocks: | 1293503 | ||
|
Description
Tomas Tomecek
2016-01-04 14:52:28 UTC
Any update? williamjmorenor's scratch build of python-urwidtrees-1.0.1.1-1.fc24.src.rpm for rawhide failed http://koji.fedoraproject.org/koji/taskinfo?taskID=12597310 Sorry for the late feedback but my build is failing in f23 and rawhide. ttomecek's scratch build of python-urwidtrees-1.0.1-1.fc22.src.rpm for rawhide failed http://koji.fedoraproject.org/koji/taskinfo?taskID=12647780 ttomecek's scratch build of python-urwidtrees-1.0.1-1.fc22.src.rpm for rawhide failed http://koji.fedoraproject.org/koji/taskinfo?taskID=12647826 ttomecek's scratch build of python-urwidtrees-1.0.1-1.fc22.src.rpm for rawhide completed http://koji.fedoraproject.org/koji/taskinfo?taskID=12647868 Oh, it was working for me locally but I haven't tried koji, my bad, sorry about that. Here's a successful koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=12647868 I have also updated spec and srpm at fp: https://ttomecek.fedorapeople.org/urwidtrees/python-urwidtrees.spec https://ttomecek.fedorapeople.org/urwidtrees/python-urwidtrees-1.0.1-1.fc22.src.rpm The package named according to the Package Naming Guidelines OK
The spec file name must match the base package %{name} OK
The package must be licensed with a Fedora approved license OK
The License field match the actual license OK
The spec file must be written in American English OK
The spec file for the package MUST be legible OK
The package build in rawhide OK
BuildRequires OK
No bundled libraries OK
Directories ownership OK
Permissions on files OK
Consistently use macros OK
Source is code nor binary OK
Need work:
The text of the license must be included in %license.
See: https://fedoraproject.org/wiki/Packaging:LicensingGuidelines?rd=Packaging/LicensingGuidelines#License_Text
Large documentation files must go in a -doc subpackage. There is a sphinx doc than you can build and include as subpackage.
(In reply to William Moreno from comment #8) > Need work: > The text of the license must be included in %license. > See: > https://fedoraproject.org/wiki/Packaging:LicensingGuidelines?rd=Packaging/ > LicensingGuidelines#License_Text Upstream doesn't include it. Tried to ask for a release with license: https://github.com/pazz/urwidtrees/issues/23#issuecomment-168948700 > Large documentation files must go in a -doc subpackage. There is a sphinx > doc than you can build and include as subpackage. good point, will do There is a text with the license upstream: https://github.com/pazz/urwidtrees/blob/master/LICENSE.md You can patch it and include it in the rpm. ttomecek's scratch build of python-urwidtrees-1.0.0-1.gitfbcb183.fc22.src.rpm for rawhide failed http://koji.fedoraproject.org/koji/taskinfo?taskID=13090507 ttomecek's scratch build of python-urwidtrees-1.0.0-1.gitfbcb183.fc22.src.rpm for rawhide completed http://koji.fedoraproject.org/koji/taskinfo?taskID=13090685 Updated koji task: http://koji.fedoraproject.org/koji/taskinfo?taskID=13090686 spec: https://ttomecek.fedorapeople.org/urwidtrees/python-urwidtrees.spec srpm: https://ttomecek.fedorapeople.org/urwidtrees/python-urwidtrees-1.0-1.gitfbcb183.fc22.src.rpm Added %license to every subpackage. Added python-urwidtrees-doc subpackage with documentation. Synced version with upstream. Package Aproved =============== William, thanks a lot for the review. Package request has been approved: https://admin.fedoraproject.org/pkgdb/package/python-urwidtrees This bug was accidentally moved from POST to MODIFIED via an error in automation, please see mmccune with any questions |