fix: remove usage of XDG_RUNTIME_DIR for TMP #10883
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
tomberek
force-pushed
the
tomberek.remove_xdg_tmp
branch
from
6bceaac to
1363f51
Compare
roberth
approved these changes
Jun 10, 2024
roberth
added
backport 2.18-maintenance
This comment was marked as resolved.
This comment was marked as resolved.
1 similar comment
|
Backport failed for Please cherry-pick the changes locally and resolve any conflicts. git fetch origin 2.18-maintenance
git worktree add -d .worktree/backport-10883-to-2.18-maintenance origin/2.18-maintenance
cd .worktree/backport-10883-to-2.18-maintenance
git switch --create backport-10883-to-2.18-maintenance
git cherry-pick -x 1363f51bcb24ab9948b7b5093490a009947f7453 |
|
Successfully created backport PR for |
This comment was marked as off-topic.
This comment was marked as off-topic.
roberth
added
backport 2.18-maintenance
|
Successfully created backport PR for |
This comment was marked as duplicate.
This comment was marked as duplicate.
|
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/2024-06-10-nix-team-meeting-minutes-151/46956/1 |
tebowy
pushed a commit
to tebowy/nix
that referenced
this pull request
Jul 11, 2024
This is a squash of upstream PRs NixOS#10303, NixOS#10312 and NixOS#10883. fix: Treat empty TMPDIR as unset Fixes an instance of nix: src/libutil/util.cc:139: nix::Path nix::canonPath(PathView, bool): Assertion `path != ""' failed. ... which I've been getting in one of my shells for some reason. I have yet to find out why TMPDIR was empty, but it's no reason for Nix to break. (cherry picked from commit c3fb2aa) fix: Treat empty XDG_RUNTIME_DIR as unset See preceding commit. Not observed in the wild, but is sensible and consistent with TMPDIR behavior. (cherry picked from commit b9e7f5a) local-derivation-goal.cc: Reuse defaultTempDir() (cherry picked from commit fd31945) fix: remove usage of XDG_RUNTIME_DIR for TMP (cherry picked from commit 1363f51) tests/functional: Add count() (cherry picked from commit 6221770) Remove uncalled for message (cherry picked from commit b1fe388) Add build-dir setting (cherry picked from commit 8b16cce) Change-Id: Ic7b75ff0b6a3b19e50a4ac8ff2d70f15c683c16a
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Motivation
nix develop nixmakes clangd fill upXDG_RUNTIME_DIRtmpfs #10430Context
Removes usage of XDG_RUNTIME_DIR
Priorities and Process
Add 👍 to pull requests you find important.
The Nix maintainer team uses a GitHub project board to schedule and track reviews.