Bug 597315
Summary: | Review Request: bwa - Burrows-Wheeler Alignment tool | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Adam Huffman <bloch> |
Component: | Package Review | Assignee: | Chen Lei <supercyper1> |
Status: | CLOSED ERRATA | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | albert.bogdanowicz, fedora-package-review, notting, supercyper1 |
Target Milestone: | --- | Flags: | supercyper1:
fedora-review+
gwync: fedora-cvs+ |
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | bwa-0.5.8a-1.fc12 | Doc Type: | Bug Fix |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2010-09-15 05:41:37 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
Adam Huffman
2010-05-28 16:02:43 UTC
Some initial comments: 1.Source0 seems not right, I cannot download tarball from this URL. 2. make %{?_smp_mflags} -> make %{?_smp_mflags} CFLAGS="%{optflags}" You must honor optflags correctly in package. 3.Please also install bwa.1 to %{_mandir}/man1 4.INSTALL should not be included in %doc. I've addressed these problems in version 2 - thanks for taking a look so quickly. Spec URL: http://verdurin.org.uk/~verdurin/fedora/reviews/bwa/bwa.spec SRPM URL: http://verdurin.org.uk/~verdurin/fedora/reviews/bwa/bwa-0.5.7-2.fc12.src.rpm rpmlint reports the following warning, which I think can be ignored: bwa.src: W: spelling-error %description -l en_US bp -> BP, b, p rpmlint bwa-* bwa.x86_64: W: spelling-error %description -l en_US bp -> BP, b, p bwa.x86_64: W: no-manual-page-for-binary qualfa2fq.pl bwa.x86_64: W: no-manual-page-for-binary solid2fastq.pl 2 packages and 0 specfiles checked; 0 errors, 3 warnings. Those warings are all harmless, this package is approved . New Package CVS Request ======================= Package Name: bwa Short Description: Burrows-Wheeler Alignment tool Owners: verdurin Branches: F-12 F-13 EL-5 InitialCC: CVS done (by process-cvs-requests.py). Do you not want a EL-6 branch as well? File an additional request if you do. New Package CVS Request ======================= Package Name: bwa Short Description: Burrows-Wheeler Alignment tool Owners: verdurin Branches: EL-6 InitialCC: CVS done. bwa-0.5.7-2.fc13 has been submitted as an update for Fedora 13. http://admin.fedoraproject.org/updates/bwa-0.5.7-2.fc13 bwa-0.5.7-2.fc12 has been submitted as an update for Fedora 12. http://admin.fedoraproject.org/updates/bwa-0.5.7-2.fc12 bwa-0.5.7-2.el5 has been submitted as an update for Fedora EPEL 5. http://admin.fedoraproject.org/updates/bwa-0.5.7-2.el5 It seems bwa 0.5.8a is out now. Indeed - I had neglected to push my builds to Bodhi. Once I've updated the package, is it best to remove the old update from Bodhi? bwa-0.5.7-2.fc12 has been pushed to the Fedora 12 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 bwa'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/bwa-0.5.7-2.fc12 bwa-0.5.7-2.fc13 has been pushed to the Fedora 13 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 bwa'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/bwa-0.5.7-2.fc13 There is a compilation problem with 0.5.8a - I'll submit a new update once I've fixed it. bwa-0.5.7-2.el5 has been pushed to the Fedora EPEL 5 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 bwa'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/bwa-0.5.7-2.el5 bwa-0.5.8a-1.fc13 has been submitted as an update for Fedora 13. http://admin.fedoraproject.org/updates/bwa-0.5.8a-1.fc13 bwa-0.5.8a-1.fc12 has been submitted as an update for Fedora 12. http://admin.fedoraproject.org/updates/bwa-0.5.8a-1.fc12 bwa-0.5.8a-1.el5 has been submitted as an update for Fedora EPEL 5. http://admin.fedoraproject.org/updates/bwa-0.5.8a-1.el5 bwa-0.5.8a-1.fc12 has been pushed to the Fedora 12 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 bwa'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/bwa-0.5.8a-1.fc12 *** Bug 533738 has been marked as a duplicate of this bug. *** bwa-0.5.8a-1.el5 has been pushed to the Fedora EPEL 5 stable repository. If problems still persist, please make note of it in this bug report. bwa-0.5.8a-1.fc13 has been pushed to the Fedora 13 stable repository. If problems still persist, please make note of it in this bug report. bwa-0.5.8a-1.fc12 has been pushed to the Fedora 12 stable repository. If problems still persist, please make note of it in this bug report. Package Change Request ====================== Package Name: bwa New Branches: EL-7 Owners: verdurin Git done (by process-git-requests). |