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

[github-action] build jobs failure due to deprecated macOS-12 environment #307

Merged
merged 19 commits into from
Dec 5, 2024

Conversation

ZhangLe2016
Copy link
Contributor

@ZhangLe2016 ZhangLe2016 commented Dec 4, 2024

The macOS-12 environment is deprecated, consider switching to macOS-13, macOS-14 (macos-latest) or macOS-15.

For more details, see actions/runner-images#10721

In addition, removed the -s option because the -s option is not supported by the realpath command in macOS 13 (Ventura) and later versions, and change the swig@4 to swig due to no available formula with the name "swig@4" in macOS13.

…nment

The macOS-12 environment is deprecated, consider switching to macOS-13, macOS-14 (macos-latest) or macOS-15.

For more details, see actions/runner-images#10721
@ZhangLe2016 ZhangLe2016 marked this pull request as ready for review December 4, 2024 07:40
…nment

The macOS-12 environment is deprecated, consider switching to macOS-13, macOS-14 (macos-latest) or macOS-15.

For more details, see actions/runner-images#10721

In addition, removing -s option because the -s option is not supported by the realpath command in macOS 13 (Ventura) and later versions
…nment

The macOS-12 environment is deprecated, consider switching to macOS-13, macOS-14 (macos-latest) or macOS-15.

For more details, see actions/runner-images#10721

In addition, removing -s option because the -s option is not supported by the realpath command in macOS 13 (Ventura) and later versions
tests/integration/common.sh Outdated Show resolved Hide resolved
…nment

[github-acction] build jobs failure due to deprecated macOS-12 environment

The macOS-12 environment is deprecated, consider switching to macOS-13, macOS-14 (macos-latest) or macOS-15.

For more details, see actions/runner-images#10721

In addition, removing -s option because the -s option is not supported by the realpath command in macOS 13 (Ventura) and later versions
…nment

[github-acction] build jobs failure due to deprecated macOS-12 environment

The macOS-12 environment is deprecated, consider switching to macOS-13, macOS-14 (macos-latest) or macOS-15.

For more details, see actions/runner-images#10721

In addition, removing -s option because the -s option is not supported by the realpath command in macOS 13 (Ventura) and later versions
Here is the failure log

Run cd android
realpath: illegal option -- s
usage: realpath [-q] [path ...]
CMake Deprecation Warning at /Users/runner/Library/Android/sdk/ndk/26.3.11579264/build/cmake/android.toolchain.cmake:34 (cmake_minimum_required):
  Compatibility with CMake < 3.10 will be removed from a future version of
  CMake.
@ZhangLe2016 ZhangLe2016 marked this pull request as draft December 4, 2024 12:18
…nment

The macOS-12 environment is deprecated, consider switching to macOS-13, macOS-14 (macos-latest) or macOS-15.

For more details, see actions/runner-images#10721

In addition, removing -s option because the -s option is not supported
by the realpath command in macOS 13 (Ventura) and later versions, so execute realpath differently depending on Linux and MacOS.
@ZhangLe2016 ZhangLe2016 marked this pull request as ready for review December 4, 2024 16:04
…nment

The macOS-12 environment is deprecated, consider switching to macOS-13, macOS-14 (macos-latest) or macOS-15.

For more details, see actions/runner-images#10721

In addition, removing -s option because the -s option is not supported
by the realpath command in macOS 13 (Ventura) and later versions, so execute realpath differently depending on Linux and MacOS.
script/bootstrap.sh Show resolved Hide resolved
@ZhangLe2016 ZhangLe2016 requested a review from jwhui December 5, 2024 03:18
@ZhangLe2016 ZhangLe2016 changed the title [github-acction] build jobs failure due to deprecated macOS-12 environment [github-action] build jobs failure due to deprecated macOS-12 environment Dec 5, 2024
@@ -37,7 +37,7 @@ jobs:
matrix:
android-api: [26, 34]
android-abi: [x86_64]
os: [macos-12, ubuntu-22.04]
os: [macos-13, ubuntu-22.04]
Copy link
Member

Choose a reason for hiding this comment

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

Did you try macos-14?

Copy link
Contributor Author

@ZhangLe2016 ZhangLe2016 Dec 5, 2024

Choose a reason for hiding this comment

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

macOS-14 may introduce a new issue: 'Timeout waiting for emulator to boot.' Currently, there is no known solution to address this issue.

https://github.com/openthread/ot-commissioner/actions/runs/12174524147?pr=308
https://github.com/openthread/ot-commissioner/actions/runs/12174524147/job/33959234711?pr=308#step:6:77

Copy link
Contributor Author

@ZhangLe2016 ZhangLe2016 Dec 5, 2024

Choose a reason for hiding this comment

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

@jwhui The current reason why we cannot use macOS 14 is that the CPU architecture of macOS 14 has changed, making it incompatible with Android API 26 or 27 ABI x86_64. Upgrading the Android NDK version to 28 or above is required. I tried to upgrade the Android NDK version using apt install/brew install android-sdk, but it was unsuccessful in the current GitHub environment. Therefore, there is no effective way to directly use macOS 14 in the short term. We will temporarily use the macOS 13 environment and upgrade to macOS 14 as the GitHub environment changes. What do you think? Thanks~~

image
https://stackoverflow.com/questions/74760054/panic-avds-cpu-architecture-arm64-is-not-supported-by-the-qemu2-emulator-on

image

Copy link
Member

Choose a reason for hiding this comment

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

Thanks for checking. I'm fine with deferring change to macos-14 to later.

@jwhui jwhui merged commit 53b4774 into openthread:main Dec 5, 2024
33 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.

3 participants