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;
> +    }

Reply via email to