Merge branch 'feature/flush-logs' into testing

sourcebots · Jul 10, 2024 · e05987f · e05987f
2 parents f63ac3e + 2cdb35d
commit e05987f
Showing 1 changed file with 10 additions and 0 deletions.
diff --git a/runusb/__main__.py b/runusb/__main__.py
@@ -280,6 +280,16 @@ def cleanup(self) -> None:
         except subprocess.TimeoutExpired:
             # The process did not exit after 5 seconds, so kill it.
             self._send_signal(signal.SIGKILL)
+
+        # Ensure logs have finished writing
+        self.log_thread.join()
+
+        # Explicitly close handler before removing it
+        self.handler.close()
+
+        # Sync filesystems before reporting status
+        os.sync()
+
         self._set_leds()
 
     def close(self) -> None: