-
Notifications
You must be signed in to change notification settings - Fork 188
Remove old Ruby version from CI config & cross build rake task #455
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
Remove old Ruby version from CI config & cross build rake task #455
Conversation
@larskanis looks like you were also doing this work here: master...larskanis:update-windows-binary, was there a blocker on that? |
Can we perhaps add support for Ruby 2.7 while we're at it? |
I don't see why not |
@aharpervc I don't remember what interrupted me. Nevertheless you need to update rake-compiler-dock to 1.0 in order to cross build for Ruby-2.7. OpenSSL-1.1.1 should work as well. We cross build it in ruby-pg. |
Appveyor seems to be disabled for this repository. The badge icon in the README shows the last build two years ago. |
Great @larskanis thanks for the tip. I removed the old appveyor config and updated rake-compiler-dock & the default OpenSSL version. |
The 2.7 job is failing because tiny tds can't find the freetds headers (I think...) indicated by this right before the final error:
However, I don't know why that would be. Freetds appears to install without problem, and diffing the 2.6 & 2.7 job logs that part of each job seems to be similar. @larskanis ideas? |
Given that 2.7 is requires additional changes, I'm going to slice that work out to another branch so we can get the main stuff merged. |
@aharpervc Couldn't you reactivate Appveyor-CI instead of dropping it? That would make sure, that the gem is properly working on Windows. Or is there a particular reason to not use Appveyor any longer? |
ae44521
to
e75a379
Compare
I feel like I asked this question somewhere on this repo a long time ago, and there was an answer, but I can't recall what it was & I can't find my question. I'm open to adding more CI config, especially for Windows. Whether that would be back to appveyor or some other service is an open question to me. Travis has Windows support now I believe, and CircieCI supports Linux/macOS/Windows and might be a better choice overall. Azure Devops also supports all OS's. Github Actions can be used for CI and also supports everything. There's lots to consider. |
Hey @aharpervc is there any change we might soon see a new release of this gem with the support for 2.6 and precompiled windows gems? |
I hope so, although this branch has been superseded by #456 |
This is a prerequisite for #434 (per comment). It's effectively a followup to #428.
Adding recent contributors for review.