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

Can't find primitives::SHANGHAI when building #1544

Closed
1 task done
odyslam opened this issue Feb 24, 2023 · 9 comments
Closed
1 task done

Can't find primitives::SHANGHAI when building #1544

odyslam opened this issue Feb 24, 2023 · 9 comments
Labels
C-bug An unexpected or incorrect behavior

Comments

@odyslam
Copy link

odyslam commented Feb 24, 2023

Describe the bug

image

It may belong to the REVM repo. 🤔

Steps to reproduce

  • Use reth as a dependency on a rust project; track main
  • run cargo check

Node logs

No response

Platform(s)

Mac (Apple Silicon)

What version/commit are you on?

No response

Code of Conduct

  • I agree to follow the Code of Conduct
@odyslam odyslam added C-bug An unexpected or incorrect behavior S-needs-triage This issue needs to be labelled labels Feb 24, 2023
@rkrasiuk
Copy link
Member

@odyslam do you have a separate revm dependency?

@odyslam
Copy link
Author

odyslam commented Feb 24, 2023

@rkrasiuk negative. If we are confident it's something on my local project, let's close the issue, and I will debug it.

@rkrasiuk
Copy link
Member

@odyslam no, this is us. we patched revm to master here

reth/Cargo.toml

Lines 45 to 46 in dc57fba

revm = { git = "https://github.com/bluealloy/revm" }
revm-primitives = { git = "https://github.com/bluealloy/revm" }

@onbjerg
Copy link
Member

onbjerg commented Feb 24, 2023

Cargo.lock pins revm to 8e6f4f26173a245d9c3c19fd2317cd8bfd1c56a9 which definitely has the constant available. Perhaps your project has it pinned differently in Cargo.lock? I'm not sure if patched crates propagate to dependents

@odyslam
Copy link
Author

odyslam commented Feb 24, 2023

In my `Cargo.lock:

[[package]]
name = "revm"
version = "3.0.0"
source = "registry+https://github.com/rust-lang/crates.io-index"
checksum = "284747ad0324ed0e805dcf8f412e2beccb7a547fbd84f0607865001b8719c515"
dependencies = [
 "auto_impl 1.0.1",
 "revm-interpreter",
 "revm-precompile",
]

@rkrasiuk
Copy link
Member

rkrasiuk commented Feb 24, 2023

@onbjerg i'm not sure Cargo.lock is respected.
edit: look what i found rust-lang/cargo#8728
edit2: "Just to be clere a libraries Cargo.lock has no effect on resolution for a project that depends on that library. "

@odyslam
Copy link
Author

odyslam commented Feb 24, 2023

So what's the best way forward @rkrasiuk ?

On my side, I could use a rev of reth before this issue was introduced. Might be acceptable depending on when the features I am using where introduced (mainly infra like the CLI and task manager)

@rkrasiuk
Copy link
Member

@odyslam just copy the patch lines over to your Cargo.toml, that should work

@odyslam
Copy link
Author

odyslam commented Feb 24, 2023

This did the trick ✌️ Thanks @rkrasiuk

@odyslam odyslam closed this as completed Feb 24, 2023
@github-project-automation github-project-automation bot moved this from Todo to Done in Reth Tracker Feb 24, 2023
@DaniPopes DaniPopes removed the S-needs-triage This issue needs to be labelled label Apr 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-bug An unexpected or incorrect behavior
Projects
Archived in project
Development

No branches or pull requests

4 participants