From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: From: Roland Hieber Date: Wed, 11 Sep 2019 09:50:49 +0200 Message-Id: <20190911075048.8125-1-rhi@pengutronix.de> MIME-Version: 1.0 Subject: [ptxdist] [PATCH] configure_helper.py: check for emptyish ptxdist environment variables List-Id: PTXdist Development Mailing List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: ptxdist@pengutronix.de Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: ptxdist-bounces@pengutronix.de Sender: "ptxdist" To: ptxdist@pengutronix.de Cc: Roland Hieber When the environment variable exists, but is empty, os.environment.get() will return its value instead of using the supplied default. Check for cases like that to prevent calling an empty command. Signed-off-by: Roland Hieber --- scripts/configure_helper.py | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/scripts/configure_helper.py b/scripts/configure_helper.py index c7b46f3b3846..3261a548ec96 100755 --- a/scripts/configure_helper.py +++ b/scripts/configure_helper.py @@ -151,7 +151,12 @@ def abort(message): exit(1) def ask_ptxdist(pkg): - ptxdist = os.environ.get("PTXDIST", os.environ.get("ptxdist", "ptxdist")) + ptxdist = os.environ.get("PTXDIST") + if not ptxdist.strip(): + ptxdist = os.environ.get("ptxdist") + if not ptxdist.strip(): + ptxdist = "ptxdist" + p = subprocess.Popen([ ptxdist, "-k", "make", "/print-%s_DIR" % pkg, "/print-%s_SUBDIR" % pkg, -- 2.23.0 _______________________________________________ ptxdist mailing list ptxdist@pengutronix.de