You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@spark.apache.org by do...@apache.org on 2020/02/02 08:45:52 UTC

[spark] branch branch-3.0 updated: [SPARK-30704][INFRA] Use jekyll-redirect-from 0.15.0 instead of the latest

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

dongjoon pushed a commit to branch branch-3.0
in repository https://gitbox.apache.org/repos/asf/spark.git


The following commit(s) were added to refs/heads/branch-3.0 by this push:
     new 2f1fb4c  [SPARK-30704][INFRA] Use jekyll-redirect-from 0.15.0 instead of the latest
2f1fb4c is described below

commit 2f1fb4c01d0d4bfda17b3262e6f586f4f1a25bac
Author: Dongjoon Hyun <dh...@apple.com>
AuthorDate: Sun Feb 2 00:44:25 2020 -0800

    [SPARK-30704][INFRA] Use jekyll-redirect-from 0.15.0 instead of the latest
    
    ### What changes were proposed in this pull request?
    
    This PR aims to pin the version of `jekyll-redirect-from` to 0.15.0. This is a release blocker for both Apache Spark 3.0.0 and 2.4.5.
    
    ### Why are the changes needed?
    
    `jekyll-redirect-from` released 0.16.0 a few days ago and that requires Ruby 2.4.0.
    - https://github.com/jekyll/jekyll-redirect-from/releases/tag/v0.16.0
    ```
    $ cd dev/create-release/spark-rm/
    $ docker build -t spark:test .
    ...
    ERROR:  Error installing jekyll-redirect-from:
    	jekyll-redirect-from requires Ruby version >= 2.4.0.
    ...
    ```
    
    ### Does this PR introduce any user-facing change?
    
    No.
    
    ### How was this patch tested?
    
    Manually do the above command to build `spark-rm` Docker image.
    ```
    ...
    Successfully installed jekyll-redirect-from-0.15.0
    Parsing documentation for jekyll-redirect-from-0.15.0
    Installing ri documentation for jekyll-redirect-from-0.15.0
    Done installing documentation for jekyll-redirect-from after 0 seconds
    1 gem installed
    Successfully installed rouge-3.15.0
    Parsing documentation for rouge-3.15.0
    Installing ri documentation for rouge-3.15.0
    Done installing documentation for rouge after 4 seconds
    1 gem installed
    Removing intermediate container e0ec7c77b69f
     ---> 32dec37291c6
    ```
    
    Closes #27434 from dongjoon-hyun/SPARK-30704.
    
    Authored-by: Dongjoon Hyun <dh...@apple.com>
    Signed-off-by: Dongjoon Hyun <dh...@apple.com>
    (cherry picked from commit 1adf3520e3c753e6df8dccb752e8239de682a09a)
    Signed-off-by: Dongjoon Hyun <dh...@apple.com>
---
 dev/create-release/spark-rm/Dockerfile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/dev/create-release/spark-rm/Dockerfile b/dev/create-release/spark-rm/Dockerfile
index 3ba8e97..6345168 100644
--- a/dev/create-release/spark-rm/Dockerfile
+++ b/dev/create-release/spark-rm/Dockerfile
@@ -78,7 +78,7 @@ RUN apt-get clean && apt-get update && $APT_INSTALL gnupg ca-certificates && \
   # Install tools needed to build the documentation.
   $APT_INSTALL ruby2.3 ruby2.3-dev mkdocs && \
   gem install jekyll --no-rdoc --no-ri -v 3.8.6 && \
-  gem install jekyll-redirect-from && \
+  gem install jekyll-redirect-from -v 0.15.0 && \
   gem install rouge
 
 WORKDIR /opt/spark-rm/output


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