From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: mx.groups.io; dkim=missing; spf=none, err=permanent DNS error (domain: bsdio.com, ip: 65.103.231.193, mailfrom: rebecca@bsdio.com) Received: from muon.bsdio.com (muon.bsdio.com [65.103.231.193]) by groups.io with SMTP; Mon, 15 Jul 2019 15:25:34 -0700 Received: from muon.bsdio.com (localhost [127.0.0.1]) by muon.bsdio.com (Postfix) with ESMTP id 0FC0C77C59; Mon, 15 Jul 2019 16:25:54 -0600 (MDT) Received: from muon.bsdio.com ([127.0.0.1]) by muon.bsdio.com (muon.bsdio.com [127.0.0.1]) (amavisd-new, port 10026) with ESMTP id 6JAwosKgWjY8; Mon, 15 Jul 2019 16:25:53 -0600 (MDT) Received: from photon.int.bluestop.org (unknown [10.0.10.120]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by muon.bsdio.com (Postfix) with ESMTPSA; Mon, 15 Jul 2019 16:25:53 -0600 (MDT) From: rebecca@bsdio.com To: devel@edk2.groups.io, lersek@redhat.com, bob.c.feng@intel.com, liming.gao@intel.com, leif.lindholm@linaro.org, michael.d.kinney@intel.com, afish@apple.com Cc: Rebecca Cran Subject: [PATCH 5/6] edksetup.sh: Simplify SetupPython3 and SetupPython functions. Date: Mon, 15 Jul 2019 16:25:15 -0600 Message-Id: <20190715222516.53254-5-rebecca@bsdio.com> X-Mailer: git-send-email 2.22.0 In-Reply-To: <20190715222516.53254-1-rebecca@bsdio.com> References: <20190715222516.53254-1-rebecca@bsdio.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable On Linux, "whereis" matches python3, python3.7, as well as man pages, libs etc. While on macOS it only matches the specified name, and so misses python3.7. Improve this by looping over potential version numbers and seeing if such a binary exists and can be executed. Signed-off-by: Rebecca Cran --- edksetup.sh | 48 +++++++++--------------------------------------- 1 file changed, 9 insertions(+), 39 deletions(-) diff --git a/edksetup.sh b/edksetup.sh index 06d2f041e6..e2f116f8bc 100755 --- a/edksetup.sh +++ b/edksetup.sh @@ -107,24 +107,10 @@ function SetupEnv() =20 function SetupPython3() { - if [ $origin_version ];then - origin_version=3D - fi - for python in $(whereis python3) - do - python=3D$(echo $python | grep "[[:digit:]]$" || true) - python_version=3D${python##*python} - if [ -z "${python_version}" ] || (! command -v $python >/dev/null 2>= &1);then - continue - fi - if [ -z $origin_version ];then - origin_version=3D$python_version - export PYTHON_COMMAND=3D$python - continue - fi - if [[ "$origin_version" < "$python_version" ]]; then - origin_version=3D$python_version - export PYTHON_COMMAND=3D$python + for python in $(seq -f "python3.%g" 15 -1 1) python3; do + if command -v $python >/dev/null 2>&1; then + export PYTHON_COMMAND=3D$(which $python) + break fi done return 0 @@ -146,27 +132,11 @@ function SetupPython() SetupPython3 fi =20 - if [ $PYTHON3_ENABLE ] && [ $PYTHON3_ENABLE !=3D TRUE ] - then - if [ $origin_version ];then - origin_version=3D - fi - for python in $(whereis python2) - do - python=3D$(echo $python | grep "[[:digit:]]$" || true) - python_version=3D${python##*python} - if [ -z "${python_version}" ] || (! command -v $python >/dev/null = 2>&1);then - continue - fi - if [ -z $origin_version ] - then - origin_version=3D$python_version - export PYTHON_COMMAND=3D$python - continue - fi - if [[ "$origin_version" < "$python_version" ]]; then - origin_version=3D$python_version - export PYTHON_COMMAND=3D$python + if [ -n "$PYTHON3_ENABLE" ] && [ "$PYTHON3_ENABLE" !=3D "TRUE" ]; then + for python in $(seq -f "python2.%g" 10 -1 1) python2; do + if command -v $python >/dev/null 2>&1; then + export PYTHON_COMMAND=3D$(which $python) + break fi done return 0 --=20 2.22.0