squashed & rebased

• devel/ocaml-logs: Missing RUN_DEPENDS.
• x11-toolkits/ocaml-graphics: Extraneous BUILD_DEPENDS.
• devel/ocaml-fpath: Missing RUN_DEPENDS.
• devel/ocaml-fmt: Missing RUN_DEPENDS.
• devel/ocaml-camomile: Missing RUN_DEPENDS.
• devel/ocaml-camlp-streams: Package version set.
• devel/ocaml-csexp: Package version set.
• devel/ocaml-pcre2: Package version set.
• devel/ocaml-re: Package version set.
• x11-toolkits/ocaml-graphics: Package version set.
• Set WRK_ENV.

Thank you again for the reviews. There had been a bit of fallover from the moving of labltk and lablGL to site-lib/, I should have catched them before...
All plist issues should be gone too, as I've learned to use -t with poudriere-bulk ;)

Squashed, commit message preparation.

• Uses/ocaml.mk: Typo.
• devel/ocaml-sdl: bad --with-lablgldir.
• games/freetennis: LablGL moved to site-lib, tidy Makefile.
• devel/ocaml-logs: PORTDOCS adoption (un-orphaning).

• Mk/Uses/dune.mk: clean build environment
• respect MAKE_ENV
• USES=dune -> USES=ocaml:dune
• devel/ocaml-dune: fix PORTREVISION increment
• security/ocaml-ssl: PORTREVISION bump.
• textproc/ocaml-text: PORTREVISION bump.

ocaml-findlib fix, rebased&squashed.

• lang/ocaml: Missing PKGNAMESUFFIX.
• math/teyjus: redundant build dep.
• devel/ocaml-camlp5: Plist fix.
• sysutils/ledit: Plist fix, install native binary, manpage version patch.
• x11-toolkits/ocaml-labltk: Fixes
• devel/ocaml-findlib: Stop providing conflicting META.
• devel/ocaml-findlib: Broken POST-DEINSTALL.
• graphics/ocaml-lablgl: Labltk include path fix, OCAML_SITELIBDIR
• x11-toolkits/ocaml-lablgtk2: LablGL in site-lib.

In D43861#1003621, @madpilot wrote:

Hi!

I tested your patch in poudriere. In general it is very good from my POV, congrats on your work.

• devel/ocaml-camlp5: Plist fix.
• sysutils/ledit: Plist fix, install native binary, manpage version patch.
• x11-toolkits/ocaml-labltk: Fixes
• devel/ocaml-findlib: Stop providing conflicting META.
• devel/ocaml-findlib: Broken POST-DEINSTALL.
• graphics/ocaml-lablgl: Labltk include path fix, OCAML_SITELIBDIR, shared librairies stripped, missing GL dependencies

• lang/ocaml: Missing PKGNAMESUFFIX.
• math/teyjus: redundant build dep.

In D43861#1000601, @madpilot wrote:

Hi,

Thanks for your work, this is a much needed update, and not an easy one!

At a cursory look the patch looks well composed, but before approval it requires some testing.

The first commits are preparation to individual ports which must be applied before applying the last commit. Those commits work on the current tree and should cause no regressions.

While I agree that keeping the commits separated in the revision is useful, especially for you, if the patches require updates, for the final commit to the ports tree I'd be more in favour of one big commit with a succinct description for the individual changes (as in a list of one line descriptions). There are reasons for this.

Every commit would require the commit message to be modified to add our standard tags, referencing this revision and all approved by/tested by lines.

This is my opinion, maybe other involved committers have different ideas.