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

Build linux arm target on arm machine #67

Merged
merged 3 commits into from
Feb 20, 2025
Merged

Conversation

Schahen
Copy link
Contributor

@Schahen Schahen commented Feb 20, 2025

This addresses https://youtrack.jetbrains.com/issue/CMP-7630 - basically, the minimal Ubuntu should be 20.04 (this is our common agreement so far).

After rebasing to m132 it was not the case for linux arm distribution - because it was just failing (for a bunch of various reasons) on Ubuntu 20.04 we've just switched to 22.04.

Here's how this particular PR addresses this issue: We build on an arm machine however still in docker - Since minimal github configuration for arm is, again, 22.04. Apart from that there are no two different pre-publish steps - for main branch and for PR branches - the only difference was archiving and archiving is cheap (and we see if something is wrong with archiving earlier)

@Schahen Schahen requested review from igordmn and eymar February 20, 2025 16:33
strategy:
matrix:
build_type: [ Debug, Release ]
fail-fast: false
steps:
- uses: actions/checkout@v2
- uses: uraimo/run-on-arch-action@v2.8.1
- uses: addnab/docker-run-action@v3
Copy link
Member

Choose a reason for hiding this comment

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

Is docker still necessary?

Copy link
Member

Choose a reason for hiding this comment

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

I guess it's necessary because we want to get an older Ubuntu version?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@eymar unfortunately yes - because we need to build in Ubuntu 20.04 - and the minimal arm machine in github runner is Ubuntu 22.04

@igordmn igordmn merged commit 4104d7d into main Feb 20, 2025
32 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