Merge pull request #3944 from harsh-kumar-patwa/fix-headless-save-images

maliberty · web-flow · commit ae6310e6200f · 2026-03-02T19:58:08.000Z
Skip save_images on headless machines without display
diff --git a/flow/scripts/final_report.tcl b/flow/scripts/final_report.tcl
@@ -64,6 +64,7 @@ report_cell_usage
 report_metrics 6 "finish"
 
 # Save a final image if openroad is compiled with the gui
-if { [ord::openroad_gui_compiled] } {
+# and a display is available (skip on headless machines)
+if { [ord::openroad_gui_compiled] && [env_var_exists_and_non_empty DISPLAY] } {
   gui::show "source $::env(SCRIPTS_DIR)/save_images.tcl" false
 }

Original file line number	Diff line number	Diff line change
`@@ -64,6 +64,7 @@ report_cell_usage`
`64`	`64`	`report_metrics 6 "finish"`
`65`	`65`
`66`	`66`	`# Save a final image if openroad is compiled with the gui`
`67`		`-if { [ord::openroad_gui_compiled] } {`
	`67`	`+# and a display is available (skip on headless machines)`
	`68`	`+if { [ord::openroad_gui_compiled] && [env_var_exists_and_non_empty DISPLAY] } {`
`68`	`69`	`gui::show "source $::env(SCRIPTS_DIR)/save_images.tcl" false`
`69`	`70`	`}`