Bug 488910
Summary: | Review Request: bio2jack - A library for porting blocked io(OSS/ALSA) applications to jack | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Orcan Ogetbil <oget.fedora> |
Component: | Package Review | Assignee: | Christian Krause <chkr> |
Status: | CLOSED NEXTRELEASE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | chkr, dtimms, fedora-package-review, notting, tcallawa |
Target Milestone: | --- | Flags: | chkr:
fedora-review+
kevin: fedora-cvs+ |
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | 0.9-3.fc9 | Doc Type: | Bug Fix |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2009-03-26 14:53:57 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
Orcan Ogetbil
2009-03-06 06:05:53 UTC
Hi, I've just reviewed the package, everything looks good so far. Only about the license I'm not sure. Please let me ask for a peer-review this time, too. I expect that after a few more reviews we can skip this step. ;-) * rpmlint: OK rpmlint SPECS/bio2jack.spec RPMS/i386/bio2jack-* SRPMS/bio2jack-0.9-1.src.rpm SPECS/bio2jack.spec:46: W: configure-without-libdir-spec bio2jack-devel.i386: W: no-documentation bio2jack.src:46: W: configure-without-libdir-spec 4 packages and 1 specfiles checked; 0 errors, 3 warnings. - configure-without-libdir-spec is a false positive, since ./configure just appears in the sed -i -e call - no-doc - the package doesn't provide any develoment-specific documentation * naming, spec file name: OK * license: unsure - GPL/LGPL both approved by Fedora, however I'm not sure whether this ambiguity must be solved - license file included in package * sources: OK - spectool -g works - md5sum: 00b64a99856cb35f1170c97ecb6bc431 bio2jack-0.9.tar.gz * builds in mock & on all architectures for F11, F10, F9 in koji * build requirements: OK * locales: OK, since there are none * ldconfig in %post, %postun: OK * owning directory: OK, no directories are created besides %doc * no files listed twice: OK * %defattr usage: OK * %clean section and buildroot clean before install: OK * macro usage: OK * code/content: OK, only code * large documentation into subpackage: OK, no large doc * header in devel-package: OK * static libraries: OK (none) * pkgconfig: OK (no *.pc files) * *.so link in devel package: OK * devel package requires fully versioned base pkg: OK * no *.la files: OK * *.desktop files: OK (n/a) * no shared ownership of directories: OK * filenames in valid UTF-8: OK * scriptlets: OK (only ldconfig) * %{optflags} are honored: OK Best regards, Christian When the actual code conflicts with what other files may indicate, we have to trust the code. Inclusion of the wrong COPYING file is probably an error by upstream. The best course of action is to simply ask the upstrem developer to clarify the license and to fix either the COPYING file or the source itself. If they don't reply, then just go with LGPLv2+ as indicated by the code. I agree with Jason. Point out the inconsistency to upstream, they probably just copied over the wrong license text. As is, the tag should be License: LGPLv2+ (We always trust the source code over COPYING.) Thanks for the review, again. I sent an email upstream. Let's wait a week or two. If we don't get a reply, I'll set the license field to LGPLv2+. The author replied almost rightaway. It is as spot predicted: He copied the wrong license text. I changed the SPEC file accordingly: Spec URL: http://oget.fedorapeople.org/review/bio2jack.spec SRPM URL: http://oget.fedorapeople.org/review/bio2jack-0.9-2.src.rpm Changelog: 0.9-2 - License is LGPLv2+. Don't package the wrong COPYING file I've checked the new package - there are only license related changes: - License is finally set to LGPLv2+ after the confirmation by the upstream author - misleading COPYING file is not packaged anymore Otherwise there are no changes. Since the only questionable issue was fixed: APPROVED. Thanks a lot! New Package CVS Request ======================= Package Name: bio2jack Short Description: A library for porting blocked io(OSS/ALSA) applications to jack Owners: oget Branches: F-10 InitialCC: Sorry. I also want the F-9 branch for this. So: New Package CVS Request ======================= Package Name: bio2jack Short Description: A library for porting blocked io(OSS/ALSA) applications to jack Owners: oget Branches: F-9 F-10 InitialCC: cvs done. bio2jack-0.9-3.fc10 has been submitted as an update for Fedora 10. http://admin.fedoraproject.org/updates/bio2jack-0.9-3.fc10 bio2jack-0.9-3.fc9 has been submitted as an update for Fedora 9. http://admin.fedoraproject.org/updates/bio2jack-0.9-3.fc9 bio2jack-0.9-3.fc9 has been pushed to the Fedora 9 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing-newkey update bio2jack'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F9/FEDORA-2009-2697 bio2jack-0.9-3.fc10 has been pushed to the Fedora 10 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update bio2jack'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F10/FEDORA-2009-2710 bio2jack-0.9-3.fc10 has been pushed to the Fedora 10 stable repository. If problems still persist, please make note of it in this bug report. bio2jack-0.9-3.fc9 has been pushed to the Fedora 9 stable repository. If problems still persist, please make note of it in this bug report. Package Change Request ====================== Package Name: bio2jack New Branches: EL-5 EL-6 Owners: dtimms CVS done (by process-cvs-requests.py). |