Skip to content

Commit aafcff6

Browse files
tashianclaude
andcommitted
Use eval for command inputs to preserve shell semantics
Direct ${VAR} expansion doesn't recognize VAR=value prefix assignments after parameter expansion, breaking the default 'V=1 make build'. Using eval restores the original shell parsing behavior. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 6b2ec58 commit aafcff6

3 files changed

Lines changed: 3 additions & 3 deletions

File tree

.github/workflows/codeql-analysis.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -115,7 +115,7 @@ jobs:
115115
env:
116116
CODEQL_BUILD_CMD: ${{ inputs.codeql-build-cmd }}
117117
run: |
118-
${CODEQL_BUILD_CMD}
118+
eval "${CODEQL_BUILD_CMD}"
119119
-
120120
name: Perform CodeQL Analysis
121121
uses: github/codeql-action/analyze@89a39a4e59826350b863aa6b6252a07ad50cf83e # v4.32.4

.github/workflows/goBuild.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -93,4 +93,4 @@ jobs:
9393
name: Build
9494
env:
9595
BUILD_CMD: ${{ inputs.build-command }}
96-
run: ${BUILD_CMD}
96+
run: eval "${BUILD_CMD}"

.github/workflows/goTest.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -141,7 +141,7 @@ jobs:
141141
env:
142142
TEST_CMD: ${{ inputs.test-command }}
143143
GOTESTSUM_JSONFILE: gotestsum.json
144-
run: ${TEST_CMD}
144+
run: eval "${TEST_CMD}"
145145

146146
-
147147
name: Annotate Test Suite Results

0 commit comments

Comments
 (0)