[PATCH 2/3] bus: add --address switch

Colin Walters walters at verbum.org
Wed Jul 7 13:24:45 PDT 2010


On Wed, Jun 23, 2010 at 12:10 PM, Lennart Poettering <mzqohf at 0pointer.de> wrote:
>
> @@ -473,6 +513,9 @@ main (int argc, char **argv)
>  #ifdef SIGHUP
>   _dbus_set_signal_handler (SIGHUP, signal_handler);
>  #endif
> +#ifdef SIGTERM
> +  _dbus_set_signal_handler (SIGTERM, signal_handler);
> +#endif

This looks like a bad commit split; we should do clean shutdown on
SIGTERM, but you didn't change the implementation of signal_handler.

Otherwise the patch looks fine, please commit.


More information about the dbus mailing list