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.

Sign up for GitHub

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

Fix typo and change the command line order #6557

Merged
merged 2 commits into from
Nov 10, 2023
Merged

Fix typo and change the command line order #6557

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
9bc192a
Fix typo and change the command line order
mc-nv Nov 10, 2023
fd5fd4d
Improve visual experience. Add 'clang' package
mc-nv Nov 10, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
41 changes: 22 additions & 19 deletions build.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -996,8 +996,8 @@ def create_dockerfile_buildbase(ddir, dockerfile_name, argmap):
# python3-pip and libarchive-dev is needed by python backend
# libxml2-dev is needed for Azure Storage
# scons is needed for armnn_tflite backend build dep
RUN apt-get update && \
apt-get install -y --no-install-recommends \
RUN apt-get update \
&& apt-get install -y --no-install-recommends \
ca-certificates \
autoconf \
automake \
Expand All @@ -1023,9 +1023,9 @@ def create_dockerfile_buildbase(ddir, dockerfile_name, argmap):
zlib1g-dev \
libarchive-dev \
libxml2-dev \
libnuma-dev && \
libnuma-dev \
wget \
rm -rf /var/lib/apt/lists/*
&& rm -rf /var/lib/apt/lists/*
RUN pip3 install --upgrade pip && \
pip3 install --upgrade wheel setuptools docker
Expand Down Expand Up @@ -1243,30 +1243,33 @@ def dockerfile_prepare_container_linux(argmap, backends, enable_gpu, target_mach
# Common dependencies. FIXME (can any of these be conditional? For
# example libcurl only needed for GCS?)
RUN apt-get update && \
apt-get install -y --no-install-recommends \
software-properties-common \
libb64-0d \
libcurl4-openssl-dev \
libre2-9 \
RUN apt-get update \
&& apt-get install -y --no-install-recommends \
clang \
mc-nv marked this conversation as resolved.
Show resolved Hide resolved
curl \
dirmngr \
git \
gperf \
dirmngr \
libb64-0d \
libcurl4-openssl-dev \
libgoogle-perftools-dev \
libnuma-dev \
curl \
libjemalloc-dev \
libnuma-dev \
libre2-9 \
software-properties-common \
wget \
{backend_dependencies} && \
rm -rf /var/lib/apt/lists/*
{backend_dependencies} \
&& rm -rf /var/lib/apt/lists/*
# Install boost version >= 1.78 for boost::span
# Current libboost-dev apt packages are < 1.78, so install from tar.gz
RUN wget -O /tmp/boost.tar.gz \
https://boostorg.jfrog.io/artifactory/main/release/1.80.0/source/boost_1_80_0.tar.gz && \
(cd /tmp && tar xzf boost.tar.gz) && \
cd /tmp/boost_1_80_0 && ./bootstrap.sh --prefix=/usr && ./b2 install && \
rm -rf /tmp/boost*
https://boostorg.jfrog.io/artifactory/main/release/1.80.0/source/boost_1_80_0.tar.gz \
&& (cd /tmp && tar xzf boost.tar.gz) \
&& cd /tmp/boost_1_80_0 \
&& ./bootstrap.sh --prefix=/usr \
&& ./b2 install \
&& rm -rf /tmp/boost*
# Set TCMALLOC_RELEASE_RATE for users setting LD_PRELOAD with tcmalloc
ENV TCMALLOC_RELEASE_RATE 200
Expand Down