Skip to content

Commit 60da3da

Browse files
committed
chore: Aligning bazel version for consistency across different dockerfiles and documentations
Signed-off-by: Anurag Dixit <[email protected]>
1 parent da7569e commit 60da3da

File tree

5 files changed

+8
-6
lines changed

5 files changed

+8
-6
lines changed

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -111,7 +111,7 @@ torch.jit.save(trt_ts_module, "trt_torchscript_module.ts") # save the TRT embedd
111111

112112
These are the following dependencies used to verify the testcases. Torch-TensorRT can work with other versions, but the tests are not guaranteed to pass.
113113

114-
- Bazel 5.1.1
114+
- Bazel 5.2.0
115115
- Libtorch 1.12.0 (built with CUDA 11.3)
116116
- CUDA 11.3
117117
- cuDNN 8.4.1

docker/Dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ RUN rm -rf /opt/pytorch/torch_tensorrt /usr/bin/bazel
99

1010
ARG ARCH="x86_64"
1111
ARG TARGETARCH="amd64"
12-
ARG BAZEL_VERSION=5.1.1
12+
ARG BAZEL_VERSION=5.2.0
1313

1414
RUN [[ "$TARGETARCH" == "amd64" ]] && ARCH="x86_64" || ARCH="${TARGETARCH}" \
1515
&& wget -q https://github.com/bazelbuild/bazel/releases/download/${BAZEL_VERSION}/bazel-${BAZEL_VERSION}-linux-${ARCH} -O /usr/bin/bazel \

docker/Dockerfile.docs

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,12 @@
11
FROM nvcr.io/nvidia/tensorrt:22.06-py3
22

3+
ARG BAZEL_VERSION=5.2.0
4+
35
RUN curl https://bazel.build/bazel-release.pub.gpg | apt-key add -
46
RUN echo "deb [arch=amd64] https://storage.googleapis.com/bazel-apt stable jdk1.8" | tee /etc/apt/sources.list.d/bazel.list
57

6-
RUN apt-get update && apt-get install -y bazel-5.2.0 clang-format-9 libjpeg9 libjpeg9-dev
7-
RUN ln -s /usr/bin/bazel-5.2.0 /usr/bin/bazel
8+
RUN apt-get update && apt-get install -y bazel-${BAZEL_VERSION} clang-format-9 libjpeg9 libjpeg9-dev
9+
RUN ln -s /usr/bin/bazel-${BAZEL_VERSION} /usr/bin/bazel
810

911
# Workaround for bazel expecting both static and shared versions, we only use shared libraries inside container
1012
RUN cp /usr/lib/x86_64-linux-gnu/libnvinfer.so /usr/lib/x86_64-linux-gnu/libnvinfer_static.a

docker/Dockerfile.ngc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ RUN rm -rf /opt/pytorch/torch_tensorrt /usr/bin/bazel
99

1010
ARG ARCH="x86_64"
1111
ARG TARGETARCH="amd64"
12-
ARG BAZEL_VERSION=5.1.1
12+
ARG BAZEL_VERSION=5.2.0
1313

1414
RUN [[ "$TARGETARCH" == "amd64" ]] && ARCH="x86_64" || ARCH="${TARGETARCH}" \
1515
&& wget -q https://github.com/bazelbuild/bazel/releases/download/${BAZEL_VERSION}/bazel-${BAZEL_VERSION}-linux-${ARCH} -O /usr/bin/bazel \

docker/setup_nox.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ set -e
77
post=${1:-""}
88

99
# fetch bazel executable
10-
BAZEL_VERSION=5.1.1
10+
BAZEL_VERSION=5.2.0
1111
ARCH=$(uname -m)
1212
if [[ "$ARCH" == "aarch64" ]]; then ARCH="arm64"; fi
1313
wget -q https://github.com/bazelbuild/bazel/releases/download/${BAZEL_VERSION}/bazel-${BAZEL_VERSION}-linux-${ARCH} -O /usr/bin/bazel

0 commit comments

Comments
 (0)