Skip to content

Create external/_main under the execroot#29627

Draft
fmeum wants to merge 1 commit into
bazelbuild:masterfrom
fmeum:create-_main
Draft

Create external/_main under the execroot#29627
fmeum wants to merge 1 commit into
bazelbuild:masterfrom
fmeum:create-_main

Conversation

@fmeum
Copy link
Copy Markdown
Collaborator

@fmeum fmeum commented May 22, 2026

Description

Motivation

This is needed to correctly resolve rewritten cross-repo symlinks that arise as action inputs on Windows, which resolves symlinks logically (unlike Unix).

Fixes #29515

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 create-_main branch 2 times, most recently from b75072d to cdfee65 Compare May 22, 2026 20:40
@fmeum fmeum marked this pull request as ready for review May 22, 2026 21:07
@fmeum fmeum requested a review from Wyverald May 22, 2026 21:07
@github-actions github-actions Bot added team-Core Skyframe, bazel query, BEP, options parsing, bazelrc awaiting-review PR is awaiting review from an assigned reviewer labels May 22, 2026
@fmeum fmeum marked this pull request as draft May 23, 2026 09:50
Comment thread src/main/java/com/google/devtools/build/lib/buildtool/SymlinkForest.java Outdated
@fmeum fmeum force-pushed the create-_main branch 4 times, most recently from b324131 to ae47399 Compare May 24, 2026 20:51
@fmeum fmeum force-pushed the create-_main branch 3 times, most recently from 4f08ef3 to 2fecab1 Compare May 24, 2026 21:38
This is needed to correctly resolve rewritten cross-repo symlinks that
arise as action inputs on Windows, which resolves symlinks logically
(unlike Unix).

Skipped on filesystems that can only emulate symlinks (Windows without
Developer Mode / symlink privilege): a junction would create a cycle
through the `bazel-<workspace>` convenience symlink for any tool that
walks the workspace, and is also unnecessary since `replantSymlinks`
falls back to non-relative targets under the same condition.

Fixes bazelbuild#29515
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting-review PR is awaiting review from an assigned reviewer team-Core Skyframe, bazel query, BEP, options parsing, bazelrc

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Windows: Replanted repo symlinks through ../_main fail from execroot external repos

1 participant