Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix the tests to be compatible with archlinux/arm64 #5252

Merged
merged 1 commit into from
Aug 26, 2022

Conversation

kit-ty-kate
Copy link
Member

Fixes everything except for this one, which i have no idea how to fix:

diff --git a/_build/default/tests/reftests/working-dir.test b/_build/default/tests/reftests/working-dir.out
index ae774ff60..0e82e5dd7 100644
--- a/_build/default/tests/reftests/working-dir.test
+++ b/_build/default/tests/reftests/working-dir.out
@@ -218,14 +218,14 @@ The following actions will be performed:
 <><> Processing actions <><><><><><><><><><><><><><><><><><><><><><><><><><><><>
 Processing  1/5: [qux.4: rsync]
 Processing  2/5: [qux.4: rsync] [ongoing: test newfile.txt]
+Processing  2/5: [qux.4: rsync] [ongoing: cat newfile.txt]
+Processing  2/5: [qux.4: rsync]
+Processing  3/5: [qux.4: rsync]
 -> retrieved qux.4  (file://${BASEDIR}/qux)
-Processing  2/5: [ongoing: test newfile.txt]
 test "-f" "newfile.txt" (CWD=${BASEDIR}/OPAM/working-dir/.opam-switch/build/ongoing.dev)
-Processing  2/5: [ongoing: cat newfile.txt]
 cat "newfile.txt" (CWD=${BASEDIR}/OPAM/working-dir/.opam-switch/build/ongoing.dev)
 - new!
 -> compiled  ongoing.dev
-Processing  3/5: [qux: test present]
 -> installed ongoing.dev
 Processing  4/5: [qux: test present]
 test "-f" "present" (CWD=${BASEDIR}/OPAM/working-dir/.opam-switch/build/qux.4)

@kit-ty-kate kit-ty-kate force-pushed the tests-archlinux-arm64 branch 2 times, most recently from da7f50c to 93b5644 Compare August 18, 2022 18:52
@kit-ty-kate kit-ty-kate merged commit 92058b4 into ocaml:master Aug 26, 2022
Out of release : doc, test, install, etc. automation moved this from PR to done Aug 26, 2022
@kit-ty-kate kit-ty-kate deleted the tests-archlinux-arm64 branch August 26, 2022 11:47
@AltGr
Copy link
Member

AltGr commented Aug 26, 2022

Ah yeah I've had one of these before. You can't disable parallelism between downloads and compilation tasks at the moment (these use two different pools of workers with different limits)

My workaround was just to grep the problematic lines out (stg like | "^Processing " -> "\c" ?)

@rjbou rjbou added this to the 2.2.0~alpha milestone Aug 29, 2022
@rjbou rjbou added this to PR in progress in Opam 2.2.0 via automation Aug 29, 2022
@rjbou rjbou moved this from PR in progress to Done in Opam 2.2.0 Aug 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

None yet

3 participants