Skip to content

test(storage): enable long paths for bare Git fixtures - #2435

Merged
jackwener merged 1 commit into
apache:mainfrom
liugddx:fix/windows-git-ref-longpaths
Aug 7, 2026
Merged

test(storage): enable long paths for bare Git fixtures#2435
jackwener merged 1 commit into
apache:mainfrom
liugddx:fix/windows-git-ref-longpaths

Conversation

@liugddx

@liugddx liugddx commented Aug 7, 2026

Copy link
Copy Markdown
Member

Summary

  • run bare-repository test Git commands with core.longpaths=true
  • match the fixed Git runtime used by the managed workspace implementation
  • keep the conflicting managed head-ref fixture valid under deeply nested Windows CI temp roots

Closes the remaining Git workspace failure tracked by #2142.

Validation

  • Core and Storage builds
  • managed head-ref conflict test passes with TEMP/TMP nested to match the failing Actions path depth
  • Biome format check
  • git diff --check

@jackwener jackwener left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Adversarial review — approve

Test-only: gitBare now passes -c core.longpaths=true, matching production git-workspace-service fixed runtime. Deep Windows CI temp paths no longer break bare-repo head-ref fixtures. No production code change. LGTM.

@jackwener
jackwener merged commit ba63744 into apache:main Aug 7, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants