From: Ryota MIBU <r-mibu@cq.jp.nec.com>
Date: Mon, 25 Jul 2016 01:59:18 +0000 (+0900)
Subject: fix pre-hook
X-Git-Tag: colorado.1.0~20
X-Git-Url: https://gerrit.opnfv.org/gerrit/gitweb?a=commitdiff_plain;h=7d6d582ffc69597ef82e7eed7c34e27f71b4889e;p=opnfvdocs.git

fix pre-hook

- execute pre-hook after required python package installed

- remove execution flag of docs/pre-hook.sh as it's unnecessary

- add howto document regarding 'Hooks'

Change-Id: Ib94888b018347ce8660d53db93dba3586c098948
Signed-off-by: Ryota MIBU <r-mibu@cq.jp.nec.com>
---

diff --git a/docs/how-to-use-docs/documentation-example.rst b/docs/how-to-use-docs/documentation-example.rst
index 85133ad83..bbd710961 100644
--- a/docs/how-to-use-docs/documentation-example.rst
+++ b/docs/how-to-use-docs/documentation-example.rst
@@ -255,3 +255,11 @@ document build.
 
 .. _opnfvdocs/scripts/docs-build.sh:
     https://gerrit.opnfv.org/gerrit/gitweb?p=opnfvdocs.git;a=blob;f=scripts/docs-build.sh;
+
+Hooks
+=====
+
+The document builder script supports pre-hook. If you need to execute some
+commands before starting to build process (e.g. generate API rst from codes),
+create '<your repo>/docs/pre-hook.sh' which will be executed (using 'source')
+once before all document build start.
diff --git a/scripts/docs-build.sh b/scripts/docs-build.sh
index afa68ec65..043fd1b94 100755
--- a/scripts/docs-build.sh
+++ b/scripts/docs-build.sh
@@ -190,13 +190,12 @@ fi
 
 virtualenv "$VENV_DIR"
 source "$VENV_DIR/bin/activate"
+pip install -r "$OPNFVDOCS_DIR/etc/requirements.txt"
 
 if [ -e "$DOCS_DIR/pre-hook.sh" ]; then
     source "$DOCS_DIR/pre-hook.sh"
 fi
 
-pip install -r "$OPNFVDOCS_DIR/etc/requirements.txt"
-
 find $DOCS_DIR -name $INDEX_RST -printf '%h\n' | while read dir
 do
     name=$(generate_name $dir)