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 FreeBSD CI using Cirrus CI. #520

Merged
merged 1 commit into from
Oct 30, 2023
Merged

Add FreeBSD CI using Cirrus CI. #520

merged 1 commit into from
Oct 30, 2023

Conversation

arrowd
Copy link
Contributor

@arrowd arrowd commented Sep 1, 2023

Testing is omitted for the moment because AsMonitor tests fail due to FreeBSD-specific bug in glib: https://gitlab.gnome.org/GNOME/glib/-/issues/576

I might also miss some dependencies, so the commit might need amending after the repository enables Cirrus CI.

Obsoletes #485 PR

@arrowd
Copy link
Contributor Author

arrowd commented Sep 12, 2023

Bump, can we get this in?

@ximion
Copy link
Owner

ximion commented Oct 17, 2023

To make this a bit more maintainable, would it be possible to make Cirrus call tests/ci/run-build.sh and use different build options of *BSD? Or does FreeBSD not support /etc/os-release, so adding support for it would be more tedious in that file? (it would be nice to configure all CI builds in one place...)

@arrowd
Copy link
Contributor Author

arrowd commented Oct 18, 2023

Sure, it seems we can reuse that.

.cirrus-ci.yml Outdated Show resolved Hide resolved
Sponsored by:	Serenity Cybersecurity, LLC
@arrowd
Copy link
Contributor Author

arrowd commented Oct 30, 2023

I see you changed this line 2 weeks ago. I guess, no it shouldn't be needed now.

@ximion
Copy link
Owner

ximion commented Oct 30, 2023

Good! I guess this will not actually build before I merge it, so let's merge it now and hopefully it will be fine (I will probably need you to address any non-obvious issues, as I am less familiar with FreeBSD).

Thank you for the patch and sorry for this taking so long!

@ximion ximion merged commit 4b6f785 into ximion:main Oct 30, 2023
9 checks passed
@ximion
Copy link
Owner

ximion commented Oct 30, 2023

Looks like that build environment has no C compiler. Can you please fix that?

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