Bug 470325
Summary: | Review Request: qd - Double-Double and Quad-Double Arithmetic | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Susi Lehtola <susi.lehtola> |
Component: | Package Review | Assignee: | Jason Tibbitts <j> |
Status: | CLOSED NEXTRELEASE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | cse.cem+redhatbugz, fedora-package-review, notting |
Target Milestone: | --- | Flags: | j:
fedora-review+
kevin: fedora-cvs+ |
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | 2.3.7-5.fc10 | Doc Type: | Bug Fix |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2009-03-11 18:02: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: |
Description
Susi Lehtola
2008-11-06 18:16:13 UTC
I would posit that this shouldn't go in until we can either get a shared library out of this or at least get some kind of upstream confirmation that you really can't build it that way. You can ship the static librar(y|ies), if you like, in a separate -static package. And the headers should go in a -devel package, as rpmlint dutifully complains. (In reply to comment #1) > I would posit that this shouldn't go in until we can either get a shared > library out of this or at least get some kind of upstream confirmation that you > really can't build it that way. You can ship the static librar(y|ies), if you > like, in a separate -static package. And the headers should go in a -devel > package, as rpmlint dutifully complains. I have contacted upstream about this. It seems that the support to build shared libraries was added in version 2.1, but removed in 2.1.213 with libtool. The lack of a shared library isn't the blocker (though it would be nice). The issue is, as tibbs says: "You can ship the static librar(y|ies), if you like, in a separate -static package. And the headers should go in a -devel package, as rpmlint dutifully complains." (In reply to comment #3) > The lack of a shared library isn't the blocker (though it would be nice). The > issue is, as tibbs says: "You can ship the static librar(y|ies), if you > like, in a separate -static package. And the headers should go in a -devel > package, as rpmlint dutifully complains." Very well. In accordance with the Packaging Guidelines, I packaged everything in qd-devel, which provides qd-static. rpmlint output: qd-devel-debuginfo.x86_64: E: empty-debuginfo-package 2 packages and 1 specfiles checked; 1 errors, 0 warnings. Spec URL: http://theory.physics.helsinki.fi/~jzlehtol/rpms/qd-devel.spec SRPM URL: http://theory.physics.helsinki.fi/~jzlehtol/rpms/qd-devel-2.3.7-2.fc10.src.rpm (In reply to comment #4) > rpmlint output: > qd-devel-debuginfo.x86_64: E: empty-debuginfo-package > 2 packages and 1 specfiles checked; 1 errors, 0 warnings. Add a line to the top of your spec: %define debug_package %{nil} (Since rpmbuild doesn't get useful debuginfo from static libs.) Done. Spec URL: http://theory.physics.helsinki.fi/~jzlehtol/rpms/qd-devel.spec SRPM URL: http://theory.physics.helsinki.fi/~jzlehtol/rpms/qd-devel-2.3.7-3.fc10.src.rpm Er, the spec/%{name} should be qd.spec, not qd-devel.spec. You should however omit the %files section for the main package and only have one for the devel subpackage. (In reply to comment #7) > Er, the spec/%{name} should be qd.spec, not qd-devel.spec. You should however > omit the %files section for the main package and only have one for the devel > subpackage. Right, since we want to be prepared for having a shared library. Otherwise the base name would have been qd-devel (as pstreams-devel and a bunch of other packages). http://theory.physics.helsinki.fi/~jzlehtol/rpms/qd.spec http://theory.physics.helsinki.fi/~jzlehtol/rpms/qd-2.3.7-4.fc10.src.rpm Sorry for not getting back to this sooner. Any reason why you believe the license is GPLv2+? The COPYING file included the rather standard BSD license. (The copyright is even held by UCB.) The actual source files are missing any license statment but README explicitly refers to the COPYING file which contains the BSD text. Generally we'd prefer the source files to include license statements but I don't think the licensing situation is ambiguous. Of course, I could be missing something. I would also comment why you're disabling the debug package, because it's not immediately obvious from looking at the spec. * source files match upstream. sha256sum: c886c96ca8081196089fe00eac74765a43e7500ae27e2e107a7c4f10fa95b4f1 qd-2.3.7.tar.gz * package meets naming and versioning guidelines. * specfile is properly named, is cleanly written and uses macros consistently. * summary is OK. * description is OK. * dist tag is present. * build root is OK. X license field does not match the actual license. * license is open source-compatible. * license text included in package. * latest version is being packaged. * BuildRequires are proper. * compiler flags are appropriate. * %clean is present. * package builds in mock (rawhide, x86_64). * package installs properly. * rpmlint is silent. * final provides and requires are sane: qd-static = qd-2.3.7 qd-devel = 2.3.7-4.fc11 qd-devel(x86-64) = 2.3.7-4.fc11 = /bin/sh * %check is present and all tests pass: All 4 tests passed * no shared libraries are added to the regular linker search paths. * owns the directories it creates. * doesn't own any directories it shouldn't. * no duplicates in %files. * file permissions are appropriate. * no generically named files * code, not content. * documentation is small, so no -doc subpackage is necessary. * %docs are not necessary for the proper functioning of the package. * headers are in the -devel package. * no pkgconfig files. * only static libraries are present: They are in the -devel package. -static is provided. * no libtool .la files. (In reply to comment #9) > I would also comment why you're disabling the debug package, because it's not > immediately obvious from looking at the spec. In my experience rpmbuild doesn't grab any debugging information from static libraries; is this not the case? (In reply to comment #9) > Sorry for not getting back to this sooner. > > Any reason why you believe the license is GPLv2+? The COPYING file included > the rather standard BSD license. Oh my, thanks for noticing. Fixed. Also added a comment about disabling debuginfo due to static library. http://theory.physics.helsinki.fi/~jzlehtol/rpms/qd.spec http://theory.physics.helsinki.fi/~jzlehtol/rpms/qd-2.3.7-5.fc10.src.rpm (In reply to comment #10) > In my experience rpmbuild doesn't grab any debugging information from static > libraries; is this not the case? That is the case, and is why the debug package is disabled in this spec. I was merely asking that this be commented. Anyway, this looks good now; thanks. APPROVED New Package CVS Request ======================= Package Name: qd Short Description: Double-Double and Quad-Double Arithmetic Owners: jussilehtola Branches: F-9 F-10 EL-5 InitialCC: cvs done. qd-2.3.7-5.fc9 has been submitted as an update for Fedora 9. http://admin.fedoraproject.org/updates/qd-2.3.7-5.fc9 qd-2.3.7-5.fc10 has been submitted as an update for Fedora 10. http://admin.fedoraproject.org/updates/qd-2.3.7-5.fc10 qd-2.3.7-5.fc9 has been pushed to the Fedora 9 stable repository. If problems still persist, please make note of it in this bug report. qd-2.3.7-5.fc10 has been pushed to the Fedora 10 stable repository. If problems still persist, please make note of it in this bug report. |