Skip to content

Commit 5eb6c94

Browse files
VoltrexKeyvamarco-ippolito
authored andcommittedJan 24, 2025
build: fix path concatenation
- 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: #55387 Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com> Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Richard Lau <rlau@redhat.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
1 parent 85d8eb3 commit 5eb6c94

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed
 

Diff for: ‎configure.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -1308,7 +1308,7 @@ def configure_zos(o):
13081308
o['variables']['node_static_zoslib'] = b(True)
13091309
if options.static_zoslib_gyp:
13101310
# Apply to all Node.js components for now
1311-
o['variables']['zoslib_include_dir'] = Path(options.static_zoslib_gyp).parent + '/include'
1311+
o['variables']['zoslib_include_dir'] = Path(options.static_zoslib_gyp).parent / 'include'
13121312
o['include_dirs'] += [o['variables']['zoslib_include_dir']]
13131313
else:
13141314
raise Exception('--static-zoslib-gyp=<path to zoslib.gyp file> is required.')

0 commit comments

Comments
 (0)