Skip to content

Fix relative 'path:' flakerefs in the CLI#12254

Merged
Ericson2314 merged 2 commits intoNixOS:masterfrom
DeterminateSystems:fix-relative-path-on-cli
Jan 15, 2025
Merged

Fix relative 'path:' flakerefs in the CLI#12254
Ericson2314 merged 2 commits intoNixOS:masterfrom
DeterminateSystems:fix-relative-path-on-cli

Conversation

@edolstra
Copy link
Copy Markdown
Member

Motivation

Fixes #12248.

This also handles relative git+file: flakerefs while we're at it (these crashed with an assertion failure).

Context


Add 👍 to pull requests you find important.

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

And handle relative 'git+file:' flakerefs while we're at it (these
crashed with an assertion failure).

Fixes NixOS#12248.
@github-actions github-actions bot added the with-tests Issues related to testing. PRs with tests have some priority label Jan 14, 2025
@edolstra edolstra added regression Something doesn't work anymore backport 2.25-maintenance labels Jan 14, 2025
@github-actions github-actions bot added the fetching Networking with the outside (non-Nix) world, input locking label Jan 14, 2025
@edolstra edolstra requested a review from Ericson2314 January 15, 2025 20:17
@Ericson2314 Ericson2314 merged commit e02026a into NixOS:master Jan 15, 2025
@edolstra edolstra deleted the fix-relative-path-on-cli branch January 15, 2025 20:20
edolstra added a commit that referenced this pull request Jan 15, 2025
…2254

Fix relative 'path:' flakerefs in the CLI (backport #12254)
@nixos-discourse
Copy link
Copy Markdown

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/2024-01-15-nix-team-meeting-minutes-208/58993/1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fetching Networking with the outside (non-Nix) world, input locking regression Something doesn't work anymore 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.

Regression: nix 2.25 not allowing path:. as flake-url

3 participants