Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issue/gh action macos aarch64 #3669

Merged
merged 1 commit into from
May 17, 2024

Conversation

ikappaki
Copy link
Contributor

@ikappaki ikappaki commented May 17, 2024

Hi,

Could you please review the patch to add a separate flow in the CI for macOS on Apple Silicon? It fixes #3668

The macos-latest runner now targets Apple Silicon, and nix-emacs-ci supports version 28.x and above on this platform. I don't see this as an issue since I've also restored the old flow on the Intel macos 13.

By the way, as I've mentioned on other occasions, my personal opinion is that it is sufficient to only run/support the latest Emacs version on non-Unix platforms. :)

actions/runner-images#9741

purcell/nix-emacs-ci@ff5333a#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5

Thanks


Before submitting the PR make sure the following things have been done (and denote this
by checking the relevant checkboxes):

  • The commits are consistent with our contribution guidelines
  • You've added tests (if possible) to cover your change(s)
  • All tests are passing (eldev test)
  • All code passes the linter (eldev lint) which is based on elisp-lint and includes
  • You've updated the changelog (if adding/changing user-visible functionality)
  • You've updated the user manual (if adding/changing user-visible functionality)

@ikappaki ikappaki changed the title Issue/gh action macos aarch Issue/gh action macos aarch64 May 17, 2024
@ikappaki ikappaki force-pushed the issue/gh-action-macos-aarch branch from 701bbb1 to 2aba539 Compare May 17, 2024 17:57
@vemv
Copy link
Member

vemv commented May 17, 2024

Thanks for giving this a shot!

At the moment it's failing with:

The template is not valid. .github/workflows/test.yml (Line: 38, Col: 18): A sequence was not expected

Moreover, IIRC macos-13 will be retired from GHA.

I'd be fine with removing the combinations that won't possibly work (newer macOS + older emacsen).

However ci/circleci: test-ubuntu-emacs-26 is also failing. I don't know the reason for those ones.

Would be good to run emacs 26 on some or other CI.

@ikappaki ikappaki force-pushed the issue/gh-action-macos-aarch branch from 2aba539 to 78b303d Compare May 17, 2024 18:09
@ikappaki ikappaki force-pushed the issue/gh-action-macos-aarch branch from c1e0dd1 to 3e68914 Compare May 17, 2024 18:19
@ikappaki
Copy link
Contributor Author

ikappaki commented May 17, 2024

Hi @vemv :)

At the moment it's failing with:

The template is not valid. .github/workflows/test.yml (Line: 38, Col: 18): A sequence was not expected

I think it should be fine now. I forgot the matrix/include section does not accept arrays, thus I had to repeat the entries twice, and due to economy of space I've only included java version=11, let me know if you want me to add two more entries for java=17 to get it inline with the rest.

Moreover, IIRC macos-13 will be retired from GHA.

Let me know what you think we should do. I'm fine supporting it as long as it is around.

I'd be fine with removing the combinations that won't possibly work (newer macOS + older emacsen).

👍 , just a note, this is a limitation of nix-emacs-ci, not a hard limit. If we try really hard enough we might be able to get older versions (but can't see the merit of it).

However ci/circleci: test-ubuntu-emacs-26 is also failing. I don't know the reason for those ones.

This is due to this commit 8b2ef5f #3661 , unrelated to this patch.

max-mini-window-lines was introduced in v28 I think, thus the failures on earlier versions.

Would be good to run emacs 26 on some or other CI.

#3207 :)

@vemv vemv merged commit 6e89ad2 into clojure-emacs:master May 17, 2024
35 of 37 checks passed
@vemv
Copy link
Member

vemv commented May 17, 2024

let me know if you want me to add two more entries for java=17 to get it inline with the rest.

Mmm, we should have it as there's some JVM-level testing. Another PR welcome.

👍 , just a note, this is a limitation of nix-emacs-ci, not a hard limit. If we try really hard enough we might be able to get older versions (but can't see the merit of it).

So arm macs can run older emacsen? That would seem worth looking into. No rush atm.

(the merit being to support our very diverse user base)

Would be good to run emacs 26 on some or other CI.

#3207 :)

The incident seems to support the choice of using both GHA and CCI - the redundancy keeps us (more-or-less) isolated from churn.

@vemv
Copy link
Member

vemv commented May 17, 2024

This is due to this commit 8b2ef5f #3661 , unrelated to this patch.

Well spotted, thanks!

@alexander-yakushev , could you please take a look at the small breakage?

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.

Integration tests are broken
2 participants