[PATCH weston 5/7] tests: put screenshots to ./logs by default

[Quentin Glidic]

On 27/01/2017 17:30, Emilio Pozuelo Monfort wrote:
> From: Pekka Paalanen <pekka.paalanen at collabora.co.uk>
> 
> Logs is where we write all our custom test logs, let's also put the
> screenshots in the same place by default from cluttering the base
> directory.
> 
> Signed-off-by: Pekka Paalanen <pekka.paalanen at collabora.co.uk>
> Reviewed-by: Emilio Pozuelo Monfort <emilio.pozuelo at collabora.co.uk>

Not sure I like "logs" as a path for temp screenshot, but still better 
than cluttering the base dir, indeed. Until we change it to a betterer 
place:
Reviewed-by: Quentin Glidic <sardemff7+git at sardemff7.net>

Thanks,


> ---
>   tests/weston-test-client-helper.c | 5 +++--
>   1 file changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/tests/weston-test-client-helper.c b/tests/weston-test-client-helper.c
> index fd6ebaf8..dd411925 100644
> --- a/tests/weston-test-client-helper.c
> +++ b/tests/weston-test-client-helper.c
> @@ -932,9 +932,10 @@ output_path(void)
>   	char *path = getenv("WESTON_TEST_OUTPUT_PATH");
>   
>   	if (!path)
> -		return ".";
> +		return "./logs";
> +
>   	return path;
> -	}
> +}
>   
>   char*
>   screenshot_output_filename(const char *basename, uint32_t seq)
> 


-- 

Quentin “Sardem FF7” Glidic