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

[Backport 2.20-maintenance] Fix adding symlink to the sandbox paths #10471

Merged
merged 2 commits into from
Apr 11, 2024

Conversation

github-actions[bot]
Copy link

Automatic backport to 2.20-maintenance, triggered by a label in #10456.

Regression test for #9579

(cherry picked from commit 872d93e)
…o a symlink out of the store

Bind-mounting symlinks is apparently not possible, which is why the
thing was failing.

Fortunately, symlinks are small, so we can fallback to copy them at no cost.

Fix #9579

Co-authored-by: Artturin <[email protected]>
(cherry picked from commit 913db9f)
@thufschmitt thufschmitt reopened this Apr 11, 2024
@github-actions github-actions bot added the with-tests Issues related to testing. PRs with tests have some priority label Apr 11, 2024
@Ericson2314
Copy link
Member

Need backports of #10362 for this. I kicked that off. (It doesn't make a breaking change to Nix's headers so it should be OK.)

@thufschmitt
Copy link
Member

Dropped the last commit, should be good now

@thufschmitt thufschmitt merged commit 1cf8c57 into 2.20-maintenance Apr 11, 2024
12 checks passed
@thufschmitt thufschmitt deleted the backport-10456-to-2.20-maintenance branch April 11, 2024 16:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
with-tests Issues related to testing. PRs with tests have some priority
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants