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

Add Github CI for Haiku r1beta4 using build_matrix.sh #1151

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

korli
Copy link
Contributor

@korli korli commented Mar 19, 2024

clang 17 doesn't work as expected on x86.

@infrastation
Copy link
Member

Thank you for preparing these changes. Do you know if there is a feasible way to implement this using Buildbot and a standalone VM?

@korli
Copy link
Contributor Author

korli commented Mar 19, 2024

Sorry, I don't know much about buildbot.

@infrastation
Copy link
Member

For reference, on my old PC with 4 cores and ccache tcpdump build matrix in a Haiku VM took almost 70 minutes. Given a budget of 2000 minutes per month, that would be 28 builds per month.

Let me have a look and think about it.

@korli
Copy link
Contributor Author

korli commented Mar 20, 2024

Github CI does it in 26 minutes: https://github.com/korli/tcpdump/actions/runs/8349002774 What's the problem with Github actions?

@fxlb
Copy link
Member

fxlb commented Mar 21, 2024

Please don't use 'git merge', always use 'git rebase' (See CONTRIBUTING.md)

clang 17 doesn't work as expected on x86.
@infrastation
Copy link
Member

infrastation commented Sep 24, 2024

I suspected there is a way to extend an existing CI facility for Haiku instead of introducing an additional CI facility just for Haiku (because there are 4 of them already). One such way would be using a Haiku image in Cirrus CI. Here is the document and the script that prove such an image exists, although there seems to be a bug.

@kallisti5
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants