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

Fix CI workflow deprecation errors #353

Merged
merged 4 commits into from
Feb 1, 2021
Merged

Conversation

ianpurvis
Copy link
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
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 mymindstorm/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
Collaborator Author

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

@kripken
Copy link
Owner

kripken commented Feb 1, 2021

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

@ianpurvis
Copy link
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