Skip to content

ci: move Linux release builds to GitHub Actions #2302

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

Merged
merged 1 commit into from
Nov 24, 2021

Conversation

aykevl
Copy link
Member

@aykevl aykevl commented Nov 21, 2021

Work in progress. Making a PR because it won't run otherwise.

@aykevl aykevl force-pushed the ci-github-actions-linux branch 5 times, most recently from 0c0f521 to 38ffc2d Compare November 22, 2021 01:04
@aykevl aykevl force-pushed the ci-github-actions-linux branch from 38ffc2d to e4e2773 Compare November 22, 2021 01:35
@aykevl aykevl marked this pull request as ready for review November 22, 2021 01:35
@aykevl
Copy link
Member Author

aykevl commented Nov 22, 2021

This is now ready.

Note that TinyHCI will likely need to be updated to pull the release artifact from GitHub Actions instead of from CircleCI.

The motivation for this PR is to avoid the 1 hour build limit on CircleCI, which we can't stay within when we're building LLVM. Right now we usually stay within that limit, but the cache will eventually expire at which point we have to do something. Also: LLVM 12 support (#2300) needs to rebuild LLVM.

@deadprogram
Copy link
Member

I have made a number of changes to TinyHCI to accommodate the new workflow. It was a bit more complicated to obtain the build artifacts, and then also process them, which is why this took longer than expected.

Since the TinyHCI tests are now passing, I am going to merge this PR now. Thanks for working on it @aykevl

@deadprogram deadprogram merged commit 18f4ffd into dev Nov 24, 2021
@deadprogram deadprogram deleted the ci-github-actions-linux branch November 24, 2021 12:00
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.

2 participants