scikit-learn · cmarmo · Oct 23, 2019 · Oct 24, 2019 · Oct 24, 2019 · Oct 24, 2019
diff --git a/build_tools/circle/build_doc.sh b/build_tools/circle/build_doc.sh
@@ -58,6 +58,44 @@ get_build_type() {
         return
     fi
     changed_examples=$(echo "$filenames" | grep -E "^examples/(.*/)*plot_")
+
+    # The following is used to extract the list of filenames of example python
+    # files that sphinx-gallery needs to run to generate png files used as
+    # figures or images in the .rst files  from the documentation.
+    # If the contributor changes a .rst file in a PR we need to run all
+    # the examples mentioned in that file to get sphinx build the
+    # documentation without generating spurious warnings related to missing
+    # png files.
+
+    if [[ -n "$filenames" ]]
+    then
+        # get rst files
+        rst_files="$(echo "$filenames" | grep -E "rst$")"
+
+        # get lines with figure or images
+        img_fig_lines="$(echo "$rst_files" | xargs grep -shE "(figure|image)::")"
+
+        # get only auto_examples
+        auto_example_files="$(echo "$img_fig_lines" | grep auto_examples | awk -F "/" '{print $NF}')"
+
+        # remove "sphx_glr_" from path and accept replace _\d\d\d.png with .py
+        scripts_names="$(echo "$auto_example_files" | sed 's/sphx_glr_//' | sed -e 's/_[[:digit:]][[:digit:]][[:digit:]].png/.py/')"
+
+        # get unique values
+        examples_in_rst="$(echo "$scripts_names" | uniq )"
+    fi
+
+    # executed only if there are examples in the modified rst files
+    if [[ -n "$examples_in_rst" ]]
+    then
+	if [[ -n "$changed_examples" ]]
+	then
+		changed_examples="$changed_examples|$examples_in_rst"
+	else
+		changed_examples="$examples_in_rst"
+	fi
+    fi
+
     if [[ -n "$changed_examples" ]]
     then
         echo BUILD: detected examples/ filename modified in $git_range: $changed_examples
@@ -66,6 +104,7 @@ get_build_type() {
         echo "$pattern"
         return
     fi
+
     echo QUICK BUILD: no examples/ filename modified in $git_range:
     echo "$filenames"
 }
@@ -190,6 +229,8 @@ then
     if [ -z "$warnings" ]
     then
         warnings="/home/circleci/project/ no warnings"
+    else
+        check=1
     fi
     echo "$warnings"
 
@@ -204,5 +245,12 @@ then
     echo "$warnings" | sed 's/\/home\/circleci\/project\//<li>/g'
     echo '</ul></body></html>'
     ) > 'doc/_build/html/stable/_changed.html'
+
+    if [ $check ]
+    then
+        echo "Sphinx generated warnings when building the documentation related to files modified in this PR."
+        echo "Please check doc/_build/html/stable/_changed.html"
+        exit 1
+    fi
 fi