Skip to content

Commit

Permalink
Install locales and generate UTF-8
Browse files Browse the repository at this point in the history
Don't set LANG to UTF-8, leave it to the test script

Signed-off-by: gashen <gashen@nvidia.com>
  • Loading branch information
GaryShen2008 committed May 26, 2022
1 parent 96eb861 commit 6c1ce9b
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 2 deletions.
4 changes: 4 additions & 0 deletions jenkins/Dockerfile-blossom.integration.ubuntu
Original file line number Diff line number Diff line change
Expand Up @@ -60,3 +60,7 @@ RUN export CUDA_VER=`echo ${CUDA_VER} | cut -d '.' -f 1,2` && \
conda clean -ay

RUN apt install -y inetutils-ping expect

# install locale and generate UTF-8, but don't set it as default, the test cases should specify it somehow
RUN apt-get install -y locales
RUN locale-gen en_US.UTF-8
3 changes: 1 addition & 2 deletions jenkins/Dockerfile-blossom.ubuntu
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,6 @@ RUN mkdir -p /tmp/ucx && \
# export JAVA_HOME for mvn option 'source-javadoc'
ENV JAVA_HOME /usr/lib/jvm/java-8-openjdk-amd64

# install locale and use UTF-8
# install locale and generate UTF-8, but don't set it as default, the test cases should specify it somehow
RUN apt-get install -y locales
RUN locale-gen en_US.UTF-8
ENV LANG en_US.UTF-8

0 comments on commit 6c1ce9b

Please sign in to comment.