Skip to content

Conversation

Alexey-Ayupov
Copy link
Contributor

Description

Add Java to the macOS 13 arm64

Related issue:

https://github.com/actions/runner-images-internal/issues/5447

Check list

  • Related issue / work item is attached
  • Tests are written (if applicable)
  • Documentation is updated (if applicable)
  • Changes are tested and related VM images are successfully generated

@Alexey-Ayupov
Copy link
Contributor Author

/azp run macos11,macos12

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@Alexey-Ayupov
Copy link
Contributor Author

/azp run macos11,macos12

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@Alexey-Ayupov Alexey-Ayupov force-pushed the macos-add-openjdk-arm64 branch 2 times, most recently from 519f64b to 0ba0715 Compare September 28, 2023 08:30
@Alexey-Ayupov
Copy link
Contributor Author

/azp run macos11,macos12

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@Alexey-Ayupov
Copy link
Contributor Author

/azp run macos11,macos12

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@Alexey-Ayupov Alexey-Ayupov force-pushed the macos-add-openjdk-arm64 branch from 0ba0715 to 4bf28f3 Compare September 29, 2023 13:39
Comment on lines +74 to +80
It "Gradle is installed to /usr/local/bin" -Skip:($os.IsVenturaArm64) {
(Get-Command "gradle").Path | Should -BeExactly "/usr/local/bin/gradle"
}

It "Gradle is installed to /opt/homebrew/bin/gradle" -Skip:($os.IsVentura) {
(Get-Command "gradle").Path | Should -BeExactly "/opt/homebrew/bin/gradle"
}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Won't these two test cases interfere in case of Mac OS 11 or 12?

@mikhailkoliada mikhailkoliada merged commit 60963fb into actions:main Oct 5, 2023
@Alexey-Ayupov Alexey-Ayupov deleted the macos-add-openjdk-arm64 branch October 13, 2023 08:04
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.

3 participants