Skip to content

CI: Use fiddle 1.0.8 for Ruby 3.0 on Windows#3392

Merged
ashie merged 2 commits into
fluent:masterfrom
ashie:fix-ruby3-win-tests
May 25, 2021
Merged

CI: Use fiddle 1.0.8 for Ruby 3.0 on Windows#3392
ashie merged 2 commits into
fluent:masterfrom
ashie:fix-ruby3-win-tests

Conversation

@ashie

@ashie ashie commented May 25, 2021

Copy link
Copy Markdown
Member

Which issue(s) this PR fixes:
Fixes #3263

What this PR does / why we need it:
On Ruby 3.0, we need to use fiddle 1.0.8 or later to retrieve correct
error code. In addition, we have to specify the path of fiddle by RUBYLIB
because RubyInstaller loads Ruby's bundled fiddle before initializing gem.
See also:

Docs Changes:
none

Release Note:
none

On Ruby 3.0, we need to use fiddle 1.0.8 or later to retrieve correct
error code. In addition, we have to specify the path of fiddle by RUBYLIB
because RubyInstaller loads Ruby's bundled fiddle before initializing gem.
See also:
* ruby/fiddle#72
* https://bugs.ruby-lang.org/issues/17813
* https://github.com/oneclick/rubyinstaller2/blob/8225034c22152d8195bc0aabc42a956c79d6c712/lib/ruby_installer/build/dll_directory.rb

Fix fluent#3263

Signed-off-by: Takuro Ashie <ashie@clear-code.com>
@ashie ashie requested a review from kenhys May 25, 2021 09:04
@kenhys

kenhys commented May 25, 2021

Copy link
Copy Markdown
Contributor

I'll check it.

Comment thread .github/workflows/windows-test.yaml
Signed-off-by: Takuro Ashie <ashie@clear-code.com>

@kenhys kenhys left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ashie

ashie commented May 25, 2021

Copy link
Copy Markdown
Member Author

Passed on Ruby 3.0.1 on Windows ㊗️

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.

Some unit tests are broken on Ruby 3.0

2 participants