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

[Bug] Upgrade c-kzg dependency to 2.0 #1745

Open
tarassh opened this issue Dec 3, 2024 · 3 comments
Open

[Bug] Upgrade c-kzg dependency to 2.0 #1745

tarassh opened this issue Dec 3, 2024 · 3 comments
Labels
bug Something isn't working

Comments

@tarassh
Copy link

tarassh commented Dec 3, 2024

Component

consensus, eips, genesis

What version of Alloy are you on?

No response

Operating System

None

Describe the bug

I can't use latest c-kzg library with alloy library.

error: failed to select a version for `c-kzg`.
    ... required by package `alloy-consensus v0.7.0`
    ... which satisfies dependency `alloy-consensus = "^0.7"` of package `alloy-network v0.7.0`
    ... which satisfies dependency `alloy-network = "^0.7"` of package `alloy-contract v0.7.0`
    ... which satisfies dependency `alloy-contract = "^0.7"` of package `eventlogger v0.1.0 (/Users/redacted/eventlogger)`
versions that meet the requirements `^1.0` are: 1.0.3, 1.0.2, 1.0.0

the package `c-kzg` links to the native library `ckzg`, but it conflicts with a previous package which links to `ckzg` as well:
package `c-kzg v2.0.0`
    ... which satisfies dependency `c-kzg = "^2.0.0"` (locked to 2.0.0) of package `apilsevice v0.1.0 (/Users/redacted/apilsevice)`
Only one package in the dependency graph may specify the same links value. This helps ensure that only one copy of a native library is linked in the final binary. Try to adjust your dependencies so that only one package uses the `links = "ckzg"` value. For more information, see https://doc.rust-lang.org/cargo/reference/resolver.html#links.

failed to select a version for `c-kzg` which could resolve this conflict
@tarassh tarassh added the bug Something isn't working label Dec 3, 2024
@mattsse
Copy link
Member

mattsse commented Dec 3, 2024

I suggest downgrading, there are no functional changes in 2.0

we will migrate to 2.0 but needs more coordination with revm

@tarassh
Copy link
Author

tarassh commented Dec 5, 2024

EIP-7594 was not present in v1.

@mattsse
Copy link
Member

mattsse commented Dec 5, 2024

this needs to go into revm first https://github.com/bluealloy/revm before we can also bump it here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants