Skip to content

Add circleci config with mac, windows and linux testing#338

Merged
sbc100 merged 2 commits intomasterfrom
circleci
Aug 29, 2019
Merged

Add circleci config with mac, windows and linux testing#338
sbc100 merged 2 commits intomasterfrom
circleci

Conversation

@sbc100
Copy link
Collaborator

@sbc100 sbc100 commented Aug 28, 2019

Add circleci with support for mac, linux and windows

  • Add EMSDK_NOTTY variable which we can use on CI to prevent tty output.
  • Add test.bat for windows which mirrors test.sh.
  • Remove travis support

@sbc100 sbc100 force-pushed the circleci branch 15 times, most recently from f1d5a21 to 2b6e215 Compare August 28, 2019 23:15
@sbc100 sbc100 changed the title Add circleci config to test windows support Add circleci config with mac, windows and linux testing Aug 29, 2019
- Add EMSDK_NOTTY variable which we can use on CI to prevent tty output.
- Add test.bat for windows which mirrors test.sh.
- Remove travis support
@sbc100
Copy link
Collaborator Author

sbc100 commented Aug 29, 2019

This change should be good to go now!

@sbc100 sbc100 requested a review from kripken August 29, 2019 01:10
Copy link
Member

@kripken kripken left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Excellent! :) Ship it!

check_call('./emsdk update')
run_emsdk('update')

print('verify downloads exist for all OSes')
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can avoid downloading for all 3 OSes in each OS now that we have testing for them all.

But can leave for later.

@sbc100
Copy link
Collaborator Author

sbc100 commented Aug 29, 2019

We need to wait for emscripten-core/emscripten#9347 to land in a release. Or I could disable that test on mac until it does.

@kripken
Copy link
Member

kripken commented Aug 29, 2019

I can tag a release tomorrow morning probably (assuming tree looks good).

@sbc100 sbc100 merged commit 880be59 into master Aug 29, 2019
@sbc100 sbc100 deleted the circleci branch August 29, 2019 07:35
vargaz pushed a commit to vargaz/emsdk that referenced this pull request Nov 22, 2023
…426.4 (emscripten-core#338)

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>
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.

3 participants