Bug 216912
| Summary: | Review Request: rtpproxy - A symmetric RTP proxy | ||
|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Jeffrey C. Ollie <jeff> |
| Component: | Package Review | Assignee: | Kevin Fenzi <kevin> |
| Status: | CLOSED ERRATA | QA Contact: | Fedora Package Reviews List <fedora-package-review> |
| Severity: | medium | Docs Contact: | |
| Priority: | medium | ||
| Version: | rawhide | CC: | lemenkov |
| Target Milestone: | --- | Flags: | gwync:
fedora-cvs+
|
| Target Release: | --- | ||
| Hardware: | All | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Fixed In Version: | rtpproxy-2.0.0-1.el7 | Doc Type: | Bug Fix |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2006-12-04 02:11:22 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: | 163779 | ||
|
Description
Jeffrey C. Ollie
2006-11-22 17:14:29 UTC
OK - Spec has consistant macro usage. OK - Meets Packaging Guidelines. OK - License (BSD) OK - License field in spec matches OK - License file included in package OK - Spec in American English OK - Spec is legible. OK - Sources match upstream md5sum: 2a316f4854da4e8c8d358c623e5ffdb6 rtpproxy-0.3.tar.gz 2a316f4854da4e8c8d358c623e5ffdb6 rtpproxy-0.3.tar.gz.1 OK - BuildRequires correct OK - Package has %defattr and permissions on files is good. OK - Package has a correct %clean section. OK - Package has correct buildroot OK - Package is code or permissible content. OK - Packages %doc files don't affect runtime. OK - Package compiles and builds on at least one arch. OK - Package has no duplicate files in %files. OK - Package doesn't own any directories other packages own. OK - Package owns all the directories it creates. OK - No rpmlint output. OK - final provides and requires are sane: SHOULD Items: OK - Should build in mock. i386/x86_64 - Should build on all supported archs OK - Should have dist tag OK - Should package latest version Issues: 1. Is there anything aside from the including COPYING file to indicate the license? I guess including that COPYING file might be enough, but there is no mention anywhere in the code, web page or other files what the license is. Perhaps you could report to upstream that they could be more explicit about the license moving forward? Otherwise I don't see any issues... this package is APPROVED. Don't forget to close this report NEXTRELEASE once it's been imported and built. Also, do consider reviewing another waiting package to try and spread out the reviewer load. Thanks for the review... imported and build for development kicked off... Package Change Request ====================== Package Name: rtpproxy New Branches: EL-4 EL-5 Updated Fedora Owners: jcollie, peter Updated EPEL Owners: jcollie, peter cvs done. Package Change Request ====================== Package Name: rtpproxy New Branches: epel7 Owners: peter jcollie itamarjp Git done (by process-git-requests). rtpproxy-2.0.0-1.el7 has been submitted as an update for Fedora EPEL 7. https://admin.fedoraproject.org/updates/rtpproxy-2.0.0-1.el7 rtpproxy-2.0.0-1.el7 has been pushed to the Fedora EPEL 7 stable repository. |