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

Add tests for wasmer-cli create-{exe,obj} commands #3117

Merged
merged 7 commits into from
Aug 22, 2022
Merged

Conversation

epilys
Copy link
Contributor

@epilys epilys commented Aug 18, 2022

Unified generated objects interfaces: both static and serialized objects
expose the same function signature to create the module:

wasm_module_t* wasmer_object_module_new(wasm_store_t* store, const char* module_name);

Closes #3071

@epilys epilys added this to the v3.0 milestone Aug 18, 2022
@epilys epilys self-assigned this Aug 18, 2022
@epilys epilys changed the title create-exe: fix serialized create-exe not working Add tests for wasmer-cli create-{exe,obj} commands Aug 18, 2022
@epilys epilys marked this pull request as ready for review August 18, 2022 11:58
@epilys epilys requested a review from syrusakbary as a code owner August 18, 2022 11:58
@epilys epilys requested review from fschutt and syrusakbary and removed request for syrusakbary August 18, 2022 11:58
@epilys epilys force-pushed the create-exe-tests branch 2 times, most recently from 3764dd8 to 2f2b895 Compare August 18, 2022 17:41
Copy link
Contributor

@fschutt fschutt left a comment

Choose a reason for hiding this comment

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

Added a few comments regarding style + some questions

lib/cli/src/commands/create_exe.rs Show resolved Hide resolved
lib/cli/src/commands/create_exe.rs Outdated Show resolved Hide resolved
lib/cli/src/commands/create_exe.rs Outdated Show resolved Hide resolved
lib/cli/src/commands/create_exe.rs Show resolved Hide resolved
lib/cli/src/commands/create_exe.rs Outdated Show resolved Hide resolved
bors bot added a commit that referenced this pull request Aug 19, 2022
3117: Add tests for wasmer-cli create-{exe,obj} commands r=epilys a=epilys

Unified generated objects interfaces: both static and serialized objects
expose the same function signature to create the module:

  wasm_module_t* wasmer_object_module_new(wasm_store_t* store, const char* module_name);

Closes #3071 

Co-authored-by: Manos Pitsidianakis <[email protected]>
@wasmerio wasmerio deleted a comment from bors bot Aug 20, 2022
@wasmerio wasmerio deleted a comment from syrusakbary Aug 20, 2022
@epilys
Copy link
Contributor Author

epilys commented Aug 20, 2022

@epilys epilys requested review from syrusakbary and fschutt August 20, 2022 19:18
epilys added 7 commits August 21, 2022 16:25
Unified generated objects interfaces: both static and serialized objects
expose the same function signature to create the module:

  wasm_module_t* wasmer_object_module_new(wasm_store_t* store, const char* module_name);
…at flag

Test that serialized and symbol formats work both in create-exe and
create-obj.
@epilys
Copy link
Contributor Author

epilys commented Aug 22, 2022

bors r+

@bors
Copy link
Contributor

bors bot commented Aug 22, 2022

Build succeeded:

@bors bors bot merged commit 8a6c987 into master Aug 22, 2022
@bors bors bot deleted the create-exe-tests branch August 22, 2022 15:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add tests for wasmer-cli create-{exe,obj} commands
3 participants