Skip to content

Commit

Permalink
Toolchain: Remove dependency on locales
Browse files Browse the repository at this point in the history
  • Loading branch information
oschuett committed Oct 19, 2020
1 parent 67d412b commit 4e271ea
Show file tree
Hide file tree
Showing 8 changed files with 4 additions and 20 deletions.
4 changes: 0 additions & 4 deletions tools/docker/Dockerfile.test_python
Original file line number Diff line number Diff line change
Expand Up @@ -7,10 +7,6 @@ WORKDIR /workspace
COPY ./scripts/install_basics.sh .
RUN ./install_basics.sh

# set a proper unicode-enabled locale, see https://hub.docker.com/_/ubuntu?tab=description
RUN localedef -i en_US -c -f UTF-8 -A /usr/share/locale/locale.alias en_US.UTF-8
ENV LANG en_US.UTF-8

COPY ./scripts/install_python.sh .
RUN ./install_python.sh

Expand Down
2 changes: 0 additions & 2 deletions tools/docker/scripts/install_basics.sh
Original file line number Diff line number Diff line change
Expand Up @@ -7,15 +7,13 @@ if grep -q "Ubuntu" /etc/os-release ; then
apt-get update -qq
apt-get install -qq --no-install-recommends \
ca-certificates \
locales \
git \
less \
nano \
python3 \
rsync \
wget
rm -rf /var/lib/apt/lists/*
locale-gen "en_US.UTF-8"
echo "done."

elif grep -q "Fedora" /etc/os-release ; then
Expand Down
4 changes: 3 additions & 1 deletion tools/regtesting/do_regtest
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,9 @@
# command line argument passing
#
###################################################################################
export LC_ALL=en_US.UTF-8

# ensure error messages are in English
export LC_ALL=C

# init
dir_base=$PWD
Expand Down
2 changes: 0 additions & 2 deletions tools/toolchain/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,6 @@ USER root
COPY ./install_requirements*.sh ./
RUN ./install_requirements.sh ${BASE_IMAGE}

ENV LANG en_US.UTF-8

# copy helper scripts
WORKDIR /opt/cp2k-toolchain
RUN mkdir scripts
Expand Down
2 changes: 0 additions & 2 deletions tools/toolchain/Dockerfile.cuda_mkl
Original file line number Diff line number Diff line change
Expand Up @@ -11,8 +11,6 @@ ARG LIBINT_LMAX=5
COPY ./install_requirements_ubuntu.sh .
RUN ./install_requirements_ubuntu.sh

ENV LANG en_US.UTF-8

# Install some more Ubuntu packages.
RUN apt-get update -qq && apt-get install -qq --no-install-recommends \
gfortran \
Expand Down
5 changes: 0 additions & 5 deletions tools/toolchain/Dockerfile.ubuntu_nompi
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,6 @@ RUN export DEBIAN_FRONTEND=noninteractive DEBCONF_NONINTERACTIVE_SEEN=true && \
git \
less \
libtool \
locales \
make \
nano \
pkg-config \
Expand All @@ -40,10 +39,6 @@ RUN export DEBIAN_FRONTEND=noninteractive DEBCONF_NONINTERACTIVE_SEEN=true && \
libhdf5-dev \
&& rm -rf /var/lib/apt/lists/*

# generate a unicode-enabled locale, see https://hub.docker.com/_/ubuntu?tab=description
RUN localedef -i C -c -f UTF-8 -A /usr/share/locale/locale.alias C.UTF-8
ENV LANG en_US.UTF-8

# Create links.
RUN ln -sf gcc-${GCC_VERSION} /usr/bin/gcc && \
ln -sf g++-${GCC_VERSION} /usr/bin/g++ && \
Expand Down
1 change: 0 additions & 1 deletion tools/toolchain/install_requirements_fedora.sh
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,6 @@ dnf -qy install \
diffutils \
g++ \
git \
glibc-langpack-en \
less \
libtool \
make \
Expand Down
4 changes: 1 addition & 3 deletions tools/toolchain/install_requirements_ubuntu.sh
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,6 @@ apt-get install -qq --no-install-recommends \
git \
less \
libtool \
locales \
make \
nano \
patch \
Expand All @@ -36,5 +35,4 @@ apt-get install -qq --no-install-recommends \

rm -rf /var/lib/apt/lists/*

# generate a unicode-enabled locale, see https://hub.docker.com/_/ubuntu?tab=description
localedef -i C -c -f UTF-8 -A /usr/share/locale/locale.alias C.UTF-8
#EOF

0 comments on commit 4e271ea

Please sign in to comment.