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

lcli container build fails on stable branch #4174

Closed
dboreham opened this issue Apr 8, 2023 · 1 comment
Closed

lcli container build fails on stable branch #4174

dboreham opened this issue Apr 8, 2023 · 1 comment

Comments

@dboreham
Copy link

dboreham commented Apr 8, 2023

Found while trying to build the sigp/lcli container from the currrent stable branch that it throws this compile time error:

   Compiling lighthouse_network v0.2.0 (/lighthouse/beacon_node/lighthouse_network)
   Compiling environment v0.1.2 (/lighthouse/lighthouse/environment)
error[E0658]: use of unstable library feature 'map_first_last'
  --> beacon_node/lighthouse_network/src/peer_manager/network_behaviour.rs:92:64
   |
92 |         if let Some((peer_id, maybe_enr)) = self.peers_to_dial.pop_first() {
   |                                                                ^^^^^^^^^
   |
   = note: see issue #62924 <https://github.com/rust-lang/rust/issues/62924> for more information

   Compiling operation_pool v0.2.0 (/lighthouse/beacon_node/operation_pool)
For more information about this error, try `rustc --explain E0658`.
error: could not compile `lighthouse_network` due to previous error

Also noticed that this container specifies a different version of Rust vs the version used for the sigp/lighthouse container.

And the build is fixed by using Rust 1.66:

diff --git a/lcli/Dockerfile b/lcli/Dockerfile
index feda81d03..079e5c779 100644
--- a/lcli/Dockerfile
+++ b/lcli/Dockerfile
@@ -1,7 +1,7 @@
 # `lcli` requires the full project to be in scope, so this should be built either:
 #  - from the `lighthouse` dir with the command: `docker build -f ./lcli/Dockerflie .`
 #  - from the current directory with the command: `docker build -f ./Dockerfile ../`
-FROM rust:1.65.0-bullseye AS builder
+FROM rust:1.66.0-bullseye AS builder
 RUN apt-get update && apt-get -y upgrade && apt-get install -y cmake libclang-dev protobuf-compiler
 COPY . lighthouse
 ARG PORTABLE
@michaelsproul
Copy link
Member

Thanks for the report

We've fixed this on unstable and it will be fixed on stable when the next release is published. The lcli build from unstable should be useable for everything that the stable build would be (no breaking changes, etc)

If you don't mind, I'll close this issue and the PR 🙏

@dboreham dboreham closed this as completed Apr 8, 2023
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 a pull request may close this issue.

2 participants