Skip to content

Only try to chmod /nix/var/nix/profiles/per-user when necessary (backport #12391)#12536

Merged
mergify[bot] merged 1 commit into2.25-maintenancefrom
mergify/bp/2.25-maintenance/pr-12391
Feb 20, 2025
Merged

Only try to chmod /nix/var/nix/profiles/per-user when necessary (backport #12391)#12536
mergify[bot] merged 1 commit into2.25-maintenancefrom
mergify/bp/2.25-maintenance/pr-12391

Conversation

@mergify
Copy link
Contributor

@mergify mergify bot commented Feb 20, 2025

Motivation

Try to get the current version working in renovate.

Context

#8074

in short: nix always chmod's /nix/var/nix/profiles/per-user when not using the daemon even if the permissions are already correct. By first checking if the chmod is necessary we can avoid requiring more permissions then necessary.

The error before was:

error: could not set permissions on '/nix/var/nix/profiles/per-user' to 755: Operation not permitted

and now is avoided if the permissons are already correct.


Add 👍 to pull requests you find important.

The Nix maintainer team uses a GitHub project board to schedule and track reviews.


This is an automatic backport of pull request #12391 done by [Mergify](https://mergify.com).

Co-authored-by: Eelco Dolstra <edolstra@gmail.com>
(cherry picked from commit dcbf4dc)
@mergify mergify bot requested a review from Ericson2314 as a code owner February 20, 2025 16:11
@mergify mergify bot added automatic backport This PR is a backport produced by automation (does not trigger backporting) merge-queue labels Feb 20, 2025
@github-actions github-actions bot added the store Issues and pull requests concerning the Nix store label Feb 20, 2025
@mergify mergify bot merged commit 40f33b5 into 2.25-maintenance Feb 20, 2025
23 checks passed
@mergify mergify bot deleted the mergify/bp/2.25-maintenance/pr-12391 branch February 20, 2025 16:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automatic backport This PR is a backport produced by automation (does not trigger backporting) merge-queue store Issues and pull requests concerning the Nix store

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant