Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: Moving to use execCmd in all scripts (#731686) ...................................................................... packaging: Moving to use execCmd in all scripts (#731686) https://bugzilla.redhat.com/show_bug.cgi?id=731686 * Renamed all uses of execExternalCmd() to use execCmd(). * Removed execExternalCmd function from common_utils.py Change-Id: If5428890f9dcf8eafaa1ab059ee5d9e84f3f7f9a Signed-off-by: Alex Lourie <alourie(a)redhat.com&gt; --- M packaging/fedora/setup/common_utils.py M packaging/fedora/setup/engine-cleanup.py M packaging/fedora/setup/engine-setup.py M packaging/fedora/setup/engine-upgrade.py M packaging/fedora/setup/engine_validators.py M packaging/fedora/setup/nfsutils.py 6 files changed, 313 insertions(+), 144 deletions(-) Approvals: Alex Lourie: Verified; Looks good to me, but someone else must approve Ofer Schreiber: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/4943 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If5428890f9dcf8eafaa1ab059ee5d9e84f3f7f9a Gerrit-PatchSet: 8 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alex Lourie <alourie(a)redhat.com&gt; Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com&gt; Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com&gt; Gerrit-Reviewer: Moran Goldboim <mgoldboi(a)redhat.com&gt; Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com&gt;