You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@spark.apache.org by gu...@apache.org on 2022/07/21 13:29:48 UTC

[spark] branch master updated: [SPARK-39831][BUILD] Fix R dependencies installation failure

This is an automated email from the ASF dual-hosted git repository.

gurwls223 pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git


The following commit(s) were added to refs/heads/master by this push:
     new 67efa318ec8 [SPARK-39831][BUILD] Fix R dependencies installation failure
67efa318ec8 is described below

commit 67efa318ec8cababdb5683ac262a8ebc3b3beefb
Author: Ruifeng Zheng <ru...@apache.org>
AuthorDate: Thu Jul 21 22:29:18 2022 +0900

    [SPARK-39831][BUILD] Fix R dependencies installation failure
    
    ### What changes were proposed in this pull request?
    
    move `libfontconfig1-dev libharfbuzz-dev libfribidi-dev libfreetype6-dev libpng-dev libtiff5-dev libjpeg-dev` from `Install dependencies for documentation generation` to `Install R linter dependencies and SparkR`
    
    ### Why are the changes needed?
    to make CI happy
    
    `Install R linter dependencies and SparkR` started to fail after `devtools_2.4.4` was released.
    
    ```
     --------------------------- [ANTICONF] --------------------------------
    Configuration failed to find the fontconfig freetype2 library. Try installing:
     * deb: libfontconfig1-dev (Debian, Ubuntu, etc)
     * rpm: fontconfig-devel (Fedora, EPEL)
     * csw: fontconfig_dev (Solaris)
     * brew: freetype (OSX)
     ```
    
    it seems that `libfontconfig1-dev` is needed now.
    
    also refer to https://github.com/r-lib/systemfonts/issues/35#issuecomment-633560151
    
    ### Does this PR introduce _any_ user-facing change?
    No
    
    ### How was this patch tested?
    existing tests
    
    Closes #37243 from zhengruifeng/ci_add_dep.
    
    Authored-by: Ruifeng Zheng <ru...@apache.org>
    Signed-off-by: Hyukjin Kwon <gu...@apache.org>
---
 .github/workflows/build_and_test.yml | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/.github/workflows/build_and_test.yml b/.github/workflows/build_and_test.yml
index 133f152df50..7236281d6db 100644
--- a/.github/workflows/build_and_test.yml
+++ b/.github/workflows/build_and_test.yml
@@ -552,7 +552,9 @@ jobs:
         python3.9 -m pip install 'pandas-stubs==1.2.0.53'
     - name: Install R linter dependencies and SparkR
       run: |
-        apt-get install -y libcurl4-openssl-dev libgit2-dev libssl-dev libxml2-dev
+        apt-get install -y libcurl4-openssl-dev libgit2-dev libssl-dev libxml2-dev \
+          libfontconfig1-dev libharfbuzz-dev libfribidi-dev libfreetype6-dev libpng-dev \
+          libtiff5-dev libjpeg-dev
         Rscript -e "install.packages(c('devtools'), repos='https://cloud.r-project.org/')"
         Rscript -e "devtools::install_version('lintr', version='2.0.1', repos='https://cloud.r-project.org')"
         ./R/install-dev.sh
@@ -563,8 +565,7 @@ jobs:
     - name: Install dependencies for documentation generation
       run: |
         # pandoc is required to generate PySpark APIs as well in nbsphinx.
-        apt-get install -y libcurl4-openssl-dev pandoc libfontconfig1-dev libharfbuzz-dev \
-          libfribidi-dev libfreetype6-dev libpng-dev libtiff5-dev libjpeg-dev
+        apt-get install -y libcurl4-openssl-dev pandoc
         # TODO(SPARK-32407): Sphinx 3.1+ does not correctly index nested classes.
         #   See also https://github.com/sphinx-doc/sphinx/issues/7551.
         # Jinja2 3.0.0+ causes error when building with Sphinx.


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@spark.apache.org
For additional commands, e-mail: commits-help@spark.apache.org