Fix BLAS portability issues with Ubuntu 20.04 R builds

rstudio · Aug 19, 2024 · bc81ddc · bc81ddc
1 parent 4789b35
commit bc81ddc
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/builder/Dockerfile.ubuntu-2004 b/builder/Dockerfile.ubuntu-2004
@@ -6,7 +6,7 @@ RUN set -x \
   && sed -i "s|# deb-src|deb-src|g" /etc/apt/sources.list \
   && export DEBIAN_FRONTEND=noninteractive \
   && apt-get update \
-  && apt-get install -y curl libopenblas-dev libcurl4-openssl-dev libicu-dev liblapack-dev libpcre2-dev wget python3-pip \
+  && apt-get install -y curl libopenblas0-pthread libcurl4-openssl-dev libicu-dev liblapack-dev libpcre2-dev wget python3-pip \
   && apt-get build-dep -y r-base
 
 RUN pip3 install awscli

diff --git a/builder/Dockerfile.ubuntu-2404 b/builder/Dockerfile.ubuntu-2404
@@ -6,6 +6,13 @@ RUN set -x \
   && sed -i "s|Types: deb|Types: deb deb-src|g" /etc/apt/sources.list.d/ubuntu.sources \
   && export DEBIAN_FRONTEND=noninteractive \
   && apt update \
+  # For BLAS/LAPACK, select the library-only OpenBLAS package that gets included in the
+  # the libopenblas-dev package by default. Usually this is OpenBLAS with pthreads.
+  #
+  # Note that libopenblas-dev must NOT be installed to ensure that R links to BLAS in a
+  # portable way, via the generic libblas.so provided by the libblas-dev package.
+  # This gets installed through the r-base build dependencies. If both libblas-dev and
+  # libopenblas-dev are present, R will prefer linking to OpenBLAS.
   && apt install -y curl libcurl4-openssl-dev libicu-dev libopenblas0-pthread libpcre2-dev libpcre3-dev unzip wget \
   && apt build-dep -y r-base