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 make debug-release #284

Merged
merged 3 commits into from
Mar 20, 2019
Merged

fix make debug-release #284

merged 3 commits into from
Mar 20, 2019

Conversation

MarkMcCaskey
Copy link
Contributor

@MarkMcCaskey MarkMcCaskey commented Mar 20, 2019

My guess without looking in to the history is that:

  • the unsafe block was around the debug! call
  • when debug was off, it was giving a warning because the unsafe block was empty
  • the unsafe was removed to stop the warning
  • it wasn't tested with the debug flag

This PR also

  • cleans up the Makefile
  • adds debug-release to CI

@MarkMcCaskey MarkMcCaskey added the 📦 lib-emscripten About wasmer-emscripten label Mar 20, 2019
@syrusakbary
Copy link
Member

bors try

bors bot added a commit that referenced this pull request Mar 20, 2019
@MarkMcCaskey
Copy link
Contributor Author

bors try

@bors
Copy link
Contributor

bors bot commented Mar 20, 2019

try

Already running a review

@bors
Copy link
Contributor

bors bot commented Mar 20, 2019

@MarkMcCaskey
Copy link
Contributor Author

bors try

bors bot added a commit that referenced this pull request Mar 20, 2019
@bors
Copy link
Contributor

bors bot commented Mar 20, 2019

try

Build failed

@MarkMcCaskey
Copy link
Contributor Author

bors try

bors bot added a commit that referenced this pull request Mar 20, 2019
@bors
Copy link
Contributor

bors bot commented Mar 20, 2019

@MarkMcCaskey
Copy link
Contributor Author

bors r+

bors bot added a commit that referenced this pull request Mar 20, 2019
284: fix make debug-release r=MarkMcCaskey a=MarkMcCaskey

My guess without looking in to the history is that:
- the unsafe block was around the `debug!` call
- when debug was off, it was giving a warning because the unsafe block was empty
- the unsafe was removed to stop the warning
- it wasn't tested with the debug flag

This PR also
- cleans up the Makefile
- adds debug-release to CI

Co-authored-by: Mark McCaskey <mark@wasmer.io>
@bors
Copy link
Contributor

bors bot commented Mar 20, 2019

@bors bors bot merged commit 95fc436 into master Mar 20, 2019
@bors bors bot deleted the feature/fix-make-debug-release branch March 20, 2019 18:45
surban pushed a commit to rust-wasi-web/wwrr that referenced this pull request Nov 9, 2024
surban pushed a commit to rust-wasi-web/wwrr that referenced this pull request Nov 9, 2024
Return the raw instance when instantiating. Fix wasmerio#284
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📦 lib-emscripten About wasmer-emscripten
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants