Skip to content

Test wimdows#29629

Draft
fmeum wants to merge 3 commits into
bazelbuild:masterfrom
fmeum:fix-rrcc-symlink
Draft

Test wimdows#29629
fmeum wants to merge 3 commits into
bazelbuild:masterfrom
fmeum:fix-rrcc-symlink

Conversation

@fmeum
Copy link
Copy Markdown
Collaborator

@fmeum fmeum commented May 22, 2026

Description

Motivation

Build API Changes

No

Checklist

  • I have added tests for the new use cases (if any).
  • I have updated the documentation (if applicable).

Release Notes

RELNOTES: None

@fmeum fmeum force-pushed the fix-rrcc-symlink branch 2 times, most recently from 9a2d06d to a6ce4ac Compare May 22, 2026 21:30
Resolves the long-standing inconsistency in UploadManifest's handling
of non-dangling absolute vs. relative symlinks, behind a new
preserveAbsoluteSymlinks flag that is only set by the remote repo
contents cache so existing spawn-output behavior is preserved.

Without this, an rctx.symlink(Label, ...) that points into a sibling
repo gets uploaded as the resolved directory it points to. Any
relative symlinks inside that resolved tree are then materialized at a
location where their `..` targets no longer resolve, breaking actions
that read through them.
@fmeum fmeum force-pushed the fix-rrcc-symlink branch from a6ce4ac to 3d52f55 Compare May 23, 2026 06:27
@fmeum fmeum force-pushed the fix-rrcc-symlink branch from 0ab8096 to b6d6b0d Compare May 23, 2026 08:18
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.

1 participant