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

CI: Build test on FreeBSD via Cirrus-CI #876

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

Conversation

emaste
Copy link

@emaste emaste commented Feb 4, 2024

Cirrus-CI is a hosted CI service that supports Linux, FreeBSD, Windows, and macOS. Add a Cirrus-CI config to start build testing on FreeBSD.

Example run for the commit in this pull request: https://cirrus-ci.com/build/5520302621327360

Note that building on FreeBSD needs a workaround for the cmake libavif version numbering, issue #867. That workaround plus this Cirrus-CI config file is green, see https://cirrus-ci.com/build/5063026513018880.

Cirrus-CI would need to be installed from the github marketplace for the libgd organization. Cirrus provides a limited amount of free credit for open source projects; it appears that the change rate in libgd is such that it would not exceed Cirrus's free offering.

Copy link
Member

@vapier vapier left a comment

Choose a reason for hiding this comment

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

i've enabled cirrus-ci for libgd org now

.cirrus.yml Show resolved Hide resolved
.cirrus.yml Outdated
graphics/libimagequant print/freetype2 x11/libXpm x11-fonts/fontconfig
print/libraqm
script: |
mkdir build
Copy link
Member

Choose a reason for hiding this comment

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

please be consistent with indentation. we use 2 spaces per-level in GH files, so might as well keep that here.

.cirrus.yml Show resolved Hide resolved
.cirrus.yml Outdated
-DENABLE_PNG=1 -DENABLE_FREETYPE=1 -DENABLE_JPEG=1 -DENABLE_WEBP=1 \
-DENABLE_TIFF=1 -DENABLE_XPM=1 -DENABLE_GD_FORMATS=1 -DENABLE_HEIF=1 \
-DENABLE_RAQM=1 -DENABLE_AVIF=1 -DBUILD_TEST=1
ninja
Copy link
Member

Choose a reason for hiding this comment

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

do we need to specify a -j level ?

graphics/png graphics/webp graphics/tiff graphics/libheif graphics/libavif
graphics/libimagequant print/freetype2 x11/libXpm x11-fonts/fontconfig
print/libraqm
script: |
Copy link
Member

Choose a reason for hiding this comment

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

this compiles the project but doesn't run any tests ...

Cirrus-CI is a hosted CI service that supports Linux, FreeBSD, Windows,
and macOS.  Add a Cirrus-CI config to start build testing on FreeBSD.
@emaste
Copy link
Author

emaste commented Feb 20, 2024

Thanks for the review, updated and pushed. https://cirrus-ci.com/task/6718548333035520 is the run spawned by this pull request, but fails due to #867 right now.

this compiles the project but doesn't run any tests ...

Yes - I hope to start by ensuring the build doesn't regress, and will look at incorporating the test suite.

@vapier
Copy link
Member

vapier commented Mar 12, 2024

Yes - I hope to start by ensuring the build doesn't regress, and will look at incorporating the test suite.

please leave a comment/TODO in the file explaining as much

@neheb
Copy link
Contributor

neheb commented Apr 24, 2024

@emaste why not vmactions?

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.

None yet

3 participants