[lttng-dev] [PATCH babeltrace] Fix: remove g_spawn_check_exit_status to support glib 2.22

Jérémie Galarneau jeremie.galarneau at efficios.com
Thu Aug 31 20:54:19 UTC 2017


Merged, thanks!

Jérémie

On 30 August 2017 at 14:54, Michael Jeanson <mjeanson at efficios.com> wrote:
> Signed-off-by: Michael Jeanson <mjeanson at efficios.com>
> ---
>  tests/lib/test_ctf_writer.c | 10 ++++++++--
>  1 file changed, 8 insertions(+), 2 deletions(-)
>
> diff --git a/tests/lib/test_ctf_writer.c b/tests/lib/test_ctf_writer.c
> index 3d24ea4..9c65d84 100644
> --- a/tests/lib/test_ctf_writer.c
> +++ b/tests/lib/test_ctf_writer.c
> @@ -122,10 +122,16 @@ void validate_trace(char *parser_path, char *trace_path)
>                 goto result;
>         }
>
> -       if(!g_spawn_check_exit_status(exit_status, NULL)) {
> +       /* Replace by g_spawn_check_exit_status when we require glib >= 2.34 */
> +#if G_OS_UNIX
> +       ret = WIFEXITED(exit_status) ? WEXITSTATUS(exit_status) : -1;
> +#else
> +       ret = exit_status;
> +#endif
> +
> +       if(ret != 0) {
>                 diag("Babeltrace returned an error.");
>                 diag_multiline(standard_error);
> -               ret = -1;
>                 goto result;
>         }
>
> --
> 2.7.4
>



-- 
Jérémie Galarneau
EfficiOS Inc.
http://www.efficios.com


More information about the lttng-dev mailing list