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

Adds GitHub Workflow to check if project compiles under Ubuntu and Windows #13

Draft
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

Ceyron
Copy link
Collaborator

@Ceyron Ceyron commented Nov 4, 2024

Add the suggested CMAKE GitHub workflow to compile the project. This can be a good sanity check that the project is correctly configured for various OS/compiler combinations.

I made the following changes:

  1. Commented out the part of running the compiled project. (I guess this will not work since no display is attached to a GH Action worker.)
  2. Needed to install some dependencies on Ubuntu

Room for improvement:

  • Add a configuration to the matrix that tests on MacOS?

@Ceyron Ceyron changed the title Adds GitHub Workflows to check if project compiles under Ubuntu and Windows Adds GitHub Workflow to check if project compiles under Ubuntu and Windows Nov 4, 2024
@wi-re
Copy link
Collaborator

wi-re commented Nov 5, 2024

based on some testing these are all packages that you need to install on a fresh Ubuntu install with wayland to get the project working:

mesa-common-dev
g++
cmake
git
libxkbcommon-dev
wayland-protocols
pkg-config
libwayland-dev
extra-cmake-modules
mesa-utils

Assuming you also install vscode afterwards. Is there a way to test if the project compiles with both wayland and x11 under linux?

@Ceyron
Copy link
Collaborator Author

Ceyron commented Nov 5, 2024

Good point; I suppose one could add another combination to the test matrix.

Will have to test it.

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.

3 participants