Skip to content

Upgrade from deprecated macos-13 to macos-15-intel in CI#590

Closed
cclauss wants to merge 1 commit into
apache:masterfrom
cclauss:patch-1
Closed

Upgrade from deprecated macos-13 to macos-15-intel in CI#590
cclauss wants to merge 1 commit into
apache:masterfrom
cclauss:patch-1

Conversation

@cclauss

@cclauss cclauss commented Nov 5, 2025

Copy link
Copy Markdown
Contributor

Replace GitHub Actions runner image macos-13 with macos-15-intel as recommended in:

The macOS 13 runner image will be retired by December 4th, 2025.

The currently available GitHub Actions macOS runners are:

macOS Version runner.arch
macos-13 X64
macos-14 ARM64
macos-15 ARM64
macos-15-intel X64
macos-26 ARM64
macos-latest ARM64

NO AI WAS USED TO CREATE THIS PULL REQUEST.

@garydgregory

Thanks for your contribution to Apache Commons! Your help is appreciated!

Before you push a pull request, review this list:

  • Read the contribution guidelines for this project.
  • Read the ASF Generative Tooling Guidance if you use Artificial Intelligence (AI).
  • I used AI to create any part of, or all of, this pull request.
  • Run a successful build using the default Maven goal with mvn; that's mvn on the command line by itself.
  • Write unit tests that match behavioral changes, where the tests fail if the changes to the runtime are not applied. This may not always be possible, but it is a best practice.
  • Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
  • Each commit in the pull request should have a meaningful subject line and body. Note that a maintainer may squash commits during the merge process.

Replace GitHub Actions runner image `macos-13` with `macos-15-intel` as recommended in:
* https://github.blog/changelog/2025-09-19-github-actions-macos-13-runner-image-is-closing-down
> The macOS 13 runner image will be retired by ___December 4th, 2025___.

The currently available GitHub Actions macOS runners are:

| macOS Version | runner.arch |
|---------------|-------------|
| macos-13 | X64 |
| macos-14 | ARM64 |
| macos-15 | ARM64 |
| macos-15-intel | X64 |
| macos-26 | ARM64 |
| macos-latest | ARM64 |
@sebbASF

sebbASF commented Nov 5, 2025

Copy link
Copy Markdown
Contributor

AFAIK, macos-13 was only available for intel; there is no particular reason to stick with Intel rather than choosing the new default architecture which is Arm64.
So macos-14, -15 or -latest would be fine.

Also, macos-26 is probably a typo for macos-16

@bindul

bindul commented Nov 5, 2025

Copy link
Copy Markdown

AFAIK, macos-13 was only available for intel; there is no particular reason to stick with Intel rather than choosing the new default architecture which is Arm64.

So macos-14, -15 or -latest would be fine.

Also, macos-26 is probably a typo for macos-16

Apple moved to a unified year based versions across all its OSes. All major releases for MacOS, iOS, WatchOS, etc. released this year (September on) use major version number 26.

https://support.apple.com/en-us/109033

cclauss added a commit to cclauss/commons-csv that referenced this pull request Nov 5, 2025
@cclauss

cclauss commented Nov 5, 2025

Copy link
Copy Markdown
Contributor Author

Please review an alternative approach:

@ppkarwasz

Copy link
Copy Markdown
Member

I also would prefer to upgrade to macos-latest. There is only one catch for that: there is no Temurin JDK 8 version for ARM64, so:

  • we either need to switch to another distribution (e.g. zulu), or

  • we need to use zulu for just that specific case:

    - name: Set up JDK ${{ matrix.java }}
      uses: actions/setup-java@dded0888837ed1f317902acf8a20df0ad188d165 # v5.0.0
      with:
        distribution: ${{ runner.arch == 'ARM64' && matrix.java == '8' && 'zulu' || 'temurin' }}
        java-version: ${{ matrix.java }}

@cclauss

cclauss commented Nov 5, 2025

Copy link
Copy Markdown
Contributor Author

@cclauss cclauss closed this Nov 5, 2025
@cclauss cclauss deleted the patch-1 branch November 5, 2025 11:34
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.

4 participants