Skip to content

Commit 320d02e

Browse files
author
Andrija Kolic
committed
[GR-60134] Add warning when parsing args with spaces in mx_sdk_benchmark.parse_prefixed_args
PullRequest: graal/19470
2 parents 73282d7 + d48969b commit 320d02e

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

sdk/mx.sdk/mx_sdk_benchmark.py

+5-1
Original file line numberDiff line numberDiff line change
@@ -1869,7 +1869,11 @@ def parse_prefixed_args(prefix, args):
18691869
ret = []
18701870
for arg in args:
18711871
if arg.startswith(prefix):
1872-
parsed = arg.split(' ')[0].split(prefix)[1]
1872+
words_in_arg = arg.split(' ')
1873+
if len(words_in_arg) > 1:
1874+
# We will monitor our logs for this warning and then fix this method once we are certain no jobs break (GR-60134)
1875+
mx.warn(f"A prefixed arg that includes spaces is being parsed, ignoring everything that comes after the first space! The arg in question is: '{arg}'")
1876+
parsed = words_in_arg[0].split(prefix)[1]
18731877
if parsed not in ret:
18741878
ret.append(parsed)
18751879
return ret

0 commit comments

Comments
 (0)