Skip to content

Conversation

@Artoria2e5
Copy link
Contributor

Turns out GitHub has a RUNNER_ARCH variable for precisely this need. This will hold so long as we only run one runner per architecture (we make the same assumption on Linux and Windows already).

This should fix #51.

Turns out GitHub has a RUNNER_ARCH variable for precisely this need. This will hold so long as we only run one runner per architecture (we make the same assumption on Linux and Windows already).

This should fix primesearch#51.
@Artoria2e5
Copy link
Contributor Author

Yikes, this isn't working.

mfakto-macos-

@Artoria2e5
Copy link
Contributor Author

Aha... so env only works for custom-defined variables. Welp, so much for convenience.

@Artoria2e5
Copy link
Contributor Author

Artoria2e5 commented Nov 22, 2025

GitHub people says macos-13 is on the way out. In that case let me try the 15 thing. Holy scope creeps...

actions/runner-images#13046

@Artoria2e5
Copy link
Contributor Author

Eh, it works!

@Artoria2e5 Artoria2e5 changed the title Update macOS naming to use RUNNER_ARCH variable Update macOS naming to use RUNNER_ARCH variable; replace runner that will stop working in a month Nov 22, 2025
@Artoria2e5 Artoria2e5 changed the title Update macOS naming to use RUNNER_ARCH variable; replace runner that will stop working in a month Update macOS naming to use RUNNER_ARCH variable; replace runner that will stop working in two weeks Nov 22, 2025
@Artoria2e5 Artoria2e5 changed the title Update macOS naming to use RUNNER_ARCH variable; replace runner that will stop working in two weeks Update macOS naming to use CPU architecture; replace runner that will stop working in two weeks Nov 22, 2025
@ixfd64 ixfd64 merged commit 38e43c0 into primesearch:master Nov 22, 2025
5 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.

Misleading macOS release filename

2 participants