Stream: git-wasmtime

Topic: wasmtime / PR #10790 Migrate run-tests.sh to Python


view this post on Zulip Wasmtime GitHub notifications bot (May 17 2025 at 22:36):

alexcrichton requested cfallin for a review on PR #10790.

view this post on Zulip Wasmtime GitHub notifications bot (May 17 2025 at 22:36):

alexcrichton opened PR #10790 from alexcrichton:try-to-fix-mingw to bytecodealliance:main:

Try to fix MinGW breakage in CI

prtest:full

<!--
Please make sure you include the following information:

Our development process is documented in the Wasmtime book:
https://docs.wasmtime.dev/contributing-development-process.html

Please ensure all communication follows the code of conduct:
https://github.com/bytecodealliance/wasmtime/blob/main/CODE_OF_CONDUCT.md
-->

view this post on Zulip Wasmtime GitHub notifications bot (May 17 2025 at 22:36):

alexcrichton requested wasmtime-default-reviewers for a review on PR #10790.

view this post on Zulip Wasmtime GitHub notifications bot (May 17 2025 at 22:38):

alexcrichton updated PR #10790.

view this post on Zulip Wasmtime GitHub notifications bot (May 17 2025 at 22:38):

alexcrichton commented on PR #10790:

Another attempt after https://github.com/bytecodealliance/wasmtime/pull/10787

Some testing has shown that gcc -E foo.c fails when the shell: bash parameter is set in CI. All other shells work though. I don't know what's going on so I just want to get out of bash/MinGW and try to get to the point where we just execute the compiler in isolation.

view this post on Zulip Wasmtime GitHub notifications bot (May 17 2025 at 22:43):

alexcrichton commented on PR #10790:

Ok I'm clearly too tired for this.

view this post on Zulip Wasmtime GitHub notifications bot (May 18 2025 at 06:57):

alexcrichton updated PR #10790.

view this post on Zulip Wasmtime GitHub notifications bot (May 18 2025 at 07:22):

alexcrichton commented on PR #10790:

Ok I wasn't able to use shell: python but I was able to use a Python script with the pwsh shell to avoid unduly modifying PATH on MinGW

view this post on Zulip Wasmtime GitHub notifications bot (May 18 2025 at 12:14):

alexcrichton updated PR #10790.

view this post on Zulip Wasmtime GitHub notifications bot (May 18 2025 at 16:54):

cfallin submitted PR review:

Wow -- everything about "we can't use bash because it breaks the environment on the pre-set-up runner" is cursed, but looks like this works, but this seems to work so ship it!

view this post on Zulip Wasmtime GitHub notifications bot (May 18 2025 at 17:16):

cfallin merged PR #10790.


Last updated: Dec 06 2025 at 07:03 UTC