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: added windows and osx builds as well as tests to the github pipelines #57

Merged
merged 5 commits into from
Jul 25, 2023

Conversation

mgerhardy
Copy link
Collaborator

@mgerhardy mgerhardy commented Jul 23, 2023

I've added a very basic cmake script to perform the cl.exe search - I'm not very familiar with windows and could find an easy way to find cl.exe.

I've also added a test case in form of the demo_vox application to the pipeline.

The build artifacts are attached to each pipeline run.

see https://github.com/jpaver/opengametools/actions/runs/5637436975

@mgerhardy
Copy link
Collaborator Author

/WX and -Werror are not enabled - as there are a few warnings included that would prevent a successful run.

@mgerhardy mgerhardy force-pushed the ci-win branch 3 times, most recently from fa937f5 to 1a38e43 Compare July 23, 2023 11:28
also store the artifacts of the build for all platforms

that way users can easily download the binaries without having to compile
the binaries on their own
@mgerhardy mgerhardy changed the title CI: added windows build CI: added windows and osx builds Jul 23, 2023
@mgerhardy mgerhardy marked this pull request as ready for review July 23, 2023 11:33
@mgerhardy mgerhardy changed the title CI: added windows and osx builds CI: added windows and osx builds as well as tests to the github pipelines Jul 23, 2023
Copy link
Owner

@jpaver jpaver left a comment

Choose a reason for hiding this comment

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

Looks great, thanks for the contribution!

@jpaver jpaver merged commit c40fe60 into jpaver:master Jul 25, 2023
3 checks passed
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

2 participants