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

Add repository_ctx.rename() for renaming files or directories #24020

Closed
wants to merge 2 commits into from

Conversation

jmillikin
Copy link
Contributor

This allows the directory structure of third-party archives to be rearranged without platform-specific commands, and without requiring symlink support from the platform.

@github-actions github-actions bot added team-ExternalDeps External dependency handling, remote repositiories, WORKSPACE file. awaiting-review PR is awaiting review from an assigned reviewer labels Oct 17, 2024
This allows the directory structure of third-party archives to be
rearranged without platform-specific commands, and without requiring
symlink support from the platform.
Copy link
Collaborator

@fmeum fmeum left a comment

Choose a reason for hiding this comment

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

@meteorcloudy Could you review for approval?

Copy link
Member

@meteorcloudy meteorcloudy left a comment

Choose a reason for hiding this comment

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

Thanks!

@meteorcloudy meteorcloudy added awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally and removed awaiting-review PR is awaiting review from an assigned reviewer labels Oct 18, 2024
@jmillikin
Copy link
Contributor Author

Gentle ping?

@meteorcloudy
Copy link
Member

@Wyverald Any additional concerns?

@copybara-service copybara-service bot closed this in 467ea0c Nov 7, 2024
@github-actions github-actions bot removed the awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally label Nov 7, 2024
@fmeum
Copy link
Collaborator

fmeum commented Nov 7, 2024

@bazel-io fork 8.0.0

bazel-io pushed a commit to bazel-io/bazel that referenced this pull request Nov 7, 2024
This allows the directory structure of third-party archives to be rearranged without platform-specific commands, and without requiring symlink support from the platform.

Closes bazelbuild#24020.

PiperOrigin-RevId: 694246834
Change-Id: I4831d610c5fb841640af1b1b864172ee38319a6b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team-ExternalDeps External dependency handling, remote repositiories, WORKSPACE file.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants