ActivityWatch / aw-qt

Tray icon that manages ActivityWatch processes, built with Qt.
Mozilla Public License 2.0
27 stars 29 forks source link

switched to using atexit instead of handling signals #21

Closed ErikBjare closed 6 years ago

johan-bjareholt commented 7 years ago

With this the modules do not even exit properly with SIGINT (ctrl+c) and doesn't handle when it crashes any better.

ErikBjare commented 7 years ago

Oh, that sucks. I thought atexit was a neat way to manage cases like this.

Edit: Someone on SO apparently knew this

Do you think try: ...; finally: ... would work?

Alternatively we could just make all signals call sys.exit() which would trigger the atexit.

ErikBjare commented 6 years ago

Won't merge, closing.