build: update pinned package versions

[tsani]

Several versions of packages mentioned in the lockfiles before this commit do not exist on opam anymore. This causes a build failure on a fresh clone of this repo in docker, which uses the package versions pinned in the lockfiles.

To find these package versions, I simply deleted the lockfiles, installed all dependencies in a fresh OCaml 5.1.1 switch, and used opam lock. This process resulted in version 5.9.1 for
js_of_ocaml{,-toplevel,-compiler,...} on which Learn-OCaml's toplevel fails to initialize, making the webapp unusable. I then manually lowered the pinned version of js_of_ocaml back to the previously pinned version, 5.8.2.

• Kind: bugfix

• Close Bug: docker build failure, bogus pinned package versions, "Toplevel not initialized" #621

Description

Fresh clone of learn-ocaml fails to build in docker due to bogus pinned package versions, which no longer exist in opam.

Checklist

• Read the CONTRIBUTING.md guide and:
  • Use Atomic Commits so each commit gathers a single logical change
  • Use Conventional Commits regarding commit messages (needed by our release toolchain)
• Add/update tests N/A
• Add/update documentation N/A

Note to maintainers

• Read this wiki page.
• Make sure the PR has a milestone.
• Assign yourself before merging.
• Either do a regular merge:
  • for PRs containing several commits following conventional-commits,
  • or for PRs containing 1 commit shared with a later PR (to preserve the SHA1)
• Or do a squash-merge:
  • for PRs containing only 1 commit (not shared with a later PR),
  • or for PRs containing several commits that need not be kept in the history;
  • → Update the commit message header with a conventional-commit type,
  • → Add a footer Close #… if a related issue exists.

[erikmd]

@tsani the CI is fine, and I sucessfully tested your branch locally. Thanks!

But given the details you gave when preparing the PR (I then manually lowered the pinned version of js_of_ocaml), I'd suggest you add this upper bound in the two learn-ocaml*.opam files:

  "js_of_ocaml" {>= "5.0.0" & < "5.9.0"}

and/or

  "js_of_ocaml-compiler" {>= "5.0.0" & < "5.9.0"}

[tsani]

I'd suggest you add this upper bound in the two learn-ocaml*.opam files

Done. Makes perfect sense to do so, as it should make regenerating the lockfiles easier next time.

[erikmd]

Thanks!