[Github-comments] [geany/geany] Re-enable SIGTERM handling (#1255)

Matthew Brush notifications at xxxxx
Wed Oct 5 16:51:27 UTC 2016


Now that a new enough GLib is available the signal can be handled cleanly on the main loop using the GSource for Unix signals. This replaces the illegal SIGTERM handling that was disabled in
fbb89f523af47b35e238678d348cfa98e56c760a.

In the future, we might also like to handle `SIGPIPE` which was ignored in cc511a78d80fde742e1794c32b2738ae8ddd4792 to log the error instead of just ignoring it (or handling `write()` failures at their source).
You can view, comment on, or merge this pull request online at:

  https://github.com/geany/geany/pull/1255

-- Commit Summary --

  * Re-enable SIGTERM handling

-- File Changes --

    M src/libmain.c (21)

-- Patch Links --

https://github.com/geany/geany/pull/1255.patch
https://github.com/geany/geany/pull/1255.diff

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/1255
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20161005/994169b9/attachment.html>


More information about the Github-comments mailing list