Bug 1185099
| Summary: | Review Request: opam - A source-based package manager for OCaml | ||
|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Jon Ludlam <jon> |
| Component: | Package Review | Assignee: | Nobody's working on this, feel free to take it <nobody> |
| Status: | CLOSED DUPLICATE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
| Severity: | medium | Docs Contact: | |
| Priority: | unspecified | ||
| Version: | rawhide | CC: | anil, edwin+bugs, package-review, rjones, rosser.bjr |
| Target Milestone: | --- | ||
| Target Release: | --- | ||
| Hardware: | All | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Fixed In Version: | Doc Type: | Bug Fix | |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2017-10-13 16:23:28 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: | |||
| Bug Depends On: | 1172771, 1174036, 1183191, 1183195, 1183825, 1183826, 1480794, 1486068, 1790146 | ||
| Bug Blocks: | |||
|
Description
Jon Ludlam
2015-01-22 21:56:27 UTC
Nb, this depends upon: https://bugzilla.redhat.com/show_bug.cgi?id=1172771 https://bugzilla.redhat.com/show_bug.cgi?id=1174036 https://bugzilla.redhat.com/show_bug.cgi?id=1183191 https://bugzilla.redhat.com/show_bug.cgi?id=1183195 https://bugzilla.redhat.com/show_bug.cgi?id=1183825 https://bugzilla.redhat.com/show_bug.cgi?id=1183826 Updated with feedback from Anil Madhavapeddy. Spec URL: http://www.recoil.org/~jon/opam.spec SRPM URL: http://www.recoil.org/~jon/opam-1.2.0-2.fc22.src.rpm Patch file should be *.patch. The rest of the spec file looks reasonable. Fixed thanks. Spec URL: http://www.recoil.org/~jon/opam.spec SRPM URL: http://www.recoil.org/~jon/opam-1.2.0-3.fc21.src.rpm Are you installing the cmxs files (dynamic native loading) along with the static native code files (cmxa,a,o)? Rest of the spec file looks good to me. New version posted. I've made it install the cmxs files as well, and also changed the Requires line in ocaml-opam-lib-devel - previously it was requiring opam, now it requires ocaml-opam-lib%{?_isa}.
Also, I tested the smp flags with make, but the parallel build fails. I think this still fails on the master branch, so I'll double check then make an issue on github.
Spec URL: http://www.recoil.org/~jon/opam.spec
SRPM URL: http://www.recoil.org/~jon/opam-1.2.0-4.fc21.src.rpm
Spec URL: http://www.recoil.org/~jon/opam.spec SRPM URL: http://www.recoil.org/~jon/opam-1.2.1-1.fc22.src.rpm Upstream released a new version. *** This bug has been marked as a duplicate of bug 1501992 *** |