-
Notifications
You must be signed in to change notification settings - Fork 474
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
solcjs --standard-json - TypeError: Second argument needs to be a buffer #126
Comments
Thanks, this seems to be a bug. |
Will push a fix soon. |
Hi, |
@munawarb the empty object is not enough, some of the fields are required. More information: https://solidity.readthedocs.io/en/develop/using-the-compiler.html#compiler-input-and-output-json-description |
The |
I am running into the exact issue, is there a workaround or fix? |
having the same issue. is there any way to use the solcjs standard json interface at this point?
|
Not that I know of. I just passed the source code as stdin. An example
using Java:
ProcessBuilder processBuilder = new ProcessBuilder("solc",
"--combined-json", "abi,bin", "--optimize");
processBuilder.redirectErrorStream(true);
Process process = processBuilder.start();
OutputStream stdin = process.getOutputStream();
stdin.write(smartContractCode.getBytes());
stdin.flush();
stdin.close();
|
@axic , what's your disposition on this issue and the proposed fix? I'm using the changes from @tzapu 's PR #229 , which seem to include your own changes as well, and it's all working for my purposes. I'd love to see the fix merged into master so that I can go into production with my own development that's depending on it, and I see @snd and others are waiting on it as well. Are there any reservations you have? Is there anything we can do to help facilitate? |
Here's the last comment on the issue: #127 (comment) |
Hello,
solcjs --standard-json myinput.json
doesn't work. See the error bellow.The text was updated successfully, but these errors were encountered: