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

CI: crux-llvm nightly Docker job is broken #1257

Closed
RyanGlScott opened this issue Sep 24, 2024 · 0 comments · Fixed by #1258
Closed

CI: crux-llvm nightly Docker job is broken #1257

RyanGlScott opened this issue Sep 24, 2024 · 0 comments · Fixed by #1258
Labels
CI Docker Issues relating to the Docker images.

Comments

@RyanGlScott
Copy link
Contributor

The changes from #1256 inadvertently broke the nightly crux-llvm Docker job, as seen here:

#31 45.61           clang-range=recent-clang
#31 45.61             z3 4.8.14
#31 45.61               write-stdout
#31 45.61                 write-stdout crux run:                   OK
#31 45.61                 write-stdout crux output:                FAIL
#31 45.61                   test/Test.hs:326:
#31 45.61                   MISMATCH for expected (test-data/golden/stdio/write-stdout.z3.good)
#31 45.61                              and actual (test-data/golden/stdio/write-stdout.z3.z3.out) output:
#31 45.61                       F-expect>|output
#31 45.61                       F-actual>|clang-12: readCreateProcessWithExitCode: posix_spawnp: does not exist (No such file or directory)
#31 45.61                       F-expect>|[Crux] Overall status: Valid.
#31 45.61                   
#31 45.61                   Use -p '/loop-merging=loopmerge.clang-range=recent-clang.z3 4.8.14.write-stdout.write-stdout crux output/' to rerun this test only.
#31 45.61 
#31 45.61 1337 out of 2335 tests failed (6.12s)
#31 45.61 
#31 45.61 Test suite crux-llvm-test: FAIL
#31 45.61 Test suite logged to:
#31 45.61 /crux-llvm/build/dist-newstyle/build/x86_64-linux/ghc-9.4.8/crux-llvm-0.9.0.99/t/crux-llvm-test/test/crux-llvm-0.9.0.99-crux-llvm-test.log
#31 45.61 0 of 1 test suites (0 of 1 test cases) passed.
#31 ERROR: process "/bin/sh -c cabal v2-test crux-llvm" did not complete successfully: exit code: 1

Judging from the clang-12: readCreateProcessWithExitCode: posix_spawnp: does not exist (No such file or directory) part of the error message, the Dockerfile is expecting clang-12 when it should actually be looking for clang-14.

@RyanGlScott RyanGlScott added the CI label Sep 24, 2024
RyanGlScott added a commit that referenced this issue Sep 24, 2024
Previously, the Dockerfile was inconsistent about whether it referred to LLVM
12 or 14. This patch changes the Dockerfile to uniformly refer to the current
LLVM version using an `LLVM_VER` `ARG` (defined to be `14`), which is passes to
both stages of the build.

Fixes #1257.
RyanGlScott added a commit that referenced this issue Sep 24, 2024
Previously, the Dockerfile was inconsistent about whether it referred to LLVM
12 or 14. This patch changes the Dockerfile to uniformly refer to the current
LLVM version using an `LLVM_VER` `ARG` (defined to be `14`), which is passes to
both stages of the build.

Fixes #1257.
@RyanGlScott RyanGlScott added the Docker Issues relating to the Docker images. label Oct 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Docker Issues relating to the Docker images.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant