Skip save_images on headless machines without display

harsh-kumar-patwa · harsh-kumar-patwa · commit 49167a7adaad · 2026-03-02T23:31:11.000+05:30
Resolves #3034 final_report.tcl only checks if OpenROAD was compiled with GUI support before calling gui::show to run save_images.tcl. On headless machines without a display, this causes a fatal crash (std::runtime_error abort from Qt platform plugin initialization). Add a DISPLAY environment check alongside the existing GUI compilation check so image generation is gracefully skipped on headless systems. Signed-off-by: Harsh Kumar Patwa <harshkumarpatwa@gmail.com> Signed-off-by: Harsh Kumar <harshkumar3446@gmail.com>
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`	`}`