Skip to content

Fix CI workflow deprecation errors#353

Merged
ianpurvis merged 4 commits into
masterfrom
fix-workflow-deprecations
Feb 1, 2021
Merged

Fix CI workflow deprecation errors#353
ianpurvis merged 4 commits into
masterfrom
fix-workflow-deprecations

Conversation

@ianpurvis
Copy link
Copy Markdown
Collaborator

@ianpurvis ianpurvis commented Jan 27, 2021

Fixes build errors from deprecated add-path and set-env surfacing in #352

More info at https://github.blog/changelog/2020-10-01-github-actions-deprecating-set-env-and-add-path-commands/

  Fixes errors from deprecated `add-path` and `set-env`
  Try running cmake before setup-emsdk to verify that setup-emsdk is not
  corrupting the environment on windows 2019 runners
@ianpurvis
Copy link
Copy Markdown
Collaborator Author

I found that setup-emsdk can corrupt PATH if emsdk construct_env emits a path containing spaces (Program Files). I've submitted a PR to fix this at emscripten-core/setup-emsdk#11

@ianpurvis ianpurvis changed the title Upgrades ci workflow to setup-emsdk@v7 Fix CI workflow deprecation errors Feb 1, 2021
@ianpurvis
Copy link
Copy Markdown
Collaborator Author

@kripken Quick heads-up, I'm going to merge this fix for CI.

@kripken
Copy link
Copy Markdown
Owner

kripken commented Feb 1, 2021

Thanks @ianpurvis , sorry for the delay in getting to this.

@ianpurvis
Copy link
Copy Markdown
Collaborator Author

@kripken All good- thanks!! 👍

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.

2 participants