-
Notifications
You must be signed in to change notification settings - Fork 52
Install dependencies before building on Linux #828
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
base: master
Are you sure you want to change the base?
Conversation
Snake case is ssssssuperior
Yes I am hastily doing this on my phone
So, there is nothing to explain
Okay, apt is doing the heavy lifting with the "check" here
You are adding a lot of dependency, but in Godot's own CI I only see this single install: Is there a reason for the extra stuff ? Hard Disk space is really limited in runner and we already ran into issues with it more than once. We shouldn't bloat it with so many libs if we are not sure to use them. |
Remove excess dependencies
The instructions seem to be very different depending on the distro but these should be good. I think its a good idea just to make sure libalsa and libpulse are installed, so I left those, and the wayland and udev dev files. (Sidenote: we should probably be building with udev=yes) |
|
Alright. The dependency is probably still smart to install, but if we haven't seen any other issues with the build yet, I suppose it's safe to simply just install wayland-devel. |
Fixes #827