Skip to content

Upgrade to OCaml 5.0#26

Closed
sim642 wants to merge 3 commits intomasterfrom
ocaml-5
Closed

Upgrade to OCaml 5.0#26
sim642 wants to merge 3 commits intomasterfrom
ocaml-5

Conversation

@sim642
Copy link
Copy Markdown
Member

@sim642 sim642 commented Aug 16, 2023

I haven't checked whether Gobview actually works on OCaml 5.0 with these changes though.

On a side note, we really need to do #9 because jsoo-react 0.1 is now on opam, so we could avoid the pin and having to hack version bounds of some old jsoo-react fork ourselves.

@sim642 sim642 added bug Something isn't working enhancement New feature or request labels Aug 16, 2023
@sim642 sim642 marked this pull request as ready for review August 16, 2023 09:06
@sim642 sim642 requested a review from stilscher August 16, 2023 09:06
@sim642
Copy link
Copy Markdown
Member Author

sim642 commented Sep 11, 2023

The Gobview CI failure in goblint/analyzer#1137 seems to suggest that it doesn't actually work for some reason.

@michael-schwarz michael-schwarz removed the request for review from stilscher September 11, 2023 18:07
@sim642
Copy link
Copy Markdown
Member Author

sim642 commented Sep 12, 2023

I cherry-picked parts of this to master to fix the OCaml 5 compatibility, but not make it default yet.

Also the jsoo aliases didn't seem to work although jsoo's own unix.js uses them. Not sure what that's about, so I just duplicated the few functions for now.

@sim642
Copy link
Copy Markdown
Member Author

sim642 commented May 21, 2025

Replaced by #52. Somehow it seems that the unix stubs changes aren't required anymore.

@sim642 sim642 closed this May 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants