This is an automated email from the ASF dual-hosted git repository. jensg pushed a commit to branch 0.23.0 in repository https://gitbox.apache.org/repos/asf/thrift.git
commit 8e3dc179a3d3f6f8390979a5a7772dab737c07f0 Author: Jens Geyer <[email protected]> AuthorDate: Wed Apr 8 23:49:32 2026 +0200 Fix ubuntu-jammy Docker build: update NodeSource and ENV format - Replace deprecated apt-key NodeSource setup (node_16.x/focal) with modern gpg keyring method using node_20.x/nodistro, fixing GPG signature verification failure - Convert all legacy ENV key value syntax to ENV key=value format to eliminate 15 Dockerfile linting warnings Client: build Co-Authored-By: Claude Sonnet 4.6 <[email protected]> Generated-by: Claude Sonnet 4.6 --- build/docker/ubuntu-jammy/Dockerfile | 35 ++++++++++++++++++----------------- 1 file changed, 18 insertions(+), 17 deletions(-) diff --git a/build/docker/ubuntu-jammy/Dockerfile b/build/docker/ubuntu-jammy/Dockerfile index a2331ab69..3fc3da130 100644 --- a/build/docker/ubuntu-jammy/Dockerfile +++ b/build/docker/ubuntu-jammy/Dockerfile @@ -52,8 +52,9 @@ RUN curl https://dl-ssl.google.com/linux/linux_signing_key.pub | apt-key add - & /etc/apt/sources.list.d/dart_stable.list # node.js -RUN curl -sL https://deb.nodesource.com/gpgkey/nodesource.gpg.key | apt-key add - && \ - echo "deb https://deb.nodesource.com/node_16.x focal main" | tee /etc/apt/sources.list.d/nodesource.list +RUN mkdir -p /etc/apt/keyrings && \ + curl -fsSL https://deb.nodesource.com/gpgkey/nodesource-repo.gpg.key | gpg --dearmor -o /etc/apt/keyrings/nodesource.gpg && \ + echo "deb [signed-by=/etc/apt/keyrings/nodesource.gpg] https://deb.nodesource.com/node_20.x nodistro main" | tee /etc/apt/sources.list.d/nodesource.list ### install general dependencies RUN apt-get update -yq && \ @@ -76,14 +77,14 @@ RUN apt-get update -yq && \ unzip \ valgrind \ vim -ENV PATH /usr/lib/llvm-6.0/bin:$PATH +ENV PATH=/usr/lib/llvm-6.0/bin:$PATH # lib/as3 (ActionScript) RUN mkdir -p /usr/local/adobe/flex/4.6 && \ cd /usr/local/adobe/flex/4.6 && \ wget -q "http://download.macromedia.com/pub/flex/sdk/flex_sdk_4.6.zip"; && \ unzip flex_sdk_4.6.zip -ENV FLEX_HOME /usr/local/adobe/flex/4.6 +ENV FLEX_HOME=/usr/local/adobe/flex/4.6 # TODO: "apt-get install" without "apt-get update" in the same "RUN" step can cause cache issues if modified later. # See https://docs.docker.com/develop/develop-images/dockerfile_best-practices/#run @@ -95,7 +96,7 @@ RUN apt-get install -y --no-install-recommends \ qtbase5-dev \ qtbase5-dev-tools -ENV SBCL_VERSION 1.5.3 +ENV SBCL_VERSION=1.5.3 RUN \ `# Common Lisp (sbcl) dependencies` \ curl --version && \ @@ -107,8 +108,8 @@ RUN \ cd .. && \ rm -rf sbcl* -ENV D_VERSION 2.087.0 -ENV DMD_DEB dmd_2.087.0-0_amd64.deb +ENV D_VERSION=2.087.0 +ENV DMD_DEB=dmd_2.087.0-0_amd64.deb RUN \ `# D dependencies` \ wget -q http://downloads.dlang.org/releases/2.x/${D_VERSION}/${DMD_DEB} && \ @@ -124,11 +125,11 @@ RUN \ mv deimos-openssl-1.1.0h/C/* /usr/include/dmd/druntime/import/C/ && \ rm -rf deimos-openssl-1.1.0h -ENV DART_VERSION 2.7.2-1 +ENV DART_VERSION=2.7.2-1 RUN apt-get install -y --no-install-recommends \ `# Dart dependencies` \ dart=$DART_VERSION -ENV PATH /usr/lib/dart/bin:$PATH +ENV PATH=/usr/lib/dart/bin:$PATH RUN add-apt-repository ppa:dotnet/backports RUN apt-get install -y --no-install-recommends \ @@ -146,16 +147,16 @@ RUN apt-get update && apt-get install -y --no-install-recommends libncurses5-dev kerl build $ERLANG_OTP_VERSION && kerl install $ERLANG_OTP_VERSION /usr/local/lib/otp/ && . /usr/local/lib/otp/activate && \ curl -ssLo /usr/local/bin/rebar3 https://github.com/erlang/rebar3/releases/download/${ERLANG_REBAR_VERSION}/rebar3 && chmod +x /usr/local/bin/rebar3 && \ rebar3 --version -ENV PATH /usr/local/lib/otp/bin:$PATH +ENV PATH=/usr/local/lib/otp/bin:$PATH RUN apt-get install -y --no-install-recommends \ `# GlibC dependencies` \ libglib2.0-dev # golang -ENV GOLANG_VERSION 1.24.3 -ENV GOLANG_DOWNLOAD_URL https://go.dev/dl/go$GOLANG_VERSION.linux-amd64.tar.gz -ENV GOLANG_DOWNLOAD_SHA256 3333f6ea53afa971e9078895eaa4ac7204a8c6b5c68c10e6bc9a33e8e391bdd8 +ENV GOLANG_VERSION=1.24.3 +ENV GOLANG_DOWNLOAD_URL=https://go.dev/dl/go$GOLANG_VERSION.linux-amd64.tar.gz +ENV GOLANG_DOWNLOAD_SHA256=3333f6ea53afa971e9078895eaa4ac7204a8c6b5c68c10e6bc9a33e8e391bdd8 RUN curl -fsSL "$GOLANG_DOWNLOAD_URL" -o golang.tar.gz && \ echo "$GOLANG_DOWNLOAD_SHA256 golang.tar.gz" | sha256sum -c - && \ tar -C /usr/local -xzf golang.tar.gz && \ @@ -175,7 +176,7 @@ RUN cd $HOME && \ tar xvf neko-${NEKO_VERSION}-linux64.tar.gz && \ rm neko-${NEKO_VERSION}-linux64.tar.gz && \ mv neko-* /opt/neko -ENV PATH /opt/haxe:/opt/neko:$PATH +ENV PATH=/opt/haxe:/opt/neko:$PATH RUN echo "/opt/neko" > /etc/ld.so.conf.d/neko.conf && \ ldconfig USER ${user} @@ -267,7 +268,7 @@ RUN apt-get install -y --no-install-recommends \ USER ${user} RUN `# Rust dependencies` \ curl https://sh.rustup.rs -sSf | sh -s -- --default-toolchain 1.83.0 -y -ENV PATH /home/${user}/.cargo/bin:$PATH +ENV PATH=/home/${user}/.cargo/bin:$PATH USER root # Swift on Linux for cross tests @@ -282,7 +283,7 @@ RUN apt-get install -yq \ mv swift-5.7-RELEASE-ubuntu22.04 /usr/share/swift && \ rm swift-5.7-RELEASE-ubuntu22.04.tar.gz -ENV PATH /usr/share/swift/usr/bin:$PATH +ENV PATH=/usr/share/swift/usr/bin:$PATH RUN swift --version # Locale(s) for cpp unit tests @@ -306,7 +307,7 @@ RUN apt-get install -y --no-install-recommends \ # rm -rf /tmp/* && \ # rm -rf /var/tmp/* -ENV THRIFT_ROOT /thrift +ENV THRIFT_ROOT=/thrift RUN mkdir -p $THRIFT_ROOT/src && \ chown -R ${uid}:${uid} $THRIFT_ROOT/ COPY Dockerfile $THRIFT_ROOT/
