| Summary: | Review Request: libscs - Software Carry-Save Multiple-Precision Library | ||
|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Tim Niemueller <tim> |
| Component: | Package Review | Assignee: | Dmitrij S. Kryzhevich <kryzhev> |
| Status: | CLOSED ERRATA | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
| Severity: | medium | Docs Contact: | |
| Priority: | medium | ||
| Version: | rawhide | CC: | fedora-package-review, kryzhev, notting |
| Target Milestone: | --- | Flags: | kryzhev:
fedora-review+
j: fedora-cvs+ |
| Target Release: | --- | ||
| Hardware: | All | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Fixed In Version: | libscs-1.4.1-4.fc14 | Doc Type: | Bug Fix |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2011-03-03 08:23:10 UTC | Type: | --- |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | --- | Target Upstream Version: | |
| Bug Depends On: | |||
| Bug Blocks: | 676159 | ||
|
Description
Tim Niemueller
2011-02-19 14:32:38 UTC
Just after posting I discovered a typo in the configure line, causing the library to be built without MPFR support and thus rendering scs_test_accuracy disfunctional. Spec changed in place, new SRPM is at http://fedorapeople.org/~timn/misc/libscs-1.4.1-2.fc14.src.rpm. Why do you move "#include <stdio.h>" in scslib-1.4.1/tests/test_accuracy.c? Suggestion: make "%doc DocsDev/html" to get html subfolder in -devel. Are the test programs required? One more. libscs-devel must Require mpfr-devel and gmp-devel. (In reply to comment #2) > Why do you move "#include <stdio.h>" in scslib-1.4.1/tests/test_accuracy.c? Because it is required to get a macro defined in gmp.h, which consequently makes mpfr_out_str available. > Suggestion: make "%doc DocsDev/html" to get html subfolder in -devel. Since the HTML documentation is the only thing packaged as %doc it only adds on more meaningless level of hierarchy. > Are the test programs required? They provide valuable run-time data specific to the host system the application is running on, therefore they should be shipped with the library afaik. > One more. libscs-devel must Require mpfr-devel and gmp-devel. Added. Spec changed in place, new SRPM is at http://fedorapeople.org/~timn/misc/libscs-1.4.1-3.fc14.src.rpm. (In reply to comment #4) > Because it is required to get a macro defined in gmp.h, which consequently > makes mpfr_out_str available. Hm. The order of #include influences on the result. Greate. > Since the HTML documentation is the only thing packaged as %doc it only adds on > more meaningless level of hierarchy. Ok. > They provide valuable run-time data specific to the host system the application > is running on, therefore they should be shipped with the library afaik. Ok. ======== Review =========================== + rpmlint was run on the source rpm and all binary rpms the build produces. + The package was named according to the Package Naming Guidelines. + The spec file name matchs the base package %{name}. + The package meets the Packaging Guidelines. + The package is licensed with a Fedora approved license LGPLv2+. + The License field in the package spec file must match the actual license. + File, containing the text of the license(s) for the package is included in %doc. + The spec file must is written in American English. + The spec file for the package is legible. + The sources used to build the package matchs the upstream source. + The package is successfully compiled and build into binary rpms on at least one primary architecture. + All build dependencies must are listed in BuildRequires. + Binary RPM package calls ldconfig in %post and %postun. + Packages do NOT bundle copies of system libraries. + A package owns all directories that it creates. + A package do not list a file more than once in the spec file's %files listings. + Permissions on files are set properly. + Package consistently uses macros. + The package contains code, or permissable content. * No localization. * No large documentaion. + Everything included as %doc do not affect the runtime of the application. + Header files are in a -devel package. * No static libraries. + Library files that end in .so (without suffix) are in a -devel package. + Devel packages requires the base package in proper way. + Packages do NOT contain any .la libtool archives. * Not a GUI application. + Packages do not own files or directories already owned by other packages. + All filenames in rpm packages must be valid UTF-8. Approved. Thanks for the thorough review. New Package SCM Request ======================= Package Name: libscs Short Description: Software Carry-Save Multiple-Precision Library Owners: timn Branches: f14 f15 el5 el6 InitialCC: Git done (by process-git-requests). libscs-1.4.1-4.fc14 has been submitted as an update for Fedora 14. https://admin.fedoraproject.org/updates/libscs-1.4.1-4.fc14 libscs-1.4.1-4.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/libscs-1.4.1-4.fc15 libscs-1.4.1-4.el5 has been submitted as an update for Fedora EPEL 5. https://admin.fedoraproject.org/updates/libscs-1.4.1-4.el5 libscs-1.4.1-4.el6 has been submitted as an update for Fedora EPEL 6. https://admin.fedoraproject.org/updates/libscs-1.4.1-4.el6 libscs-1.4.1-4.fc14 has been pushed to the Fedora 14 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 libscs'. You can provide feedback for this update here: https://admin.fedoraproject.org/updates/libscs-1.4.1-4.fc14 libscs-1.4.1-4.fc15 has been pushed to the Fedora 15 stable repository. If problems still persist, please make note of it in this bug report. libscs-1.4.1-4.fc14 has been pushed to the Fedora 14 stable repository. If problems still persist, please make note of it in this bug report. libscs-1.4.1-4.el5 has been pushed to the Fedora EPEL 5 stable repository. libscs-1.4.1-4.el6 has been pushed to the Fedora EPEL 6 stable repository. |