-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
nix store repair --keep-going
#11877
base: master
Are you sure you want to change the base?
Conversation
So that we can recognize it in an upcoming commit.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
// Remove duplicates. | ||
StorePathSet storePathsSet; | ||
for (auto & path : storePaths) | ||
store->repairPath(path); | ||
storePathsSet.insert(path); | ||
|
||
for (auto & path : storePathsSet) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be bottom-up, ie dependencies first. That way when we do a "substitution fallback" build, we don't use broken inputs.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
At first glance, this looks okay to me.
Motivation
Fix what we can.
Tested in anger before adding
RepairFailure
.TODO
Context
Priorities and Process
Add 👍 to pull requests you find important.
The Nix maintainer team uses a GitHub project board to schedule and track reviews.