Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Handle file type changes with in-process symlink creation #22087

Closed
wants to merge 2 commits into from

Conversation

fmeum
Copy link
Collaborator

@fmeum fmeum commented Apr 23, 2024

SymlinkTreeUpdater now correctly replaces symlinks with directories and vice versa in runfiles trees.

Fixes #20266

`SymlinkTreeUpdater` now correctly replaces symlinks with directories and vice versa in runfiles trees.
@github-actions github-actions bot added the awaiting-review PR is awaiting review from an assigned reviewer label Apr 23, 2024
@fmeum fmeum requested a review from tjgq April 23, 2024 10:03
@iancha1992 iancha1992 added the team-Rules-Server Issues for serverside rules included with Bazel label Apr 23, 2024
Copy link
Contributor

@tjgq tjgq left a comment

Choose a reason for hiding this comment

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

Thanks for fixing this!

src/test/shell/bazel/runfiles_test.sh Outdated Show resolved Hide resolved
src/test/shell/bazel/runfiles_test.sh Outdated Show resolved Hide resolved
src/test/shell/bazel/runfiles_test.sh Outdated Show resolved Hide resolved
@fmeum fmeum requested a review from tjgq April 24, 2024 10:46
@tjgq tjgq added awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally team-Performance Issues for Performance teams and removed awaiting-review PR is awaiting review from an assigned reviewer team-Rules-Server Issues for serverside rules included with Bazel labels Apr 24, 2024
@fmeum
Copy link
Collaborator Author

fmeum commented Apr 24, 2024

@bazel-io fork 7.2.0

bazel-io pushed a commit to bazel-io/bazel that referenced this pull request Apr 25, 2024
`SymlinkTreeUpdater` now correctly replaces symlinks with directories and vice versa in runfiles trees.

Fixes bazelbuild#20266

Closes bazelbuild#22087.

PiperOrigin-RevId: 628028591
Change-Id: Id94cea42f9cdfab001e56242a329b8d6d253ba29
@sgowroji sgowroji removed the awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally label Apr 26, 2024
github-merge-queue bot pushed a commit that referenced this pull request Apr 29, 2024
…2116)

`SymlinkTreeUpdater` now correctly replaces symlinks with directories
and vice versa in runfiles trees.

Fixes #20266

Closes #22087.

PiperOrigin-RevId: 628028591
Change-Id: Id94cea42f9cdfab001e56242a329b8d6d253ba29

Commit
4b18dbe

Co-authored-by: Fabian Meumertzheim <fabian@meumertzhe.im>
Kila2 pushed a commit to Kila2/bazel that referenced this pull request May 13, 2024
`SymlinkTreeUpdater` now correctly replaces symlinks with directories and vice versa in runfiles trees.

Fixes bazelbuild#20266

Closes bazelbuild#22087.

PiperOrigin-RevId: 628028591
Change-Id: Id94cea42f9cdfab001e56242a329b8d6d253ba29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team-Performance Issues for Performance teams
Projects
None yet
Development

Successfully merging this pull request may close these issues.

In-process runfile tree creation from SymlinkTreeAction fails for some consecutive builds
4 participants