Skip to content

libstore: decide how to build in one spot#15104

Merged
Ericson2314 merged 1 commit intoNixOS:masterfrom
obsidiansystems:refactor-buildlocally
Jan 28, 2026
Merged

libstore: decide how to build in one spot#15104
Ericson2314 merged 1 commit intoNixOS:masterfrom
obsidiansystems:refactor-buildlocally

Conversation

@amaanq
Copy link
Member

@amaanq amaanq commented Jan 27, 2026

Motivation

This cleans up the logic for checking if the worker's store is a valid local store when we're not hooking it. If we have a local store, we then pass that as an argument to DerivationBuildingGoal::buildLocally, rather than checking inside the function itself.


Add 👍 to pull requests you find important.

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

This cleans up the logic for checking if the worker's store is a valid
local store when we're not hooking it. If we have a local store, we then
pass that as an argument to `DerivationBuildingGoal::buildLocally`,
rather than checking inside the function itself.
@amaanq amaanq requested a review from Ericson2314 as a code owner January 27, 2026 23:35
@amaanq amaanq changed the title architecture overview: add link to Nix expression language reference libstore: decide how to build in one spot Jan 27, 2026
@Ericson2314 Ericson2314 enabled auto-merge January 27, 2026 23:38
@Ericson2314 Ericson2314 added this pull request to the merge queue Jan 28, 2026
Merged via the queue into NixOS:master with commit d5e4b0b Jan 28, 2026
15 checks passed
@Ericson2314 Ericson2314 deleted the refactor-buildlocally branch January 28, 2026 01:03
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.

2 participants