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

Restore testing accidentally removed in #7499 #8382

Merged
merged 1 commit into from
Apr 2, 2019

Conversation

sbc100
Copy link
Collaborator

@sbc100 sbc100 commented Apr 2, 2019

No description provided.

@sbc100 sbc100 requested a review from kripken April 2, 2019 00:26
@sbc100
Copy link
Collaborator Author

sbc100 commented Apr 2, 2019

@kripken any idea whats going on with the test failure test_dylink_zlib? Could that be related to this change somehow?

@sbc100 sbc100 force-pushed the restore_test_dead_functions branch from 39f9c2b to 34fc283 Compare April 2, 2019 19:28
@sbc100 sbc100 merged commit 2a832de into incoming Apr 2, 2019
@sbc100 sbc100 deleted the restore_test_dead_functions branch April 2, 2019 19:59
@kripken
Copy link
Member

kripken commented Apr 2, 2019

I haven't seen dynlink_zlib fail - where do you see that?

@sbc100
Copy link
Collaborator Author

sbc100 commented Apr 2, 2019

It showed up consistently on this PR.. but now it seems fine :-/

VirtualTim pushed a commit to VirtualTim/emscripten that referenced this pull request May 21, 2019
VirtualTim added a commit to VirtualTim/emscripten that referenced this pull request May 23, 2019
VirtualTim added a commit to VirtualTim/emscripten that referenced this pull request May 23, 2019
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