From c2272c6ecb7c3fd6e9bdeb10a237f7bba3edf1c7 Mon Sep 17 00:00:00 2001
From: Yu Yang <yuyang18@baidu.com>
Date: Sat, 5 Aug 2017 19:05:52 +0800
Subject: [PATCH] Only make install when it necessary in build.sh

---
 paddle/scripts/docker/build.sh | 15 +++++++++++----
 1 file changed, 11 insertions(+), 4 deletions(-)

diff --git a/paddle/scripts/docker/build.sh b/paddle/scripts/docker/build.sh
index 92e59a27ac..e0916788a3 100644
--- a/paddle/scripts/docker/build.sh
+++ b/paddle/scripts/docker/build.sh
@@ -57,13 +57,11 @@ cmake .. \
 
 cat <<EOF
 ============================================
-Building and installing in /paddle/build ...
+Building in /paddle/build ...
    Build unit tests: ${WITH_TESTING:-OFF}
 ============================================
 EOF
-make install -j `nproc`
-pip install /usr/local/opt/paddle/share/wheels/*.whl
-paddle version
+make -j `nproc`
 
 if [ ${WITH_TESTING:-OFF} == "ON" ] && [ ${RUN_TEST:-OFF} == "ON" ] ; then
 cat <<EOF
@@ -71,6 +69,10 @@ cat <<EOF
 Running unit tests ...
 ========================================
 EOF
+    # make install should also be test when unittest
+    make install -j `nproc`
+    pip install /usr/local/opt/paddle/share/wheels/*.whl
+    paddle version
     ctest --output-on-failure
 fi
 
@@ -86,6 +88,11 @@ Building documentation ...
    In /paddle/build_doc
 ========================================
 EOF
+    # build documentation need install Paddle before
+    make install -j `nproc`
+    pip install /usr/local/opt/paddle/share/wheels/*.whl
+    paddle version
+
     mkdir -p /paddle/build_doc
     pushd /paddle/build_doc
     cmake .. \
-- 
GitLab