Skip to content

Integrate Cygwin setup into build_vanagon.yml workflow - #125

Merged
Sharpie merged 3 commits into
OpenVoxProject:mainfrom
Sharpie:integrate-cygwin-setup
Aug 5, 2026
Merged

Integrate Cygwin setup into build_vanagon.yml workflow#125
Sharpie merged 3 commits into
OpenVoxProject:mainfrom
Sharpie:integrate-cygwin-setup

Conversation

@Sharpie

@Sharpie Sharpie commented Aug 5, 2026

Copy link
Copy Markdown
Member

Short description

This changeset integrates the various setup.ps1 scripts that were scattered about puppet-runtime, openvox, and openbolt into steps in the build_vanagon.yml workflow.

This change also fixes build failures that resulted from the retirement of Ruby 3.2 from ustream Cygwin repositories. vanagon build for OpenVox 8.x and OpenBolt releases is now run under the latest Ruby that Cygwin provides (currently 4.0.6).

There is also a bit of re-factoring to clean up and consolidate build logic between the Cygwin and MSYS2 build paths.

Checklist

I have:

Sharpie added 3 commits August 5, 2026 09:36
Prior to this commit, the `puppet-runtime`, `openvox,` and `openbolt`
repositories each carried a `setup.ps1` script that installed Cygwin
and its dependencies. These scripts were nearly identical, with the
only variation being that `puppet-runtime` did not install Wix.

This commit moves the Cygwin setup into the build_vanagon workflow,
drops pins on the Cygwin Ruby 3.2 packages that no longer exist
upstream, and adds a new dependency on the rubygems package.

Wix installation is consolidated in a setup action that is shared
with the modern MSYS2 build.

Signed-off-by: Charlie Sharpsteen <charlie@overlookinfratech.com>
This commit consolidates Windows setup actions that have to run in
either the Cygwin or MSYS2 shells:

  - Disable bundler concurrency, otherwise `bundle install` looses
    track of completed jobs and hangs.

  - Add a `root` entry to `/etc/passwd` to make Vanagon happy.

Signed-off-by: Charlie Sharpsteen <charlie@overlookinfratech.com>
The `git` commands that set CRLF behavior are failing when run after
Cygwin installation, yet Cygwin builds are succeeding despite heavy
use of patches. Continued success indicates the override is no longer
required to make `patch.exe` happy.

Signed-off-by: Charlie Sharpsteen <charlie@overlookinfratech.com>
@Sharpie

Sharpie commented Aug 5, 2026

Copy link
Copy Markdown
Member Author

Successful build of agent-runtime-main, exercising the MSYS2 pathway:

https://github.com/Sharpie/puppet-runtime/actions/runs/31021050641/job/92357589668

Successful build of openbolt-runtime, exercising the Cygwin pathway:

https://github.com/Sharpie/puppet-runtime/actions/runs/31025974540/job/92374497443

@Sharpie
Sharpie enabled auto-merge August 5, 2026 17:50
@Sharpie
Sharpie merged commit 90cdeae into OpenVoxProject:main Aug 5, 2026
2 checks passed
@Sharpie
Sharpie deleted the integrate-cygwin-setup branch August 5, 2026 17:51
Sharpie added a commit to Sharpie/puppet-runtime that referenced this pull request Aug 5, 2026
The logic in this script has been integrated into the shared-actions
repository by PR OpenVoxProject/shared-actions#125 and the updated
workflow no longer calls down to `setup.ps1`.

Signed-off-by: Charlie Sharpsteen <charlie@overlookinfratech.com>
Sharpie added a commit to OpenVoxProject/openbolt that referenced this pull request Aug 5, 2026
The logic in this script has been integrated into the shared-actions
repository by PR OpenVoxProject/shared-actions#125 and the updated
workflow no longer calls down to `setup.ps1`.

Signed-off-by: Charlie Sharpsteen <charlie@overlookinfratech.com>
Sharpie added a commit to OpenVoxProject/openvox that referenced this pull request Aug 5, 2026
The logic in this script has been integrated into the shared-actions
repository by PR OpenVoxProject/shared-actions#125 and the updated
workflow no longer calls down to `setup.ps1`.

Signed-off-by: Charlie Sharpsteen <charlie@overlookinfratech.com>
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