-
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
fix: ensure CLI replaces path separator on Windows platforms correctly #367
Conversation
Due to differences between solc and solcjs in their generated output on Windows machines (ethereum/solc-js#365), compilation will always break because solcjs doesn't handle output paths correctly. A fix for this has been proposed here ethereum/solc-js#367, once that is merged, we can roll back to solcjs again.
Due to differences between solc and solcjs in their generated output on Windows machines (ethereum/solc-js#365), compilation will always break because solcjs doesn't handle output paths correctly. A fix for this has been proposed here ethereum/solc-js#367, once that is merged, we can roll back to solcjs again.
Due to differences between solc and solcjs in their generated output on Windows machines (ethereum/solc-js#365), compilation will always break because solcjs doesn't handle output paths correctly. A fix for this has been proposed here ethereum/solc-js#367, once that is merged, we can roll back to solcjs again.
Thanks! |
Wonderful, thanks for merging this @chriseth ! Do you know when a release can be expected, that includes this fix? |
The releases are synchronized with Solidity's releases. We haven't set a release date yet, but should be in about 2 weeks. |
Hey @chriseth just wanted to check in, any updates on the release? :) |
@PascalPrecht we released on npm last week, it should contain this PR. |
Now that ethereum/solc-js#367 has landed in solcjs and published, we can roll back to using it again, making the CI setup significantly more trivial and faster. Closes #63
Now that ethereum/solc-js#367 has landed in solcjs and published, we can roll back to using it again, making the CI setup significantly more trivial and faster. Closes #63
Ah, thank you @chriseth ! 🙏 I didn't even check haha |
Now that ethereum/solc-js#367 has landed in solcjs and published, we can roll back to using it again, making the CI setup significantly more trivial and faster. Closes #63
Fixes #365