Skip to content

Commit 3011f9a

Browse files
Lumengineclaude
andcommitted
ci: fix sed delimiter in CUDA-arch strip regex
The `|` alternation inside the pattern collided with `|` used as the s/// delimiter, so sed bailed with "unknown option to `s'" and the arches were never stripped. Switch delimiter to `#`. Verified locally that the pattern leaves only compute_75. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent dd7a1a0 commit 3011f9a

1 file changed

Lines changed: 3 additions & 1 deletion

File tree

.github/workflows/BuildPhysX.yml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,9 @@ jobs:
7474
changed=0
7575
for f in compiler/vc17win64-gpu-md/**/*.vcxproj; do
7676
# Strip sm_80 / sm_86 / sm_89 / sm_90 generate-code entries.
77-
sed -i -E 's| --generate-code=arch=compute_(80\|86\|89\|90),code=\[[^]]+\]||g' "$f"
77+
# NB: `#` delimiter avoids clashing with `|` alternation inside the
78+
# pattern (which `s|...|` would mis-parse as the end of the regex).
79+
sed -i -E 's# --generate-code=arch=compute_(80|86|89|90),code=\[[^]]+\]##g' "$f"
7880
# Serialize ptxas threads per nvcc invocation.
7981
sed -i 's| -t0 | -t1 |g' "$f"
8082
changed=$((changed+1))

0 commit comments

Comments
 (0)