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 Linux editor without Mono, sanitizers with Mono #45727

Merged
merged 1 commit into from
Feb 5, 2021

Conversation

akien-mga
Copy link
Member

The Mono builds are with mono_glue=no so they're not usable,
and it would be convenient if the main tools=yes target=release_debug
artifacts could actually be used.

I tested and confirmed that the linux-editor artifact now works well on a recent Linux distro. It's built on Ubuntu 20.04 so it requires glibc >= 2.31, it won't run on older distros.

The Mono builds are with mono_glue=no so they're not usable,
and it would be convenient if the main tools=yes target=release_debug
artifacts could actually be used.
@akien-mga akien-mga added enhancement topic:buildsystem cherrypick:3.x Considered for cherry-picking into a future 3.x release labels Feb 5, 2021
@akien-mga akien-mga added this to the 4.0 milestone Feb 5, 2021
@akien-mga akien-mga requested a review from a team February 5, 2021 12:41
@aaronfranke
Copy link
Member

At some point we could add a version that generates the glue and builds with glue enabled, but this would require the headless DisplayServer because Godot currently has to open a window to generate the glue.

@akien-mga akien-mga merged commit 80a4d0f into godotengine:master Feb 5, 2021
@akien-mga akien-mga deleted the ci-linux-swap-mono branch February 5, 2021 13:20
@akien-mga
Copy link
Member Author

Instead of cherry-picking this for 3.2 I made the Mono build actually functional, since we can use xvfb-run to generate the glue there with OpenGL: #45732.

@akien-mga akien-mga removed the cherrypick:3.x Considered for cherry-picking into a future 3.x release label Feb 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants