Skip to content

Conversation

@MSP-Greg
Copy link
Contributor

Today the ruby head mswin build broke from fiddle. ruby/ruby appears to download libffi 3.2.1 for the mswin build, but 3.3 is vendored here.

I started here and created a GitHub Actions workflow, covering Ruby 2.3 thru master/head for Ubuntu, macOS, and Windows. All are passing except Windows mswin. I did add a compile step just to see what was happening.

This may not run, as often the first PR with a GHA workflow doesn't. A link to a run in my fork is:
https://github.com/MSP-Greg/fiddle/actions/runs/116195157

@kou
Copy link
Member

kou commented May 31, 2020

It seems that "windows mswin" is failed: https://github.com/MSP-Greg/fiddle/runs/711008486?check_suite_focus=true

What should we do for this?

@kou
Copy link
Member

kou commented May 31, 2020

Ah, you want to show "windows mswin" is failed.

@kou
Copy link
Member

kou commented May 31, 2020

Should we merge this and fix the problem?

@MSP-Greg
Copy link
Contributor Author

MSP-Greg commented May 31, 2020

Ah, you want to show "windows mswin" is failed.

Yes. Sorry, I should have opened an issue for that, rather than noting it in this PR. Also, the issue with ext/fiddle/extlibs using 3.2.1, both here and in ruby/ruby.

Should we merge this and fix the problem?

Re merging, whatever you prefer. The CI runs quickly, so I used a 'complete' matrix of jobs. As to fixing the problem, my c knowledge is limited, so I wouldn't be much help. Thanks.

@kou
Copy link
Member

kou commented May 31, 2020

OK.
I'll merge this.

@kou kou merged commit 2ed66a8 into ruby:master May 31, 2020
@MSP-Greg MSP-Greg deleted the gha branch June 2, 2020 03:43
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.

2 participants