From f48d30eb9fdfdac6d82551dae59ad8667a1b412a Mon Sep 17 00:00:00 2001 From: Mohammed Keyvanzadeh Date: Thu, 17 Oct 2024 16:35:02 +0330 Subject: [PATCH] build: fix path concatenation MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - The `Path` class does not support concatenation with the `+` operator, so use the `/` operator instead. - When concatenating paths, if the operand is an absolute path the previous path is ignored, so change `/include` to `include`. PR-URL: https://github.com/nodejs/node/pull/55387 Reviewed-By: Yagiz Nizipli Reviewed-By: Michaël Zasso Reviewed-By: Richard Lau Reviewed-By: Luigi Pinca --- configure.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure.py b/configure.py index 12faf7abaa8c96..a5f47af1748903 100755 --- a/configure.py +++ b/configure.py @@ -1356,7 +1356,7 @@ def configure_zos(o): o['variables']['node_static_zoslib'] = b(True) if options.static_zoslib_gyp: # Apply to all Node.js components for now - o['variables']['zoslib_include_dir'] = Path(options.static_zoslib_gyp).parent + '/include' + o['variables']['zoslib_include_dir'] = Path(options.static_zoslib_gyp).parent / 'include' o['include_dirs'] += [o['variables']['zoslib_include_dir']] else: raise Exception('--static-zoslib-gyp= is required.')