commit ecc1f41d5053807a70b51bcf2247b51d0ac81ae1
author Federico Ressi <federico.ressi@intel.com> Mon Dec 28 15:14:13 2015 +0100
committer Federico Ressi <federico.ressi@intel.com> Fri Jan 08 15:29:16 2016 +0000
tree ae6f491f51f5d14101b841e24b7b5a9519a05171
parent 828936c6ac8045fbb985ed2e89cdaa9a7c01241a

Don't reinstall package just after installing.

Boolean logic used in install_package was wrong:
on nominal flow packages were actually installed twice.
This should fix it.

Change-Id: Ia465414936b272d04523a11f83d6ded378fe1daf
Closes-Bug: #1518544

functions-common

diff --git a/functions-common b/functions-common
index cb2998b..361d27c 100644
--- a/functions-common
+++ b/functions-common
@@ -1245,7 +1245,9 @@
 # install_package package [package ...]
 function install_package {
     update_package_repo
-    real_install_package $@ || RETRY_UPDATE=True update_package_repo && real_install_package $@
+    if ! real_install_package "$@"; then
+        RETRY_UPDATE=True update_package_repo && real_install_package "$@"
+    fi
 }
 
 # Distro-agnostic function to tell if a package is installed