From d26095883921ac187dfccde229f38ceee5eea745 Mon Sep 17 00:00:00 2001
From: Ondrej Samohel <ondrej.samohel@orbi.tools>
Date: Mon, 6 Sep 2021 10:43:50 +0200
Subject: [PATCH] try to get error log from failed build

---
 tools/docker_build.sh | 44 +++++++++++++++++++++++++++++++------------
 1 file changed, 32 insertions(+), 12 deletions(-)

diff --git a/tools/docker_build.sh b/tools/docker_build.sh
index dca217d534..c27041a1af 100755
--- a/tools/docker_build.sh
+++ b/tools/docker_build.sh
@@ -20,6 +20,28 @@ realpath () {
   echo $(cd $(dirname "$1"); pwd)/$(basename "$1")
 }
 
+create_container () {
+  if [ ! -f "$openpype_root/build/docker-image.id" ]; then
+    echo -e "${BIRed}!!!${RST} Docker command failed, cannot find image id."
+    exit 1
+  fi
+  local id=$(<"$openpype_root/build/docker-image.id")
+  echo -e "${BIYellow}---${RST} Creating container from $id ..."
+  local cid="$(docker create $id bash)"
+  if [ $? -ne 0 ] ; then
+    echo -e "${BIRed}!!!${RST} Cannot create container."
+    exit 1
+  fi
+  return $cid
+}
+
+retrieve_build_log () {
+  create_container
+  local cid=$?
+  echo -e "${BIYellow}***${RST} Copying build log to ${BIWhite}$openpype_root/build/build.log${RST}"
+  docker cp "$cid:/opt/openpype/build/build.log" "$openpype_root/build"
+}
+
 # Main
 main () {
   openpype_root=$(realpath $(dirname $(dirname "${BASH_SOURCE[0]}")))
@@ -28,34 +50,32 @@ main () {
   echo -e "${BIYellow}---${RST} Cleaning build directory ..."
   rm -rf "$openpype_root/build" && mkdir "$openpype_root/build" > /dev/null
 
-  version_command="import os;exec(open(os.path.join('$openpype_root', 'openpype', 'version.py')).read());print(__version__);"
-  openpype_version="$(python3 <<< ${version_command})"
+  local version_command="import os;exec(open(os.path.join('$openpype_root', 'openpype', 'version.py')).read());print(__version__);"
+  local openpype_version="$(python3 <<< ${version_command})"
 
   echo -e "${BIGreen}>>>${RST} Running docker build ..."
   # docker build --pull --no-cache -t pypeclub/openpype:$openpype_version .
-  docker build --pull -t pypeclub/openpype:$openpype_version .
+  docker build --pull --iidfile $openpype_root/build/docker-image.id -t pypeclub/openpype:$openpype_version .
   if [ $? -ne 0 ] ; then
+    echo $?
     echo -e "${BIRed}!!!${RST} Docker build failed."
+    retrieve_build_log
     return 1
   fi
 
   echo -e "${BIGreen}>>>${RST} Copying build from container ..."
-  echo -e "${BIYellow}---${RST} Creating container from pypeclub/openpype:$openpype_version ..."
-  id="$(docker create -ti pypeclub/openpype:$openpype_version bash)"
-  if [ $? -ne 0 ] ; then
-    echo -e "${BIRed}!!!${RST} Cannot create just built container."
-    return 1
-  fi
+  create_container
+  local cid=$?
   echo -e "${BIYellow}---${RST} Copying ..."
-  docker cp "$id:/opt/openpype/build/exe.linux-x86_64-3.7" "$openpype_root/build"
-  docker cp "$id:/opt/openpype/build/build.log" "$openpype_root/build"
+  docker cp "$cid:/opt/openpype/build/exe.linux-x86_64-3.7" "$openpype_root/build"
+  docker cp "$cid:/opt/openpype/build/build.log" "$openpype_root/build"
   if [ $? -ne 0 ] ; then
     echo -e "${BIRed}!!!${RST} Copying failed."
     return 1
   fi
 
   echo -e "${BIGreen}>>>${RST} Fixing user ownership ..."
-  username="$(logname)"
+  local username="$(logname)"
   chown -R $username ./build
 
   echo -e "${BIGreen}>>>${RST} All done, you can delete container:"