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 CI via GitHub Actions #82

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Add CI via GitHub Actions #82

wants to merge 3 commits into from

Conversation

Croydon
Copy link

@Croydon Croydon commented Jan 29, 2022

This adds some basic CI builds via GitHub Actions.

CMake

This produces a build matrix [Linux, Windows, MacOS] x [Debug, Release] and builds libmodplug with CMake with the default detected compiler in the respective GitHub Actions environment (currently: GCC 9, Visual Studio 2019 and Apple Clang 13).

Autotools

This produces a build matrix [Linux] x [Debug, Release] and build libmodplug with Autotools with the default detected compiler in the Ubuntu GitHub Actions environment (currently: GCC 9)


An exemplary run can be seen here: https://github.com/Croydon/libmodplug/actions/runs/1772652809

Later on, this could be extended to cover more compilers, compiler versions and build systems, but it is a start to get some CI going.

If this is merged, GitHub will run this CI for every future push and pull request, no further configuration is required whatsoever.

@sezero
Copy link
Contributor

sezero commented Jan 29, 2022

AFAICS, cmake support here is still not fully mature (e.g. non-identical library output names for mingw and mac compared to autotools, maybe other issues as well), so, if it were me I'd use autotools for this.

@Croydon
Copy link
Author

Croydon commented Jan 29, 2022

autotools could be added in a separate pull request as well, but I don't think that this is a reason against adding CMake coverage

@Croydon
Copy link
Author

Croydon commented Jan 31, 2022

I have added Autotools builds

.github/workflows/ci.yml Outdated Show resolved Hide resolved
.github/workflows/ci.yml Outdated Show resolved Hide resolved
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