nix ps: Check whether stdout is a tty

edolstra · edolstra · commit 72abd2a7e551 · 2025-12-04T22:26:14.000+01:00
diff --git a/src/nix/ps.cc b/src/nix/ps.cc
@@ -137,7 +137,7 @@ struct CmdPs : MixJSON, StoreCommand
             }
         }
 
-        auto width = isTTY() ? getWindowWidth() : std::numeric_limits<unsigned int>::max();
+        auto width = isTTY() && isatty(STDOUT_FILENO) ? getWindowWidth() : std::numeric_limits<unsigned int>::max();
 
         printTable(std::cout, table, width);
     }

Original file line number	Diff line number	Diff line change
`@@ -137,7 +137,7 @@ struct CmdPs : MixJSON, StoreCommand`
`137`	`137`	`}`
`138`	`138`	`}`
`139`	`139`
`140`		`- auto width = isTTY() ? getWindowWidth() : std::numeric_limits<unsigned int>::max();`
	`140`	`+ auto width = isTTY() && isatty(STDOUT_FILENO) ? getWindowWidth() : std::numeric_limits<unsigned int>::max();`
`141`	`141`
`142`	`142`	`printTable(std::cout, table, width);`
`143`	`143`	`}`