Skip to content
This repository has been archived by the owner on Feb 26, 2024. It is now read-only.

compile-solidity package depends on an outdated solc-js version #4134

Closed
cameel opened this issue Jun 23, 2021 · 5 comments
Closed

compile-solidity package depends on an outdated solc-js version #4134

cameel opened this issue Jun 23, 2021 · 5 comments

Comments

@cameel
Copy link

cameel commented Jun 23, 2021

Issue

compile-solidity depends on solc-js 0.6.0. This is an old version that had a bug that is the cause of ethereum/solidity#11489 / #4039. The bug has been fixed a year ago (ethereum/solc-js#470) in 0.6.9 so an update to at least that version is needed. Preferably Truffle should always use the latest one.

Note that Truffle used to depend on the latest solc-js from 0.6.x series but looks like it was changed to strictly 0.6.0 in #3975.

@fainashalts
Copy link
Contributor

Hi @cameel thanks for bringing this up! We have deliberately kept this version of solc-js to maintain our support for Node 10 (our tests failed when we tried to make the upgrade in the past). When we drop support for Node 10 in the near future, we will revisit this.

@cameel
Copy link
Author

cameel commented Jun 23, 2021

Maybe it would be a good idea to at least bump it back up to 0.6.9 to make it work with 0.8.2? Truffle used to use that (or higher) version due to ^0.6.0 up to 5.3.2.

By the way, what was the exact issue with node 10? We did have some yet unresolved issues with that version (ethereum/solidity#9297) so I'm curious what that might be about.

@haltman-at
Copy link
Contributor

I don't remember the exact issue, so I'm making a branch where we bump it back up to 0.6.9 and we can see!

@haltman-at
Copy link
Contributor

OK, I put up #4136 and... it's actually passing just fine? Huh. I guess maybe there was no problem after all.

@haltman-at
Copy link
Contributor

OK, closing this for now, now that the immediate problem has been addressed. We likely should update further, though. Feel free to let us know if you want us to reopen this, or just file a new issue.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants