Skip to content

ci: add macOS CI for LibreSSL / OpenSSL coverage#84

Merged
atoomic merged 1 commit intocpan-authors:mainfrom
atoomic:koan.atoomic/macos-ci
Mar 12, 2026
Merged

ci: add macOS CI for LibreSSL / OpenSSL coverage#84
atoomic merged 1 commit intocpan-authors:mainfrom
atoomic:koan.atoomic/macos-ci

Conversation

@Koan-Bot
Copy link
Copy Markdown
Contributor

@Koan-Bot Koan-Bot commented Mar 11, 2026

What

Add a macOS CI job to test against system LibreSSL and Homebrew OpenSSL 3.x.

Why

The LIBRESSL_VERSION_NUMBER and OLD_CRUFTY_SSL_VERSION code paths in RSA.xs (9 conditional blocks) have zero CI coverage today. macOS ships LibreSSL by default, making it the cheapest way to cover these paths. The Homebrew OpenSSL variant adds a second macOS data point for OpenSSL 3.x on ARM64.

How

  • Two matrix entries: system LibreSSL (no extra install) and Homebrew OpenSSL (brew install openssl@3 + OPENSSL_PREFIX env).
  • Gates on needs: [ubuntu] to fail fast on the cheap test.
  • cpanm installed manually (no perl-actions/install-with-cpm on macOS — no containers).
  • Renamed workflow from linuxtestsuite since it now covers three platforms.

Testing

YAML validated locally. CI will run on this PR branch — the macOS jobs themselves are the test.


Quality Report

Changes: 1 file changed, 46 insertions(+), 1 deletion(-)

Code scan: clean

Tests: passed (OK)

Branch hygiene: clean

Generated by Kōan post-mission quality pipeline

The LIBRESSL_VERSION_NUMBER and OLD_CRUFTY_SSL_VERSION code paths in
RSA.xs had no CI coverage. macOS ships LibreSSL by default, making it
a natural target for testing these paths cheaply.

Add a macOS job with two matrix entries:
- System LibreSSL (Apple's default, exercises OLD_CRUFTY_SSL_VERSION)
- Homebrew OpenSSL 3.x (sets OPENSSL_PREFIX, exercises OpenSSL 3.x paths)

The job gates on the ubuntu quick-test to stay cheap on failure.
Also rename the workflow from "linux" to "testsuite" since it now
covers Linux, Windows, and macOS.
@atoomic atoomic marked this pull request as ready for review March 12, 2026 02:20
@atoomic atoomic merged commit afc26e6 into cpan-authors:main Mar 12, 2026
55 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