Skip to content

Commit

Permalink
CI: hack to shorten windows paths
Browse files Browse the repository at this point in the history
 - win has a 32k char process arg limit
 - test classpath args for the junit JVM can go over this limit if
   the workspace path is too long and the process will fail to start
 - this moves the workspace close to the drive letter as workaround
  • Loading branch information
mbien committed Apr 6, 2024
1 parent cbb5eaa commit 266d869
Showing 1 changed file with 14 additions and 1 deletion.
15 changes: 14 additions & 1 deletion .github/workflows/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2417,8 +2417,20 @@ jobs:
exclude:
- os: ${{ (contains(github.event.pull_request.labels.*.name, 'PHP') || contains(github.event.pull_request.labels.*.name, 'ci:all-tests') || github.event_name != 'pull_request') && 'nothing' || 'windows-latest' }}
fail-fast: false
defaults:
run:
# hack: shorten paths to stay below 32k char process arg limit of windows
working-directory: ${{ contains(matrix.os, 'windows') && 'D:\\nb' || github.workspace }}

steps:

- name: shorten paths for windows
if: contains(matrix.os, 'windows')
run: |
mkdir nb && cd nb
echo "$(pwd)" > GITHUB_WORKSPACE
working-directory: "D:\\"

- name: Set up JDK ${{ matrix.java }}
uses: actions/setup-java@v4
with:
Expand All @@ -2444,6 +2456,7 @@ jobs:
uses: actions/download-artifact@v4
with:
name: build
path: $GITHUB_WORKSPACE

- name: Extract
run: tar --zstd -xf build.tar.zst
Expand Down Expand Up @@ -2544,7 +2557,7 @@ jobs:
uses: test-summary/action@v2
if: failure()
with:
paths: "./*/*/build/test/*/results/TEST-*.xml"
paths: "$GITHUB_WORKSPACE/*/*/build/test/*/results/TEST-*.xml"


lsp-test:
Expand Down

0 comments on commit 266d869

Please sign in to comment.