Skip to content

Conversation

tjgq
Copy link
Contributor

@tjgq tjgq commented Oct 17, 2025

On Windows, a different kind of filesystem object (junction or symlink) needs to be created depending on the target type, which is not always inspectable in "building without the bytes" scenarios where the symlink is created ahead of its target.

This CL only introduces the formal argument and adjusts the WindowsFileSystem#createSymbolicLink implementation accordingly. A followup CL will pass in the argument where required.

Progress on #21747 and #26701.

PiperOrigin-RevId: 802504976
Change-Id: I912a84ba6734bf0562f369c7659bcc25f259e70f

@tjgq tjgq requested a review from a team as a code owner October 17, 2025 14:05
@github-actions github-actions bot added team-Remote-Exec Issues and PRs for the Execution (Remote) team awaiting-review PR is awaiting review from an assigned reviewer labels Oct 17, 2025
@tjgq tjgq force-pushed the 8.5.0-symlink-type branch 3 times, most recently from d702136 to 1793820 Compare October 17, 2025 14:20
… the target.

On Windows, a different kind of filesystem object (junction or symlink) needs to be created depending on the target type, which is not always inspectable in "building without the bytes" scenarios where the symlink is created ahead of its target.

This CL only introduces the formal argument and adjusts the WindowsFileSystem#createSymbolicLink implementation accordingly. A followup CL will pass in the argument where required.

Progress on bazelbuild#21747 and bazelbuild#26701.

PiperOrigin-RevId: 802504976
Change-Id: I912a84ba6734bf0562f369c7659bcc25f259e70f
@tjgq tjgq force-pushed the 8.5.0-symlink-type branch from 1793820 to 19a51a0 Compare October 17, 2025 14:25
@tjgq
Copy link
Contributor Author

tjgq commented Oct 17, 2025

#27323 must be submitted first, then it should be fine to come back to this one.

@iancha1992 iancha1992 enabled auto-merge October 17, 2025 17:43
@iancha1992 iancha1992 added this to the 8.5.0 release blockers milestone Oct 17, 2025
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-Remote-Exec Issues and PRs for the Execution (Remote) team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants