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

Docker build failing on x86_64_dev since moving to the c-kzg from crate-io #5572

Closed
jimmygchen opened this issue Apr 12, 2024 · 4 comments
Closed

Comments

@jimmygchen
Copy link
Member

Description

Our docker x86_64_dev cross build has been failing since moving to the crate-io version of c-kzg (#5542). We should fix this before the next release.

thread 'main' panicked at /cargo/registry/src/index.crates.io-6f17d22bba15001f/blst-0.3.11/build.rs:173:25:
Cannot compile with both \`portable\` and \`force-adx\` features

https://github.com/sigp/lighthouse/actions/runs/8639102912/job/23684787522

@pawanjay176
Copy link
Member

We can remove force-adx since the portable build should detect it on runtime and use it if available.
cc @michaelsproul

@michaelsproul
Copy link
Member

Yeah, agreed. We could go full steam ahead with Anton's change:

I think the alternative would be to compile czkg with no-default-features so that it doesn't turn on blst/portable: https://github.com/ethereum/c-kzg-4844/blob/1ef67c17bf659547749750e6ebaaf09563e0149d/Cargo.toml#L21-L23

@antondlr
Copy link
Member

Yeah, agreed. We could go full steam ahead with Anton's change:

I think the alternative would be to compile czkg with no-default-features so that it doesn't turn on blst/portable: https://github.com/ethereum/c-kzg-4844/blob/1ef67c17bf659547749750e6ebaaf09563e0149d/Cargo.toml#L21-L23

fyi I split out the docker and release/binaries parts into PR's #5614 and #5615
the docker one (#5614) can thus be merged without (too much) worries of breaking the release flow.

@jimmygchen
Copy link
Member Author

Fixed in #5614 🎉

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

4 participants