Git Product home page Git Product logo

Comments (9)

yrashk avatar yrashk commented on June 22, 2024

Bridge relies on these contracts to generate calls, transactions and event filters, so I don't think it's reasonable to drop this dependency at this moment. What will happen after we're done transitioning integration tests is that we can replace contracts in this project with interfaces from actual contracts to keep things leaner inside. But we'll still need solc.

from poa-bridge.

rstormsf avatar rstormsf commented on June 22, 2024

maybe you only need ABIs ?
those are just JSON files.
Just get latest ABIs or I can provide it to you and that should be it.
I don't think you need solc in order to get it. solc is only used to produce bytecode and ABIs

from poa-bridge.

rstormsf avatar rstormsf commented on June 22, 2024

https://gist.github.com/rstormsf/998ae8acbe467788bdf95778fbc68cf0

3 files

from poa-bridge.

akolotov avatar akolotov commented on June 22, 2024

Agree with Roman. I think you could use git submodules to link to repos: poa-bridge and poa-parity-bridge-contracts.

from poa-bridge.

yrashk avatar yrashk commented on June 22, 2024

Yes, ABI will do if we want to get rid of solc dependency. They will just make it a tad harder to synchronize between sub-projects (instead of copying files, we'd need to use solc to compile them to .abi every time we sync). If we're willing to compromise here, then sure. Ideally, interfaces should not change too often.

@akolotov poa-parity-bridge-contracts doesn't seem to contain ABIs as is.

from poa-bridge.

rstormsf avatar rstormsf commented on June 22, 2024

Correct, those are artifacts, which I already provided in the gist above

from poa-bridge.

akolotov avatar akolotov commented on June 22, 2024

OK. Let's take ABI's from poa-parity-bridge-contracts and will update them in poa-bridge as soon as interfaces are being changed. Since interfaces changes will require changes on Rust side anyway it seems to be logical to follow this procedure.

from poa-bridge.

yrashk avatar yrashk commented on June 22, 2024

As per @rstormsf it might be better to store those in https://github.com/poanetwork/poa-chain-spec

from poa-bridge.

akolotov avatar akolotov commented on June 22, 2024

Created omni/tokenbridge-contracts#38 to upload ABIs to specification repo

from poa-bridge.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.