Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: use files instead of pipe in execCmd ......................................................................
packaging: use files instead of pipe in execCmd Add a flag for common_utils.execCmd to use real files instead of the subprocess pipes (subprocess.PIPE) this change has been crafted in order to by pass "forked fork" processes which holds stdout & stderr open and hangs subprocess when using pipes for stdout & stderr. Change-Id: Ica39e946b9ee3e13b343ebe619ef2c770d170e8f --- M packaging/fedora/setup/common_utils.py M packaging/fedora/setup/engine-setup.py 2 files changed, 25 insertions(+), 5 deletions(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1377 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ica39e946b9ee3e13b343ebe619ef2c770d170e8f Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ronen Angluster <[email protected]> Gerrit-Reviewer: Alex Lourie <[email protected]> Gerrit-Reviewer: Barak Azulay <[email protected]> Gerrit-Reviewer: Eyal Edri <[email protected]> Gerrit-Reviewer: Idan Mansano <[email protected]> Gerrit-Reviewer: Ofer Schreiber <[email protected]> Gerrit-Reviewer: Ronen Angluster <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
