bbjubjub2494/lix
0
0
Fork 0
forked from lix-project/lix
Code Pull requests Activity

Start the pager early-enough in nix why-depends

Browse source 
`nix why-depends` is piping its output into a pager by default.
However the pager was only started after the first path is printed,
causing it to be excluded from the pager output.

(Actually the pager was started *inside* the recursive function that was
printing the dependency chain, so a new instance was started at each
level. It’s a little miracle that it worked at all).

Fix #5911
This commit is contained in:
regnat 2022-01-14 13:41:45 +01:00
parent 8a446aff75
commit 81cd0a113b
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/nix/why-depends.cc
View file

@ -239,7 +239,6 @@ struct CmdWhyDepends : SourceExprCommand
visitPath(pathS); visitPath(pathS);
RunPager pager;
for (auto & ref : refs) { for (auto & ref : refs) {
std::string hash(ref.second->path.hashPart()); std::string hash(ref.second->path.hashPart());
@ -259,6 +258,7 @@ struct CmdWhyDepends : SourceExprCommand
} }
}; };
RunPager pager;
try { try {
printNode(graph.at(packagePath), "", ""); printNode(graph.at(packagePath), "", "");
} catch (BailOut & ) { } } catch (BailOut & ) { }