Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add GitHub Actions, fix Windows 3.1 & mswin #182

Closed
wants to merge 7 commits into from

Conversation

MSP-Greg
Copy link

@MSP-Greg MSP-Greg commented May 23, 2022

  1. Actions CI - similar to the code in PR Use GitHub Actions for CI testing #178, but only the workflow file. Updated Rubies, etc. Note that the coverage specific steps all failed with the other changes, so I added if: false to skip them for now.

  2. Added one skip. Frustrating. Sometimes Windows get squirrelly when a file is opened more than once, which the skipped spec does. But, running locally on Windows 11 and Ruby master, I could not repo the failure. Tried several things, and none fixed the CI failure. Maybe.later.

  3. Publicly available Windows Rubies 2.4 thru 3.0 are all built with the MSYS2 mingw tool chain. Starting with Windows Ruby 3.1, builds are using the MSYS2 ucrt tool chain. The Windows fix only affects 3.1 and mswin builds.

Feel free to grab the Windows fix and add to PR #178.

@MSP-Greg MSP-Greg changed the title Add GitHub Actions, fix Windows Add GitHub Actions, fix Windows 3.1 & mswin May 23, 2022
@MSP-Greg
Copy link
Author

Re the second point above, the error shown is

     ChildProcess::LaunchError:
       Access is denied. (5)

I assumed that was a file error. I rewrote the test to use a UDP socket to pass the pid needed for the spec, and it still had the same error in Actions. But, as with the existing spec, the UDP version also passed locally on Windows 11 desktop. So, I'm not sure why it's failing on the Actions CI runners.

mvz added a commit to cucumber/aruba that referenced this pull request Dec 25, 2022
ChildProcess does not play nice with Ruby 3.1 build for Windows used in
GitHub Actions. See enkessler/childprocess#182
chelnak added a commit to chelnak/childprocess that referenced this pull request Mar 31, 2023
This commit updates msvcrt_name to return the value of RUBY_SO_NAME or
ucrtbase.

Change taken from enkessler#182.
sds pushed a commit that referenced this pull request Dec 13, 2023
This commit updates msvcrt_name to return the value of RUBY_SO_NAME or
ucrtbase.

Change taken from #182.
@sds sds marked this pull request as ready for review December 13, 2023 07:26
@sds sds mentioned this pull request Dec 13, 2023
@sds
Copy link
Collaborator

sds commented Jan 7, 2024

Thank you! Superseded by #175.

@sds sds closed this Jan 7, 2024
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