Skip to content

Commit 42fc0b7

Browse files
Kkuntal990claude
andcommitted
Fix LFS cache bug: always run git lfs checkout after cache restore
On cache hit, `git lfs pull` was skipped entirely. Since `git lfs pull` performs both `git lfs fetch` (download objects) and `git lfs checkout` (replace pointer files with actual content), skipping it on cache hit meant LFS pointer files were never replaced in the working tree. MATLAB then tried to read ~130-byte pointer text as binary data, causing 12 test failures and a multi-hour hang on binary_pop_snapread_wrapperTest. Fix: add an always-run `git lfs checkout` step after cache restore. Also add timeout-minutes: 120 as a safety net against future hangs. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent ae0c55a commit 42fc0b7

1 file changed

Lines changed: 5 additions & 1 deletion

File tree

.github/workflows/ci.yml

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,9 +23,10 @@ jobs:
2323
# This workflow contains a single job called "build"
2424
build:
2525
# The type of runner that the job will run on
26-
runs-on:
26+
runs-on:
2727
labels: ubuntu-latest-m
2828
continue-on-error: true
29+
timeout-minutes: 120
2930

3031
strategy:
3132
matrix:
@@ -56,6 +57,9 @@ jobs:
5657
if: steps.cache.outputs.cache-hit != 'true'
5758
run: git lfs pull
5859

60+
- name: Git LFS Checkout
61+
run: git lfs checkout
62+
5963
# Add virtual display support for graphical tests
6064
- name: Setup Xvfb for virtual display
6165
run: |

0 commit comments

Comments
 (0)