Stream: wasi

Topic: libc CI windows


view this post on Zulip John Ericson (Mar 21 2022 at 22:08):

OK last thing, the windows tests fail. But it never set WASM_CC on windows. I am trying unset CC and if that doens't work, I will just do CC=clang, and then it will work.

view this post on Zulip John Ericson (Mar 21 2022 at 22:09):

(The problem is that CC=cc in the windows env, so CC ?= clang didn't kick in whereas WASM_CC ?= clang did.

view this post on Zulip John Ericson (Mar 21 2022 at 22:10):

Error: Invalid environment variable format 'unset CC'

view this post on Zulip John Ericson (Mar 21 2022 at 22:10):

OK so plan B, will push again in a few second.

view this post on Zulip Dan Gohman (Mar 21 2022 at 22:10):

/me gives Windows a sideways glance

view this post on Zulip John Ericson (Mar 21 2022 at 22:11):

I bet they all had CC=cc, to begin with, but the others were setting CC / WASM_CC already.

view this post on Zulip John Ericson (Mar 21 2022 at 22:16):

I see the mac error. I am going to check github docs to see if it is just possible to start with a blanket set of env vars

view this post on Zulip John Ericson (Mar 21 2022 at 22:24):

That doesn't appear to exist lol https://stackoverflow.com/questions/70137245/how-to-remove-an-environment-variable-on-github-actions yay ambient authority "just ignore the env vars you don't care about, what's the problem?!"

Context On https://docs.github.com/en/actions/learn-github-actions/environment-variables they describe a way to add an environment variable with yaml: jobs: job_name: env: NAME: value On

view this post on Zulip John Ericson (Mar 21 2022 at 22:25):

OK @Dan Gohman one more push

view this post on Zulip John Ericson (Mar 21 2022 at 22:25):

oh you are really fast or I am slow :)

view this post on Zulip John Ericson (Mar 21 2022 at 22:25):

it's running

view this post on Zulip Dan Gohman (Mar 21 2022 at 22:37):

I don't quite understand your last comment

view this post on Zulip John Ericson (Mar 21 2022 at 22:38):

I thought you had to approve me each time. And I either i was wrong about that, or you are doing it really fast :)

view this post on Zulip John Ericson (Mar 21 2022 at 22:38):

undefinod-symbols.txt is in the macOS CI. WTF. Bad flash or memory?

view this post on Zulip John Ericson (Mar 21 2022 at 22:41):

I didn't make that typo according to git on my end!

view this post on Zulip Dan Gohman (Mar 21 2022 at 22:44):

woah... "undefinod" is coming from nowhere?

view this post on Zulip Dan Gohman (Mar 21 2022 at 22:46):

I'm rerunning that job to see if it was somehow a spurious failure

view this post on Zulip Dan Gohman (Mar 21 2022 at 22:46):

But that's really bizarre

view this post on Zulip Dan Gohman (Mar 21 2022 at 22:47):

(also, yes, I was just clicking the button fast :-))

view this post on Zulip John Ericson (Mar 21 2022 at 22:49):

I think this might be the first time in my life I hit some corrupted memory! Unless it was a macOS file system bug.

view this post on Zulip Dan Gohman (Mar 21 2022 at 22:54):

Looks like it failed again, the same way?

view this post on Zulip John Ericson (Mar 21 2022 at 22:57):

Huh. Well I just pushed a new version tweaking Windows CI a a bit. Curious after the files are modified a bit (just to shuffle things around) we still get this issue.

view this post on Zulip John Ericson (Mar 21 2022 at 23:18):

https://github.com/WebAssembly/wasi-libc/pull/273 I decided to just split up the PR to bisect this bug

This is less of a sledgehammer, and prepares the way for #270.

view this post on Zulip John Ericson (Mar 21 2022 at 23:18):

(or do something more different to try to avoid this corruption!)

view this post on Zulip John Ericson (Mar 21 2022 at 23:39):

https://github.com/WebAssembly/wasi-libc/pull/274 another one

Progress towards #270

Last updated: Dec 23 2024 at 12:05 UTC