zentol commented on a change in pull request #9489: [FLINK-13793][docs] build 
each language in a separate subprocess
URL: https://github.com/apache/flink/pull/9489#discussion_r357385769
 
 

 ##########
 File path: docs/build_docs.sh
 ##########
 @@ -78,4 +80,28 @@ while getopts "piez" opt; do
 done
 
 # use 'bundle exec' to insert the local Ruby dependencies
-bundle exec jekyll ${JEKYLL_CMD} ${JEKYLL_CONFIG} --source "${DOCS_SRC}" 
--destination "${DOCS_DST}"
+
+if [ "${JEKYLL_CMD}" = "build" ] && [ -z "${JEKYLL_CONFIG}" ]; then
 
 Review comment:
   if we modified the e/z cases above to instead set DOC_LANGUAGES to en/zh we 
wouldn't need this condition nor else branch

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to