Final ping from my side in case anyone else is interested.

On 2/14/23 21:24, Volker Schlecht wrote:
Alright, so here's the updated dune patch plus all the patches that need to go in alongside...

As a reminder:

- lablgtk3
- ocaml-cppo
- ocaml-graphics
- ocaml-cairo
- ocaml-yojson

--> Trivial post-install changes, because dune now installs docs directly into the correct directories + results of running update-plist.

- ocaml-pcre

--> Same, but has nothing left depending on it, so just cvs rm'ing it might be an option, too.

- ocaml-menhir

--> builds on an already submitted patch:
https://marc.info/?l=openbsd-ports&m=167314822610719&w=2

Anyone interested?

On 2/14/23 17:28, Yozo TODA wrote:
Are we sure that we aren't bending over backwards here to "fix"
something that isn't a bug, and thereby increase the chances of
introducing problems elsewhere?

I agree that we don't need to hustle to "fix".
in my update proposal patch, just omit the last three lines of post-install target.

    echo 'version = "${VERSION}"' > ${WRKINST}${LOCALBASE}/lib/ocaml/dune/META.new     cat ${WRKINST}${LOCALBASE}/lib/ocaml/dune/META >> ${WRKINST}${LOCALBASE}/lib/ocaml/dune/META.new     mv ${WRKINST}${LOCALBASE}/lib/ocaml/dune/META.new ${WRKINST}${LOCALBASE}/lib/ocaml/dune/META
-- yozo.


Reply via email to