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

fix compile error #80

Open
wants to merge 6 commits into
base: feat/v1.13.15
Choose a base branch
from
Open

fix compile error #80

wants to merge 6 commits into from

Conversation

tak1827
Copy link
Contributor

@tak1827 tak1827 commented Oct 17, 2024

Mergeしたやつのコンパイルエラーを解消したバージョンです。
プラスでETHプロトコルのv67をサポートしました。

@tak1827 tak1827 requested a review from ironbeer October 17, 2024 05:29
@@ -383,6 +383,23 @@ func handleReceipts(backend Backend, msg Decoder, peer *Peer) error {
}, metadata)
}

func handleNewPooledTransactionHashes67(backend Backend, msg Decoder, peer *Peer) error {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

eth v67をサポートするもろもろの変更

github.com/minio/sha256-simd v1.0.0 // indirect
>>>>>>> c5ba367eb6232e3eddd7d6226bfd374449c63164
github.com/mitchellh/mapstructure v1.4.1 // indirect
github.com/mitchellh/mapstructure v1.5.0 // indirect
Copy link
Contributor Author

@tak1827 tak1827 Oct 17, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

後述のgo-eth2-types/[email protected]が使ってる。BSCのマスターも使っていた。

@@ -265,7 +239,7 @@ require (
github.com/trailofbits/go-mutexasserts v0.0.0-20230328101604-8cdbc5f3d279 // indirect
github.com/uber/jaeger-client-go v2.25.0+incompatible // indirect
github.com/wealdtech/go-bytesutil v1.1.1 // indirect
github.com/wealdtech/go-eth2-types/v2 v2.5.2 // indirect
github.com/wealdtech/go-eth2-types/v2 v2.7.0 // indirect
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

BSCがフォークしたgithub.com/bnb-chain/fastsszの使用を避けるために、上げざるを得なかった。

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

Successfully merging this pull request may close these issues.

1 participant