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

Upgrade to Solidity 5.x #1264

Open
MicahZoltu opened this issue Mar 19, 2019 · 3 comments
Open

Upgrade to Solidity 5.x #1264

MicahZoltu opened this issue Mar 19, 2019 · 3 comments

Comments

@MicahZoltu
Copy link

I believe that Solidity 5.x is now considered released/stable. It would be very nice if this project upgraded so users can leverage new Solidity features (some of which are significant).

Alternatively, perhaps this project could be restructured so that users can define their own Solidity version as part of their build script? I haven't thought through how this would work exactly, but presumably as part of the build process it would fetch the binary releases of the version specified as part of the build process, rather than requiring that they be pre-bundled into solcj and then pulled in as a traditional dependency.

@mkalinin
Copy link
Contributor

There is a SolcJ project which supports 0.5.6 solidity version since this PR was merged ether-camp/solcJ#20. EthereumJ uses its old version, updating it is the first thing worth to do in order to start supporting solidity 0.5.x. It requires tests fixes as this update gonna bring some incompatible API changes.

There is another inconvenience: SolcJ is just a Java bridge for solc compiler while compiler API and ABI stuff sits in EthereumJ codebase. There is an issue that requests a segregation of solidity stuff from EthereumJ ether-camp/solcJ#14.

Alternatively, perhaps this project could be restructured so that users can define their own Solidity version as part of their build script?

This feature could, also, become a part of SolcJ. From my perspective it makes sense to work on the segregation straightaway and when it's done use SolcJ as a fully feature solc provider in EthereumJ and other projects that requires rich Solidity support.

@tbocek
Copy link
Contributor

tbocek commented Mar 19, 2019

I'm using EthereumJ and SolcJ 0.5.6 in this project: https://github.com/ICOnator/testonator. They work fine together.

@derekzuk
Copy link
Contributor

This PR was intended to fix all but one of the unit tests after solidity is updated to 0.5.x. There's one test I was not able to crack that might point to a more fundamental problem:
#1275

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

4 participants