Bug 1185099 - Review Request: opam - A source-based package manager for OCaml
Summary: Review Request: opam - A source-based package manager for OCaml
Keywords:
Status: CLOSED DUPLICATE of bug 1501992
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
unspecified
medium
Target Milestone: ---
Assignee: Nobody's working on this, feel free to take it
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On: 1172771 1174036 1183191 1183195 1183825 1183826 1480794 1486068 1790146
Blocks:
TreeView+ depends on / blocked
 
Reported: 2015-01-22 21:56 UTC by Jon Ludlam
Modified: 2020-01-12 05:09 UTC (History)
5 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2017-10-13 16:23:28 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Jon Ludlam 2015-01-22 21:56:27 UTC
Spec URL: http://www.recoil.org/~jon/opam.spec
SRPM URL: http://www.recoil.org/~jon/opam-1.2.0-1.fc22.src.rpm
Description:
OPAM is a source-based package manager for OCaml. It supports multiple
simultaneous compiler installations, flexible package constraints, and
a Git-friendly development workflow.

Fedora Account System Username: jonludlam

Comment 2 Jon Ludlam 2015-01-22 23:14:31 UTC
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

Comment 3 Richard W.M. Jones 2015-01-23 18:21:52 UTC
Patch file should be *.patch.

The rest of the spec file looks reasonable.

Comment 4 Jon Ludlam 2015-01-24 22:45:06 UTC
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

Comment 5 Anil Madhavapeddy 2015-01-26 14:56:41 UTC
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.

Comment 6 Jon Ludlam 2015-02-26 10:32:14 UTC
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

Comment 7 Jon Ludlam 2015-03-29 01:14:55 UTC
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.

Comment 8 Ben Rosser 2017-10-13 16:23:28 UTC

*** This bug has been marked as a duplicate of bug 1501992 ***


Note You need to log in before you can comment on or make changes to this bug.