Dan Kenigsberg has submitted this change and it was merged. Change subject: utils: Make CommandPath try to find the executable ......................................................................
utils: Make CommandPath try to find the executable Before this patch, if the executable was not one of the proposed paths in the parameters, it would raise straight an ENOENT OSError. With this patch, it will use distutils find_executable before bailing out. Change-Id: I0bfa722778e295e23f03791179bf8c53172d34b2 Signed-off-by: Antoni Segura Puimedon <t...@midokura.com> Reviewed-on: http://gerrit.ovirt.org/36166 Reviewed-by: Yaniv Bronhaim <ybron...@redhat.com> Reviewed-by: Dan Kenigsberg <dan...@redhat.com> --- M lib/vdsm/utils.py M tests/utilsTests.py 2 files changed, 17 insertions(+), 3 deletions(-) Approvals: Yaniv Bronhaim: Looks good to me, but someone else must approve Antoni Segura Puimedon: Verified Dan Kenigsberg: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/36166 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0bfa722778e295e23f03791179bf8c53172d34b2 Gerrit-PatchSet: 5 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Antoni Segura Puimedon <t...@midokura.com> Gerrit-Reviewer: Antoni Segura Puimedon <t...@midokura.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Federico Simoncelli <fsimo...@redhat.com> Gerrit-Reviewer: Ryu Ishimoto <r...@midokura.com> Gerrit-Reviewer: Yaniv Bronhaim <ybron...@redhat.com> Gerrit-Reviewer: Yeela Kaplan <ykap...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches