Stream: git-wasmtime

Topic: wasmtime / PR #5287 Split CMake tests into a separate step


view this post on Zulip Wasmtime GitHub notifications bot (Nov 17 2022 at 00:23):

elliottt opened PR #5287 from trevor/remove-cmake-from-test to main:

Split the CMake tests for exercising the c-api out into a separate step. There
doesn't appear to be any reuse between build artifacts from the CMake-based
tests and the cargo test step later on in the Test job, so this should speed
up the average build time.

Additionally, this PR only runs the CMake tests on ubuntu and windows. I'd be
happy to expand the matrix out to the same set as the Test job if we think
it's worth it.
<!--

Please ensure that the following steps are all taken care of before submitting
the PR.

Please ensure all communication adheres to the code of conduct.
-->

view this post on Zulip Wasmtime GitHub notifications bot (Nov 17 2022 at 00:41):

elliottt requested alexcrichton for a review on PR #5287.

view this post on Zulip Wasmtime GitHub notifications bot (Nov 17 2022 at 00:45):

jameysharp submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Nov 17 2022 at 00:45):

jameysharp submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Nov 17 2022 at 00:45):

jameysharp created PR review comment:

I suppose you could remove the matrix.target bits since this job doesn't try cross-compiling anything. Anyone wanting to test the C API on, say, riscv would need to copy all the qemu machinery from the other job, so expecting them to also add matrix.target in appropriate places seems okay too.

view this post on Zulip Wasmtime GitHub notifications bot (Nov 17 2022 at 00:49):

elliottt submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Nov 17 2022 at 00:49):

elliottt created PR review comment:

Yeah, good point!

view this post on Zulip Wasmtime GitHub notifications bot (Nov 17 2022 at 00:50):

elliottt updated PR #5287 from trevor/remove-cmake-from-test to main.

view this post on Zulip Wasmtime GitHub notifications bot (Nov 17 2022 at 00:53):

elliottt has marked PR #5287 as ready for review.

view this post on Zulip Wasmtime GitHub notifications bot (Nov 17 2022 at 00:57):

alexcrichton submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Nov 17 2022 at 00:57):

alexcrichton submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Nov 17 2022 at 00:57):

alexcrichton created PR review comment:

It's ok to skip these steps since that's only required for one builder to do it.

view this post on Zulip Wasmtime GitHub notifications bot (Nov 17 2022 at 00:57):

alexcrichton created PR review comment:

I know windows is weird but we're tight enough on CI budget I'd be ok to drop windows here and just test on Linux

view this post on Zulip Wasmtime GitHub notifications bot (Nov 17 2022 at 01:14):

elliottt updated PR #5287 from trevor/remove-cmake-from-test to main.

view this post on Zulip Wasmtime GitHub notifications bot (Nov 17 2022 at 05:23):

elliottt updated PR #5287 from trevor/remove-cmake-from-test to main.

view this post on Zulip Wasmtime GitHub notifications bot (Nov 17 2022 at 06:04):

elliottt updated PR #5287 from trevor/remove-cmake-from-test to main.

view this post on Zulip Wasmtime GitHub notifications bot (Nov 17 2022 at 17:30):

elliottt closed without merge PR #5287.


Last updated: Dec 23 2024 at 13:07 UTC