Skip to content

ci(nightly): build Windows OpenSSL with Strawberry perl, not msys perl (closes #614) - #640

Merged
noahgift merged 5 commits into
mainfrom
fix/614-windows-strawberry-perl
Sep 26, 2026
Merged

noahgift merged 5 commits into
mainfrom
fix/614-windows-strawberry-perl

Conversation

@noahgift

Copy link
Copy Markdown
Contributor

The nightly x86_64-pc-windows-msvc leg runs ./Configure for vendored OpenSSL under shell: bash, which resolves Git's msys perl. That perl lacks Params::Check (BEGIN failed--compilation aborted at /usr/share/perl5/core_perl/Params/Check.pm, run 2026-09-25 09:31Z). openssl-sys fails, the leg is red, and the all-or-nothing release job is skipped. The nightly tag is now 290h behind main, and fleet-bins reports NIGHTLY-STALE on every hourly lambda run.

This change sets OPENSSL_SRC_PERL to the Strawberry perl that windows-latest ships. The step also fails loudly if that perl cannot load the module the build needs, so a future image change stays visible.

Not verified before merge: a branch workflow_dispatch of nightly.yml would publish the nightly release from the branch commit, so I did not dispatch. The first scheduled run after merge is the proof.

Pmat-Ticket: PMAT-614

🤖 Generated with Claude Code

noahgift and others added 2 commits September 26, 2026 00:29
closes #614)

The windows-msvc leg ran ./Configure under Git's msys perl, which lacks
Params::Check, so openssl-sys failed and the all-or-nothing release job
skipped: the nightly tag is 11+ days stale and fleet-bins reports
NIGHTLY-STALE on every lambda run. OPENSSL_SRC_PERL now names the
Strawberry perl windows-latest ships, and the step fails loudly if that
perl cannot load the module the build needs.

Pmat-Ticket: PMAT-614
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Executes the parsed "Build release binary" step under the shell GitHub uses
for `shell: bash`, with stub cargo/cross and a Strawberry perl planted (or
withheld) at the path the step names. Windows builds OpenSSL with it and
fails before cargo when it is unusable or missing; the Linux, aarch64-cross
and macOS legs leave OPENSSL_SRC_PERL unset. Three mutations were measured
red: delete the block, drop the probe's exit, make it unconditional. Wired
into ci.yml.

Pmat-Ticket: PMAT-614
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@noahgift

Copy link
Copy Markdown
Contributor Author

quorum-review (AD-04): NOT agreed (auto_merge: checked=true was_armed=false disarmed=false)

{
 "ticket": "PMAT-614",
 "head": "f78aef4305bacae8c3ea4750ced7db0d392a9fdb",
 "width": 3,
 "executor": "agy",
 "agreed": false,
 "auto_merge": {
  "checked": true,
  "was_armed": false,
  "disarmed": false,
  "note": "auto-merge not armed"
 },
 "lanes": [
  {
   "lane": 1,
   "verdict": "FAIL",
   "findings": 1
  },
  {
   "lane": 2,
   "verdict": "FAIL",
   "findings": 1
  },
  {
   "lane": 3,
   "verdict": "FAIL",
   "findings": 2
  }
 ]
}

…and release: 1.33.0

The first commit carried pmat's YAML round-trip: 326 lines reflowed and 68 kind: fields
stripped, the regression cb21xx-baseline.json names. All three quorum lanes refuted
it. The file is now origin/main plus one entry (22 insertions, 0 deletions), bound
to the issue's milestone 1.33.0.

Pmat-Ticket: PMAT-614
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@noahgift

Copy link
Copy Markdown
Contributor Author

quorum-review (AD-04): three PASS — agreed (auto_merge: checked=true was_armed=false disarmed=false)

{
 "ticket": "PMAT-614",
 "head": "215f2e8a2edffa0e2cacc4109cafd3bee559b30b",
 "width": 3,
 "executor": "agy",
 "agreed": true,
 "auto_merge": {
  "checked": true,
  "was_armed": false,
  "disarmed": false,
  "note": "auto-merge not armed"
 },
 "lanes": [
  {
   "lane": 1,
   "verdict": "PASS",
   "findings": 4
  },
  {
   "lane": 2,
   "verdict": "PASS",
   "findings": 0
  },
  {
   "lane": 3,
   "verdict": "PASS",
   "findings": 4
  }
 ]
}

noahgift and others added 2 commits September 26, 2026 03:25
Pmat-Ticket: PMAT-614
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…/3 round 1 on the roadmap round-trip)

Receipt binds diff_sha256 52ed34bf5b31e6057caf287ac7a762ee9f72cd07 against
c1e49a3 and the four PMAT-614 evidence files by blob, sha256 and bytes.

Pmat-Ticket: PMAT-614
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@noahgift
noahgift enabled auto-merge (squash) September 26, 2026 03:00
@noahgift
noahgift merged commit 96ce604 into main Sep 26, 2026
28 checks passed
@noahgift
noahgift deleted the fix/614-windows-strawberry-perl branch September 26, 2026 03:24
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.

1 participant