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

Test installation on Windows without precompiled code #544

Merged
merged 3 commits into from
Dec 3, 2023

Conversation

andyundso
Copy link
Contributor

@andyundso andyundso commented Nov 28, 2023

This PR adds jobs to the CircleCI pipeline to test the fix submitted by @Largo in #536. The jobs install the gem on Windows without any precompiled code and executes it afterwards.

This branch also includes the commit from @Largo which fixes the installation.

This new job does not run with Ruby 2.4. The msys2 installation for older Ruby versions is old as well, but from Ruby 2.5, we can update it using a key refresh. With Ruby 2.4, pacman-key was no available in the shipped msys2 version. I tried to manually install a new version of msys2. Then, ridk install 2 3 failed because it tried to install quite old versions of automake. There was a fix for this as well, but if we need to go such lengths for testing on EOL Ruby versions, I rather don't do it.

andyundso and others added 3 commits December 3, 2023 15:43
…d (C:/Ruby32-x64/msys64/ucrt64/include/freetds)

(cherry picked from commit eb5f8ed)
I tried couple of things in order to get msys2 to run with Ruby 2.4, going so far that I even manually installed msys2. However, this also failed, but I found a StackOverflow thread about it:

https://stackoverflow.com/questions/73985340/error-target-not-found-when-trying-to-install-msys2-and-mingw-development-toolc

But this seems to much to me: If we need to go such lengths in order to test a "minor" thing for one Ruby version, I rather don't do it.
@andyundso
Copy link
Contributor Author

as discussed with @aharpervc via mail, we want to include this fix in the next release, so I'm going ahead and merge this branch.

@andyundso andyundso merged commit 089694f into rails-sqlserver:master Dec 3, 2023
23 checks passed
@andyundso andyundso deleted the install-on-windows branch December 3, 2023 14:55
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.

None yet

2 participants