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: Pin Emscripten version for reproducible builds #40915

Merged
merged 1 commit into from
Jul 31, 2020

Conversation

akien-mga
Copy link
Member

@akien-mga akien-mga commented Jul 31, 2020

Emscripten is a fast-moving target which gets tons of improvements all the time,
but it's not rare that some regressions affect us and make our CI builds fail.
(See e.g. #33728, #35237, #39168, #40563, and #40914.)

Let's pin to a stable version to avoid having external factors impact our CI,
and update this version manually regularly in a PR to ensure that the new
version works well for us.

Emscripten is a fast-moving target which gets tons of improvements all the time,
but it's not rare that some regressions affect us and make our CI builds fail.
(See e.g. godotengine#33728, godotengine#35237, godotengine#39168, godotengine#40563, and godotengine#40914.)

Let's pin to a stable version to avoid having external factors impact our CI,
and update this version manually regularly in a PR to ensure that the new
version works well for us.
@akien-mga akien-mga force-pushed the ci-pin-emscripten branch from e495784 to bfd5226 Compare July 31, 2020 12:19
@akien-mga akien-mga merged commit 6d4d834 into godotengine:master Jul 31, 2020
@akien-mga akien-mga deleted the ci-pin-emscripten branch July 31, 2020 12:34
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.

1 participant