[PATCH v3 1/2] cli: introduce the concept of user defined hooks

Jani Nikula jani at nikula.org
Tue Dec 6 05:30:11 PST 2011


On Tue,  6 Dec 2011 15:22:37 +0200, Jani Nikula <jani at nikula.org> wrote:
> +    if (!WIFEXITED (status) || WEXITSTATUS (status)) {
> +	if (WEXITSTATUS (status)) {

Grrh, the above should be "if (WIFEXITED (status))". Please review
otherwise.

Jani.

> +	    fprintf (stderr, "Error: %s hook failed with status %d\n",
> +		     hook, WEXITSTATUS (status));
> +	} else if (WIFSIGNALED (status)) {
> +	    fprintf (stderr, "Error: %s hook terminated with signal %d\n",
> +		     hook, WTERMSIG (status));
> +	}
> +	status = 1;
> +    }


More information about the notmuch mailing list