Skip to content

Commit

Permalink
Do not test with conf-npm on x86_32
Browse files Browse the repository at this point in the history
The installation of `conf-npm` tends to fail on CI on `x86_32` and, indeed, it
just failed on `opam-ci`.  As we test on various other architectures, it is
probably best to just skip testing on the legacy `x86_32`.
  • Loading branch information
polytypic committed Jan 12, 2025
1 parent 0a82bcf commit b9a990a
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 1 deletion.
1 change: 1 addition & 0 deletions dune-project
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,7 @@
:with-test))
(conf-npm
(and
(<> :arch "x86_32")
(<> :os "win32")
:with-test))
;; Documentation dependencies
Expand Down
2 changes: 1 addition & 1 deletion multicore-magic.opam
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ depends: [
"domain_shims" {>= "0.1.0" & with-test}
"alcotest" {>= "1.7.0" & with-test}
"js_of_ocaml" {>= "5.4.0" & with-test}
"conf-npm" {os != "win32" & with-test}
"conf-npm" {arch != "x86_32" & os != "win32" & with-test}
"sherlodoc" {>= "0.2" & with-doc}
"odoc" {>= "2.4.1" & with-doc}
]
Expand Down
1 change: 1 addition & 0 deletions test/dune
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@
(modes js)
(build_if
(and
(<> %{architecture} i386)
(<> %{os_type} Win32)))
(action
;; It is fine if 'node:fs' cannot be found. js_of_ocaml>=5.9 does not like
Expand Down

0 comments on commit b9a990a

Please sign in to comment.