[lttng-dev] [PATCH babeltrace stable-1.5] Fix: ctf writer test on Cygwin
Michael Jeanson
mjeanson at efficios.com
Mon Apr 10 22:18:29 UTC 2017
On 2017-04-06 15:00, Michael Jeanson wrote:
> In the ctf writer test we execl() some libtool wrappers that get confused
> when arg[0] is not the full path to the binary.
>
> Signed-off-by: Michael Jeanson <mjeanson at efficios.com>
> ---
> tests/lib/test_ctf_writer.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/tests/lib/test_ctf_writer.c b/tests/lib/test_ctf_writer.c
> index 961df79..53613c9 100644
> --- a/tests/lib/test_ctf_writer.c
> +++ b/tests/lib/test_ctf_writer.c
> @@ -94,7 +94,7 @@ void validate_metadata(char *parser_path, char *metadata_path)
> goto result;
> }
>
> - execl(parser_path, "ctf-parser-test", (char *) NULL);
> + execl(parser_path, parser_path, (char *) NULL);
> perror("# Could not launch the ctf metadata parser process");
> exit(-1);
> }
> @@ -201,7 +201,7 @@ void validate_trace(char *parser_path, char *trace_path)
> goto result;
> }
>
> - execl(parser_path, "babeltrace", trace_path, (char *) NULL);
> + execl(parser_path, parser_path, trace_path, (char *) NULL);
> perror("# Could not launch the babeltrace process");
> exit(-1);
> }
>
You can hold on merging this patch for now, I think it's only useful for
a very specific environment which might not be worth supporting.
Michael
More information about the lttng-dev
mailing list