-
Notifications
You must be signed in to change notification settings - Fork 17
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: add --output
to wat2wasm
, wasm2wat
and opt
cmd
#416
Conversation
Aw OK. Maybe we could have a flag then, something like |
I understand why this PR is needed, but I'm not entirely in favor of it. I believe both However, I must confess that I prefer the I'll leave the final decision to someone with a stronger opinion. |
I agree that accepting a single file as an argument is a good idea for specifying the target path, let |
@vasucp1207, do you want to make the changes to accepting a single file, to use the current directory and to accept the |
Yup, |
Yes sure, so I propose the following changes:
|
Not include the |
wat2wasm
emit files files in same dir--output
to wat2wasm
and wasm2wat
@vasucp1207, I believe we also have to do the same for |
--output
to wat2wasm
and wasm2wat
--output
to wat2wasm
, wasm2wat
and opt
cmd
Sure, but I didn't add |
CHANGES.md
Outdated
@@ -1,5 +1,6 @@ | |||
## unreleased | |||
|
|||
- `wat2wasm` emit wasm files in same dir as wat files |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be updated now :)
We are getting there! Don't forget to run and promote the tests once the last issues have been fixed so that I can merge directly. :) Also, could you add a test for each command (they each have a directory in
|
Thanks for your patience! :) |
Emit wasm files in same dir as wat files