From 30f285da314dc5c9fa7457784d948bf5ddfab361 Mon Sep 17 00:00:00 2001 From: Ben Noordhuis Date: Fri, 7 Jun 2019 11:36:39 +0200 Subject: build: fix icu-i18n pkg-config version check MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The pkg_config() helper can either return a tuple of None values (no pkg-config installed) and that was what the check was testing for, but it can also return a tuple of empty strings when the package isn't installed. PR-URL: https://github.com/nodejs/node/pull/28118 Reviewed-By: Richard Lau Reviewed-By: Rich Trott Reviewed-By: Refael Ackermann (רפאל פלחי) Reviewed-By: Michael Dawson Reviewed-By: James M Snell --- configure.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'configure.py') diff --git a/configure.py b/configure.py index 71943cae2c..9d9f0aec0b 100755 --- a/configure.py +++ b/configure.py @@ -1348,7 +1348,7 @@ def configure_intl(o): # ICU from pkg-config. o['variables']['v8_enable_i18n_support'] = 1 pkgicu = pkg_config('icu-i18n') - if pkgicu[0] is None: + if not pkgicu[0]: error('''Could not load pkg-config data for "icu-i18n". See above errors or the README.md.''') (libs, cflags, libpath, icuversion) = pkgicu -- cgit v1.2.3