Skip to content

Commit

Permalink
[SPARK-41091][BUILD][3.2] Fix Docker release tool for branch-3.2
Browse files Browse the repository at this point in the history
### What changes were proposed in this pull request?

This tries to fix `do-release-docker.sh` for branch-3.2.

### Why are the changes needed?

Currently the following error will occur if running the script in `branch-3.2`:
```
apache#5 917.4 g++ -std=gnu++14 -shared -L/usr/lib/R/lib -Wl,-Bsymbolic-functions -Wl,-z,relro -o testthat.so init.o reassign.o test-catch.o test-example.o test-runner.o -L/usr/lib/R/lib -lR
apache#5 917.5 installing to /usr/local/lib/R/site-library/00LOCK-testthat/00new/testthat/libs
apache#5 917.5 ** R
apache#5 917.5 ** inst
apache#5 917.5 ** byte-compile and prepare package for lazy loading
apache#5 924.4 ** help
apache#5 924.6 *** installing help indices
apache#5 924.7 *** copying figures
apache#5 924.7 ** building package indices
apache#5 924.9 ** installing vignettes
apache#5 924.9 ** testing if installed package can be loaded from temporary location
apache#5 925.1 ** checking absolute paths in shared objects and dynamic libraries
apache#5 925.1 ** testing if installed package can be loaded from final location
apache#5 925.5 ** testing if installed package keeps a record of temporary installation path
apache#5 925.5 * DONE (testthat)
apache#5 925.8 ERROR: dependency 'pkgdown' is not available for package 'devtools'
apache#5 925.8 * removing '/usr/local/lib/R/site-library/devtools'
apache#5 925.8
apache#5 925.8 The downloaded source packages are in
apache#5 925.8        '/tmp/Rtmp3nJI60/downloaded_packages'
apache#5 925.8 Warning messages:
apache#5 925.8 1: In install.packages(c("curl", "xml2", "httr", "devtools", "testthat",  :
apache#5 925.8   installation of package 'textshaping' had non-zero exit status
apache#5 925.8 2: In install.packages(c("curl", "xml2", "httr", "devtools", "testthat",  :
apache#5 925.8   installation of package 'ragg' had non-zero exit status
apache#5 925.8 3: In install.packages(c("curl", "xml2", "httr", "devtools", "testthat",  :
apache#5 925.8   installation of package 'pkgdown' had non-zero exit status
apache#5 925.8 4: In install.packages(c("curl", "xml2", "httr", "devtools", "testthat",  :
apache#5 925.8   installation of package 'devtools' had non-zero exit status
apache#5 926.0 Error in loadNamespace(x) : there is no package called 'devtools'
apache#5 926.0 Calls: loadNamespace -> withRestarts -> withOneRestart -> doWithOneRestart
apache#5 926.0 Execution halted
```

The same error doesn't happen on master. I checked the diff between the two and it seems the following line:
```
$APT_INSTALL libfontconfig1-dev libharfbuzz-dev libfribidi-dev libfreetype6-dev libpng-dev libtiff5-dev libjpeg-dev && \
```

introduced in apache#34728 made the difference.

I verified that after adding the line, `do-release-docker.sh` (dry run mode) was able to finish successfully.

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

Manually

Closes apache#38643 from sunchao/fix-docker-release.

Authored-by: Chao Sun <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>
  • Loading branch information
sunchao authored and dongjoon-hyun committed Nov 15, 2022
1 parent 4a99f7b commit 1d42abb
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions dev/create-release/spark-rm/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -80,6 +80,7 @@ RUN apt-get clean && apt-get update && $APT_INSTALL gnupg ca-certificates && \
$APT_INSTALL r-base r-base-dev && \
$APT_INSTALL libcurl4-openssl-dev libgit2-dev libssl-dev libxml2-dev && \
$APT_INSTALL texlive-latex-base texlive texlive-fonts-extra texinfo qpdf texlive-latex-extra && \
$APT_INSTALL libfontconfig1-dev libharfbuzz-dev libfribidi-dev libfreetype6-dev libpng-dev libtiff5-dev libjpeg-dev && \
Rscript -e "install.packages(c('curl', 'xml2', 'httr', 'devtools', 'testthat', 'knitr', 'rmarkdown', 'markdown', 'roxygen2', 'e1071', 'survival'), repos='https://cloud.r-project.org/')" && \
Rscript -e "devtools::install_github('jimhester/lintr')" && \
# Install tools needed to build the documentation.
Expand Down

0 comments on commit 1d42abb

Please sign in to comment.