Skip to content

deps(cmd): update go-internal dependency#364

Merged
rvagg merged 2 commits intomasterfrom
rvagg/go-internal-update
Feb 9, 2023
Merged

deps(cmd): update go-internal dependency#364
rvagg merged 2 commits intomasterfrom
rvagg/go-internal-update

Conversation

@rvagg
Copy link
Copy Markdown
Member

@rvagg rvagg commented Feb 8, 2023

Crossing my fingers and attempting to fix the new flaky macos CI failures for testing the cmd package. I can replicate it locally after enough test re-runs and haven't been able to replicate it with this update, but it's flaky so that doesn't prove anything.

@mvdan are you aware of anything that might be coming out of testscript that's causing flaky CI executions on macos-only? We see them pretty regularly now, e.g.:

They all have the pattern:

[signal: killed]
          FAIL: testdata/script/root.txt:1: unexpected command failure

where the the script varies, i.e. it happens across different scripts, just on macos.

@mvdan
Copy link
Copy Markdown
Contributor

mvdan commented Feb 8, 2023

Yeah, I'm afraid that's a weird but known bug for testscript on macos-12. See rogpeppe/go-internal#200. Downgrading to macos-11 is the only workaround I'm aware of, right now. I wrote all the details I know on that thread, so if any of you have a hunch or can investigate further on a MacOS 12 machine, that would be appreciated :)

@rvagg
Copy link
Copy Markdown
Member Author

rvagg commented Feb 9, 2023

great, thanks for looking @mvdan and for the ping in that thread, I'll watch progress and if I come up with any insights I'll chime in

@rvagg rvagg merged commit 755661c into master Feb 9, 2023
@rvagg rvagg deleted the rvagg/go-internal-update branch February 9, 2023 00:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants