mirror of
https://github.com/NixOS/nix.git
synced 2025-11-08 19:46:02 +01:00
Revert "Add a crude tracing mechansim for the build results"
The commit says it was added for CA testing --- manual I assume, since
there is no use of this in the test suite. I don't think we need it any
more, and I am not sure whether it was ever supposed to have made it to
`master` either.
This reverts commit 2eec2f765a.
This commit is contained in:
parent
35978ca47b
commit
0590b13156
3 changed files with 0 additions and 55 deletions
|
|
@ -1368,13 +1368,6 @@ DerivationBuildingGoal::done(BuildResult::Status status, SingleDrvOutputs builtO
|
||||||
|
|
||||||
worker.updateProgress();
|
worker.updateProgress();
|
||||||
|
|
||||||
auto traceBuiltOutputsFile = getEnv("_NIX_TRACE_BUILT_OUTPUTS").value_or("");
|
|
||||||
if (traceBuiltOutputsFile != "") {
|
|
||||||
std::fstream fs;
|
|
||||||
fs.open(traceBuiltOutputsFile, std::fstream::out);
|
|
||||||
fs << worker.store.printStorePath(drvPath) << "\t" << buildResult.toString() << std::endl;
|
|
||||||
}
|
|
||||||
|
|
||||||
return amDone(buildResult.success() ? ecSuccess : ecFailed, std::move(ex));
|
return amDone(buildResult.success() ? ecSuccess : ecFailed, std::move(ex));
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
||||||
|
|
@ -364,13 +364,6 @@ DerivationGoal::done(BuildResult::Status status, std::optional<Realisation> buil
|
||||||
|
|
||||||
worker.updateProgress();
|
worker.updateProgress();
|
||||||
|
|
||||||
auto traceBuiltOutputsFile = getEnv("_NIX_TRACE_BUILT_OUTPUTS").value_or("");
|
|
||||||
if (traceBuiltOutputsFile != "") {
|
|
||||||
std::fstream fs;
|
|
||||||
fs.open(traceBuiltOutputsFile, std::fstream::out);
|
|
||||||
fs << worker.store.printStorePath(drvPath) << "\t" << buildResult.toString() << std::endl;
|
|
||||||
}
|
|
||||||
|
|
||||||
return amDone(buildResult.success() ? ecSuccess : ecFailed, std::move(ex));
|
return amDone(buildResult.success() ? ecSuccess : ecFailed, std::move(ex));
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
||||||
|
|
@ -46,47 +46,6 @@ struct BuildResult
|
||||||
*/
|
*/
|
||||||
std::string errorMsg;
|
std::string errorMsg;
|
||||||
|
|
||||||
std::string toString() const
|
|
||||||
{
|
|
||||||
auto strStatus = [&]() {
|
|
||||||
switch (status) {
|
|
||||||
case Built:
|
|
||||||
return "Built";
|
|
||||||
case Substituted:
|
|
||||||
return "Substituted";
|
|
||||||
case AlreadyValid:
|
|
||||||
return "AlreadyValid";
|
|
||||||
case PermanentFailure:
|
|
||||||
return "PermanentFailure";
|
|
||||||
case InputRejected:
|
|
||||||
return "InputRejected";
|
|
||||||
case OutputRejected:
|
|
||||||
return "OutputRejected";
|
|
||||||
case TransientFailure:
|
|
||||||
return "TransientFailure";
|
|
||||||
case CachedFailure:
|
|
||||||
return "CachedFailure";
|
|
||||||
case TimedOut:
|
|
||||||
return "TimedOut";
|
|
||||||
case MiscFailure:
|
|
||||||
return "MiscFailure";
|
|
||||||
case DependencyFailed:
|
|
||||||
return "DependencyFailed";
|
|
||||||
case LogLimitExceeded:
|
|
||||||
return "LogLimitExceeded";
|
|
||||||
case NotDeterministic:
|
|
||||||
return "NotDeterministic";
|
|
||||||
case ResolvesToAlreadyValid:
|
|
||||||
return "ResolvesToAlreadyValid";
|
|
||||||
case NoSubstituters:
|
|
||||||
return "NoSubstituters";
|
|
||||||
default:
|
|
||||||
return "Unknown";
|
|
||||||
};
|
|
||||||
}();
|
|
||||||
return strStatus + ((errorMsg == "") ? "" : " : " + errorMsg);
|
|
||||||
}
|
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* How many times this build was performed.
|
* How many times this build was performed.
|
||||||
*/
|
*/
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue