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 <[email protected]>
> ---
>  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
_______________________________________________
lttng-dev mailing list
[email protected]
https://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev

Reply via email to