Skip to content
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

Rename Module to avoid path conflict with Ethereum go-ethereum #158

Open
FeurJak opened this issue Sep 1, 2022 · 12 comments
Open

Rename Module to avoid path conflict with Ethereum go-ethereum #158

FeurJak opened this issue Sep 1, 2022 · 12 comments

Comments

@FeurJak
Copy link

FeurJak commented Sep 1, 2022

Could the module for OffchainLabs go-ethereum be renamed (i.e. Optimism names their geth module l2geth) ?

It's currently under module github.com/ethereum/go-ethereum which means there will be path conflicts if you were trying to import both OffchainLabs go-ethereum & Ethereums go-ethereum.

@AnonJon
Copy link

AnonJon commented Sep 1, 2022

We are running into this issue as well after the nitro upgrade and needing to run the forked geth client along side the geth client. Any work around?

@hkalodner
Copy link
Contributor

hkalodner commented Sep 8, 2022

@FeurJak @jongregis can this be solved by just naming the import ex. import arblog "github.com/ethereum/go-ethereum/log"?

@AnonJon
Copy link

AnonJon commented Sep 8, 2022

@hkalodner it fails to download the module with the current geth module downloaded
go: downloading github.com/OffchainLabs/go-ethereum v0.0.0-20220901205353-ee7893f42e8e go: github.com/OffchainLabs/[email protected]: parsing go.mod: module declares its path as: github.com/ethereum/go-ethereum but was required as: github.com/OffchainLabs/go-ethereum

@hkalodner
Copy link
Contributor

Good callout, we're looking into the best option here

@huahuayu
Copy link

@hkalodner it fails to download the module with the current geth module downloaded go: downloading github.com/OffchainLabs/go-ethereum v0.0.0-20220901205353-ee7893f42e8e go: github.com/OffchainLabs/[email protected]: parsing go.mod: module declares its path as: github.com/ethereum/go-ethereum but was required as: github.com/OffchainLabs/go-ethereum

same problem, it's 29-Nov, 2022 now...

@huahuayu
Copy link

huahuayu commented Nov 29, 2022

@hkalodner please just rename the go module name to github.com/OffchainLabs/go-ethereum what preventing you to do this?

@roodeag
Copy link

roodeag commented Dec 1, 2022

@hkalodner Usually, developers of ethereum use "github.com/ethereum/go-ethereum" to develop ethereum-related projects, but since arbitrum is based on go-ethereum and has many new features, developers cannot use "github.com/ethereum/go-ethereum" directly. Since arbitrum is based on go-ethereum, developers cannot use the library "github.com/ethereum/go-ethereum" directly for normal development, such as tracking blocks. In view of this, I sincerely hope that the arbitrum team will pay attention to this problem and fix it, the solution is to change the name of go-module.

Until arbitrum fixes this issue, you can use this https://github.com/roodeag/arbitrum that I modified

@grallc
Copy link

grallc commented Jun 18, 2023

10 months later, issue still occurring.
Quite annoying as I'd like to build a project that handles both Arbitrum and Mainnet transactions but it's just not doable, as go-ethereum returns transaction type not supported with Arbitrum transactions, but I can't install Arbitrum's client.

@byte-bandit
Copy link

We also really need support for this. @hkalodner What is your plan for this issue going forward?

@anhthii
Copy link

anhthii commented Dec 17, 2023

Hello, any updates on this? We face a situation when we need to run both go-ethereum and Arbitrum go-ethereum in the same application to avoid transaction type not supported issue. Thank you!

@eprokhor
Copy link

+1 Bumping

@aemet93
Copy link

aemet93 commented Jul 25, 2024

Same. Any plans to fix this?

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

No branches or pull requests

10 participants