Skip to content

Commit

Permalink
[ci] Add 32bit 5.2.0 job
Browse files Browse the repository at this point in the history
Caveats:

- The OCaml switch in this case byte-only, so things are slow, as jsoo
  itself won't have a native mode

- JS ppx libs 0.17 don't really support 32bit builds, this will likely
  become a problem in the future (if not now actually, for example
  `ppx_inline_test.0.17.0` fails to build)

- jsCoq/coq-lsp works in this setup, which is IMO a great "data point"
  as to whether this setup is OK
  • Loading branch information
ejgallego committed Oct 1, 2024
1 parent ef42a56 commit 1b1a666
Showing 1 changed file with 7 additions and 1 deletion.
8 changes: 7 additions & 1 deletion .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -57,6 +57,12 @@ jobs:
skip-effects: false
skip-test: false
skip-doc: false
# Note this OCaml compiler is bytecode only
- os: ubuntu-latest
ocaml-compiler: "ocaml-variants.5.2.0+options,ocaml-option-32bit"
skip-effects: false
skip-test: true # time_now.0.17 is pulled and doesn't work in 32 bits
skip-doc: true
- os: macos-latest
ocaml-compiler: "5.2"
skip-effects: true
Expand All @@ -82,7 +88,7 @@ jobs:
# getting much better, but no luck yet, c.f:
# https://github.com/ocaml/opam-repository/pull/26626
- name: Install apt 32-bit dependencies
if: matrix.ocaml-compiler == 'ocaml-variants.4.14.2+options,ocaml-option-32bit'
if: matrix.ocaml-compiler == 'ocaml-variants.4.14.2+options,ocaml-option-32bit' || matrix.ocaml-compiler == 'ocaml-variants.5.2.0+options,ocaml-option-32bit'
run: |
sudo apt-get install aptitude
sudo dpkg --add-architecture i386
Expand Down

0 comments on commit 1b1a666

Please sign in to comment.