fix docs-build.sh bug in rst checks 15/5015/1
authorRyota MIBU <r-mibu@cq.jp.nec.com>
Tue, 22 Dec 2015 05:27:34 +0000 (14:27 +0900)
committerRyota MIBU <r-mibu@cq.jp.nec.com>
Tue, 22 Dec 2015 05:27:34 +0000 (14:27 +0900)
After [1] merged, docs-build.sh cannot check and edit rst files to
add revision into source files. Build error can be found in [2].

This patch fixes this bug.

[1] https://gerrit.opnfv.org/gerrit/#/c/4659
[2] https://build.opnfv.org/ci/job/opnfv-docs-verify-master/211/console

Change-Id: I506d85341d665a1e2c854ab5965ef3abffa31446
Signed-off-by: Ryota MIBU <r-mibu@cq.jp.nec.com>
utils/docs-build.sh

index 99c5aea..af31d7a 100755 (executable)
@@ -61,15 +61,18 @@ function check_rst_doc() {
 function add_html_notes() {
     _src="$1"
 
-    if grep -e ' _sha1_' "$_src"/*.rst ; then
-        # TODO: remove this, once old templates were removed from all repos.
-        echo
-        echo "Warn: '_sha1_' was found, use the latest document template."
-        echo "      See https://wiki.opnfv.org/documentation/tools ."
-        echo
-        sed -i "s/ _sha1_/ $git_sha1/g" "$_src"/*.rst
-    fi
-    sed -i -e "\$a\\\n.. only:: html\n$html_notes" "$_src"/*.rst
+    find "$_src" -name '*.rst' | while read file
+    do
+        if grep -q -e ' _sha1_' "$file" ; then
+            # TODO: remove this, once old templates were removed from all repos.
+            echo
+            echo "Warn: '_sha1_' was found in [$file], use the latest document template."
+            echo "      See https://wiki.opnfv.org/documentation/tools ."
+            echo
+            sed -i "s/ _sha1_/ $git_sha1/g" "$file"
+        fi
+        sed -i -e "\$a\\\n.. only:: html\n$html_notes" "$file"
+    done
 }
 
 function prepare_src_files() {