Bug 253349
| Summary: | devel/ocaml-opam: Update to 2.0.8 | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | Ports & Packages | Reporter: | Hannes Mehnert <hannes> | ||||||
| Component: | Individual Port(s) | Assignee: | Rainer Hurling <rhurlin> | ||||||
| Status: | Closed FIXED | ||||||||
| Severity: | Affects Only Me | CC: | hannes, rhurlin | ||||||
| Priority: | --- | Keywords: | buildisok | ||||||
| Version: | Latest | ||||||||
| Hardware: | Any | ||||||||
| OS: | Any | ||||||||
| URL: | https://github.com/ocaml/opam/blob/2.0.8/CHANGES | ||||||||
| Attachments: |
|
||||||||
[tags] in issue Titles are deprecated ;) Build and package info is available at https://gitlab.com/swills/freebsd-ports/pipelines/253127972 Created attachment 222276 [details]
patch with STRIP_CMD and ordered Makefile vars
Hi Hannes,
Thanks for the patch!
I added STRIP_CMD for bin/ocaml-installer, as suggested from 'make DEVELOPER=yes'.
While here, I also ordered Makefile vars, as suggested by portclippy ;)
If you agree, I would commit with these changes.
Comment on attachment 222276 [details]
patch with STRIP_CMD and ordered Makefile vars
Thanks, I hereby approve your additional changes.
(In reply to Rainer Hurling from comment #3) Thanks for the quick feedback :D One more suggestion: If you want to show OPTIONS in the sorted order 'RSYNC GIT HG DARCS' as in your OPTIONS_DEFINE line, one have to add NO_OPTIONS_SORT=yes. Should I do this additionally? [no more approval needed ;) ] (In reply to Rainer Hurling from comment #5) Sure, thanks for your suggestions and review. That'd be a great improvement to show the option in the above mentioned order. ^Triage: If there is a changelog or release notes URL available for this version, please add it to the URL field A commit references this bug: Author: rhurlin Date: Tue Feb 9 05:52:20 UTC 2021 New revision: 564763 URL: https://svnweb.freebsd.org/changeset/ports/564763 Log: devel/ocaml-opam: Update to 2.0.8 Changlog: https://github.com/ocaml/opam/blob/2.0.8/CHANGES PR: 253349 Submitted by: hannes@mehnert.org (maintainer) Approved by: tcberner (mentor) Differential Revision: https://reviews.freebsd.org/D28544 Changes: head/devel/ocaml-opam/Makefile head/devel/ocaml-opam/distinfo Committed, thanks :) |
Created attachment 222269 [details] output of diff -ur ocaml-opam.old ocaml-opam Hi (maintainer here), ocaml-opam 2.0.8 was just released today, please find attached the update to the port (as unified diff). Thanks a lot, Hannes