Merge pull request #303 from kleetus/bug/install_script

Making Building From Source Easier

bin/build

@@ -8,24 +8,29 @@ patch_sha=$($root_dir/bin/variables.sh patch_sha)
 export CPPFLAGS="-I${depends_dir}/${host}/include/boost -I${depends_dir}/${host}/include -L${depends_dir}/${host}/lib"
 echo "Using BTC directory: ${btc_dir}"
 
-cd "${root_dir}"
+cd "${root_dir}" || exit -1
 
 build_dependencies () {
   if [ -d "${btc_dir}" ]; then
-    pushd "${depends_dir}"
+    pushd "${depends_dir}" || exit -1
     echo "using host for dependencies: ${host}"
     if [ "${test}" = true ]; then
       make HOST=${host} NO_QT=1 NO_UPNP=1
     else
       make HOST=${host} NO_QT=1 NO_WALLET=1 NO_UPNP=1
     fi
-    popd
+    if test $? -eq 0; then
+      popd || exit -1
+    else
+      echo "Bitcoin's dependency building failed, please check the previous output for details."
+      exit -1
+    fi
   fi
 }
 
 get_patch_file () {
   if test -e "${root_dir/PATCH_VERSION}"; then
-    tag=`cat "${root_dir}/PATCH_VERSION" | xargs`
+    tag=`cat "${root_dir}/PATCH_VERSION" | xargs` || exit -1
   else
     echo "no tag file found, please create it in the root of the project as so: 'echo \"v0.10.2\" > PATCH_VERSION'"
     exit 1
@@ -33,33 +38,42 @@ get_patch_file () {
 }
 
 compare_patch () {
-  cd "${btc_dir}"
+  cd "${btc_dir}" || exit -1
   get_patch_file
   echo "running the diff command from HEAD to ${tag}"
-  diff=$(git diff ${tag}..HEAD)
+  last_commit=$(git rev-parse HEAD)
-  stripped_diff=$( echo -n "${diff}" | head -n $( expr `echo -n "${diff}" | wc -l` - 1 ) )
+  diff=$(git show ${last_commit})
+  stripped_diff=$( echo -n "${diff}" | tail -n $( expr `echo -n "${diff}" | wc -l` - 5 ) )
   matching_patch=`echo -n "${stripped_diff}" | diff -w "${root_dir}/etc/bitcoin.patch" -`
 }
 
 cache_files () {
   cache_file="${root_dir}"/cache/cache.tar
-  pushd "${btc_dir}"
+  pushd "${btc_dir}" || exit -1
   find . -type f \( -name "*.h" -or -name "*.hpp" -or -name \
 "*.ipp" -or -name "*.a" \) | tar -cf "${cache_file}" -T -
+  if test $? -ne 0; then
+    echo "We were trying to copy over your cached artifacts, but there was an issue."
+    exit -1
+  fi
   tar xf "${cache_file}" -C "${root_dir}"/cache
-  rm -fr "${cache_file}" 
+  if test $? -ne 0; then
-  popd
+    echo "We were trying to untar your cache, but there was an issue."
+    exit -1
+  fi
+  rm -fr "${cache_file}" >/dev/null 2>&1
+  popd || exit -1
 }
 
 debug=
 if [ "${BITCORENODE_ENV}" == "debug" ]; then
-  options=`cat ${root_dir}/bin/config_options_debug.sh`
+  options=`cat ${root_dir}/bin/config_options_debug.sh` || exit -1
 fi
 
 test=false
 if [ "${BITCORENODE_ENV}" == "test" ]; then
   test=true
-  options=`cat ${root_dir}/bin/config_options_test.sh`
+  options=`cat ${root_dir}/bin/config_options_test.sh` || exit -1
 fi
 
 if hash shasum 2>/dev/null; then
@@ -106,12 +120,12 @@ apply the current patch from "${root_dir}"/etc/bitcoin.patch? (y/N): "
       if [[ "${input}" =~ ^y|^Y ]]; then
         repatch=true
         echo "Removing directory: \"${btc_dir}\" and starting over!"
-        rm -fr "${btc_dir}"
+        rm -fr "${btc_dir}" >/dev/null 2>&1
       fi
     fi
     if [ "${repatch}" = false ]; then
       echo "Running make inside libbitcoind (assuming you've previously patched and configured libbitcoind)..."
-      cd "${btc_dir}"
+      cd "${btc_dir}" || exit -1
       only_make=true
     fi
   fi
@@ -120,10 +134,10 @@ apply the current patch from "${root_dir}"/etc/bitcoin.patch? (y/N): "
     echo "Cloning, patching, and building libbitcoind..."
     get_patch_file
     echo "attempting to checkout tag: ${tag} of bitcoin from github..."
-    cd "${root_dir}"
+    cd "${root_dir}" || exit -1
     #versions of git prior to 2.x will not clone correctly with --branch
     git clone --depth 1 https://github.com/bitcoin/bitcoin.git libbitcoind
-    cd "${btc_dir}" 
+    cd "${btc_dir}" || exit -1
     git fetch --tags
     git checkout "${tag}"
     echo '../patch-bitcoin.sh' "${btc_dir}"
@@ -131,13 +145,13 @@ apply the current patch from "${root_dir}"/etc/bitcoin.patch? (y/N): "
 
     if ! test -d .git; then
       echo 'Please point this script to an upstream bitcoin git repo.'
-      exit 1
+      exit -1
     fi
 
   fi
   build_dependencies
   echo './autogen.sh'
-  ./autogen.sh
+  ./autogen.sh || exit -1
 
   boost_libdir="--with-boost-libdir=${depends_dir}/${host}/lib"
 
@@ -146,7 +160,7 @@ apply the current patch from "${root_dir}"/etc/bitcoin.patch? (y/N): "
   ${full_options}
 
   echo 'make V=1'
-  make V=1
+  make V=1 || exit -1
 
   echo "Creating the sha marker for the patching in libbitcoind..."
   echo "Writing patch sha file to: \"${patch_sha}\""

bin/install

@@ -12,10 +12,10 @@ echo "Downloading binary: ${binary_url}"
 
 is_curl=true
 if hash curl 2>/dev/null; then
-  curl --fail -I $binary_url
+  curl --fail -I $binary_url >/dev/null 2>&1
 else
   is_curl=false
-  wget --server-response --spider $binary_url
+  wget --server-response --spider $binary_url >/dev/null 2>&1
 fi
 
 if test $? -eq 0; then