Bug 814542
Summary: | Review Request: lv2 - Audio Plugin standard (lv2core rename) | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Brendan Jones <brendan.jones.it> |
Component: | Package Review | Assignee: | Gwyn Ciesla <gwync> |
Status: | CLOSED ERRATA | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | unspecified | Docs Contact: | |
Priority: | unspecified | ||
Version: | 16 | CC: | gwync, notting, package-review |
Target Milestone: | --- | Flags: | gwync:
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: | 2012-05-28 01:18:13 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: | 814040, 814916 |
Description
Brendan Jones
2012-04-20 05:43:45 UTC
SPEC: http://bsjones.fedorapeople.org/lv2/lv2.spec SRPM: http://bsjones.fedorapeople.org/lv2/lv2-1.0.0-1.fc16.src.rpm Corrected Obsoletes/Provides I'll take this. . . ood: - rpmlint checks return: See above. Should these last remaining obsoletes warnings not be fixed? lv2.x86_64: W: obsolete-not-provided lv2-ui(x86-64) lv2-devel.x86_64: W: obsolete-not-provided lv2-ui-devel(x86-64) Also: lv2-debuginfo.x86_64: E: debuginfo-without-sources Fix. - package meets naming guidelines - package meets packaging guidelines - license ( ISC ) OK, text in %doc, matches source I also see MIT and CC-AT-SA - spec file legible, in am. english - source matches upstream - package compiles on devel (x86) - no missing BR - no unnecessary BR - no locales - not relocatable - owns all directories that it creates - no duplicate files - permissions ok - %clean ok - macro use consistent - code, not content - no need for -docs - nothing in %doc affects runtime - no need for .desktop file - devel package ok - no .la files - post/postun ldconfig ok Fix. - devel requires base package n-v-r Also, I see it's using a local waf, why not BuildRequire waf and use the system waf? Thanks for taking this on. I should have reposted the rpmlint out put when I re-uploaded the package. The build fails with system waf (even though they are the same version). This is a patched custom waf script used quite widely in linux audio projects. Waf upstream recommends always bundling a local waf (not saying I agree). There's no clear guidelines on this. All other fixes done. SPEC: http://bsjones.fedorapeople.org/lv2/lv2.spec SRPM: http://bsjones.fedorapeople.org/lv2/lv2-1.0.0-2.fc16.src.rpm Also, can you please confirm that I've got the Obsoletes/Requires correct? Its more of a replacement than a rename as the version of this package is 1.0.0 and lv2core is currently sitting at lv2core-6.0-3 Ok, just the docs and symlinks now, and those are more or less OK. The obsoletes and requires are good. I'm not thrilled with the bundled waf, but since it's just a build tool and not included I'll let it slide for now, but if you or upstream can ever get it building with system waf that would be highly preferable. APPROVED. Thanks for the review Jon. Let me know if I can return the favour. New Package SCM Request ======================= Package Name: lv2 Short Description: Audio Plugin Standard Owners: bsjones Branches: f16 f17 InitialCC: Nothing now, but you could always do a Merge Review. :) Git done (by process-git-requests). lv2-1.0.0-6.fc17 has been submitted as an update for Fedora 17. https://admin.fedoraproject.org/updates/lv2-1.0.0-6.fc17 lv2-1.0.0-6.fc17 has been pushed to the Fedora 17 testing repository. lv2-1.0.0-6.fc17 has been pushed to the Fedora 17 stable repository. |