Skip to content

CI: Fix Octave installation on MacOS#346

Merged
blink1073 merged 2 commits into
mainfrom
blink1073-patch-2
Jun 30, 2026
Merged

CI: Fix Octave installation on MacOS#346
blink1073 merged 2 commits into
mainfrom
blink1073-patch-2

Conversation

@blink1073

@blink1073 blink1073 commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Description

The test-other-systems macOS job restores a cached Homebrew Octave Cellar archive (key macos-brew-octave-11.3.0-image-) to skip rebuilding Octave on every run. That cache does not pin to a specific runner image version, so when GitHub bumps the bundled gcc version in the macos-15-arm64 runner image, the restored Octave binaries (linked against the old gcc's libgfortran.5.dylib via the current symlink) no longer resolve, and octave --version aborts with a dyld load failure.

Changes

  • After restoring the cached Octave Cellar, run brew reinstall octave so Octave is rebuilt/relinked against whichever gcc is actually present on the runner image, instead of trusting a stale cached binary's dependency links.
  • This trades a bit of cache effectiveness for correctness: the cache still saves the Octave download/build time when the toolchain hasn't moved, but brew reinstall self-heals the install whenever it has.

Backwards-incompatible changes

None

Testing

CI — verified the test-other-systems (macos-latest) job runs octave --version and available_graphics_toolkits successfully after cache restore + reinstall.

AI usage

  • Some or all of the content of this PR was generated by AI.
  • The human author has carefully reviewed this PR and run this code.
  • AI tools and models used: None

@codecov

codecov Bot commented Jun 30, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.90%. Comparing base (37ee303) to head (1a50bdc).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #346   +/-   ##
=======================================
  Coverage   92.90%   92.90%           
=======================================
  Files           6        6           
  Lines         437      437           
=======================================
  Hits          406      406           
  Misses         31       31           
Flag Coverage Δ
linux 92.67% <ø> (ø)
windows 92.90% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@blink1073
blink1073 merged commit 4ab7438 into main Jun 30, 2026
52 checks passed
@blink1073
blink1073 deleted the blink1073-patch-2 branch June 30, 2026 13:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant