Skip to content

libexpr: Make hash mismatches while copying lazy paths to the store a…#15950

Merged
xokdvium merged 1 commit into
masterfrom
lazy-path-hash-mismatch-error
Jun 1, 2026
Merged

libexpr: Make hash mismatches while copying lazy paths to the store a…#15950
xokdvium merged 1 commit into
masterfrom
lazy-path-hash-mismatch-error

Conversation

@xokdvium
Copy link
Copy Markdown
Contributor

@xokdvium xokdvium commented Jun 1, 2026

… proper error

Motivation

This can happen when the local source tree (dirty git repo) gets modified when the nix process is evaluating stuff from it. Make this a proper error and not an assert. I've hit this issue myself once now.

Context


Add 👍 to pull requests you find important.

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

@xokdvium xokdvium requested a review from edolstra as a code owner June 1, 2026 19:01
Comment thread src/libexpr/paths.cc Outdated
@xokdvium xokdvium force-pushed the lazy-path-hash-mismatch-error branch from 43feca5 to 52d720c Compare June 1, 2026 19:23
@xokdvium xokdvium enabled auto-merge June 1, 2026 19:43
… proper error

This can happen when the local source tree (dirty git repo) gets modified
when the nix process is evaluating stuff from it. Make this a proper error
and not an assert. I've hit this issue myself once now.
@xokdvium xokdvium force-pushed the lazy-path-hash-mismatch-error branch from 52d720c to 8ffda08 Compare June 1, 2026 19:48
@xokdvium xokdvium added this pull request to the merge queue Jun 1, 2026
Merged via the queue into master with commit d212b92 Jun 1, 2026
20 checks passed
@xokdvium xokdvium deleted the lazy-path-hash-mismatch-error branch June 1, 2026 20:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants