From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: From: Roland Hieber Date: Wed, 25 Sep 2019 15:13:40 +0200 Message-Id: <20190925131339.17018-1-rhi@pengutronix.de> In-Reply-To: <20190911075048.8125-1-rhi@pengutronix.de> References: <20190911075048.8125-1-rhi@pengutronix.de> MIME-Version: 1.0 Subject: [ptxdist] [PATCH v2] 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 --- v1 -> v2: - prevent "AttributeError: 'NoneType' object has no attribute 'strip'" if none of the checked environment variables are set 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..73dd4a2add1c 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 or not ptxdist.strip(): + ptxdist = os.environ.get("ptxdist") + if not ptxdist or 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