On 27/01/2017 17:30, Emilio Pozuelo Monfort wrote:
From: Pekka Paalanen <pekka.paala...@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.paala...@collabora.co.uk>
Reviewed-by: Emilio Pozuelo Monfort <emilio.pozu...@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+...@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
_______________________________________________
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/wayland-devel

Reply via email to