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

(maint) Update acceptance to fix windows ruby install #3219

Merged
merged 1 commit into from
Jul 20, 2023

Conversation

mcdonaldseanp
Copy link
Contributor

@mcdonaldseanp mcdonaldseanp commented Jul 18, 2023

A couple things have changed about ruby and chocolatey, requiring new
changes to ensure the windows installation can succeed.

The newest versions of chocolately no longer require the --lo flag in
the choco list command, because that command now only lists local
packages instead of doing any remote searching.

Remove use of the flag so acceptance setup can complete

Update the ruby installation to use latest ruby, version 2.5 was unable
to build with new changes to the ruby devkit.

Add new steps to install msys2 and run ridk to install the ruby devkit,
which is required to build gems with C extensions

@mcdonaldseanp mcdonaldseanp requested a review from a team as a code owner July 18, 2023 16:35
A couple things have changed about ruby and chocolatey, requiring new
changes to ensure the windows installation can succeed.

The newest versions of chocolately no longer require the `--lo` flag in
the choco list command, because that command now only lists local
packages instead of doing any remote searching.

Remove use of the flag so acceptance setup can complete

Update the ruby installation to use latest ruby, version 2.5 was unable
to build with new changes to the ruby devkit.

Add new steps to install msys2 and run ridk to install the ruby devkit,
which is required to build gems with C extensions
@mcdonaldseanp mcdonaldseanp changed the title (maint) Remove -lo option from choco list (maint) Update acceptance to fix windows ruby install Jul 19, 2023
@mcdonaldseanp mcdonaldseanp merged commit a308978 into puppetlabs:main Jul 20, 2023
3 checks passed
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