[PATCH wayland v3 1/4] Support running tests from different build directories
Peter Hutterer
peter.hutterer at who-t.net
Fri Jul 27 05:20:06 UTC 2018
On Sat, Jul 21, 2018 at 04:31:21PM +0100, Daniel Stone wrote:
> From: Emmanuele Bassi <ebassi at gnome.org>
>
> The tests that run exec-fd-leak-checker expect the binary to be located
> in the current directory. This is not always the case; for instance, the
> binaries could be built under `tests`, but be invoked under the
> top-level build directory.
>
> We can use an environment variable to control what's the location of the
> test binaries, and fall back to the current directory if the variable is
> unset.
>
> Reviewed-by: Daniel Stone <daniels at collabora.com>
> ---
> tests/test-helpers.c | 13 +++++++++++--
> 1 file changed, 11 insertions(+), 2 deletions(-)
>
> diff --git a/tests/test-helpers.c b/tests/test-helpers.c
> index b2189d8e..8ad332b1 100644
> --- a/tests/test-helpers.c
> +++ b/tests/test-helpers.c
> @@ -29,6 +29,7 @@
> #include <errno.h>
> #include <dirent.h>
> #include <stdio.h>
> +#include <stdlib.h>
> #include <unistd.h>
> #include <time.h>
> #include <sys/time.h>
> @@ -67,11 +68,19 @@ count_open_fds(void)
> void
> exec_fd_leak_check(int nr_expected_fds)
> {
> - const char *exe = "./exec-fd-leak-checker";
> + const char *exe = "exec-fd-leak-checker";
> char number[16] = { 0 };
> + const char *test_build_dir = getenv("TEST_BUILD_DIR");
> + char exe_path[256] = { 0 };
> +
> + if (test_build_dir == NULL || test_build_dir[0] == 0) {
> + test_build_dir = ".";
> + }
> +
> + snprintf(exe_path, sizeof exe_path - 1, "%s/%s", test_build_dir, exe);
indentation switched to spaces instead of tabs here.
Cheers,
Peter
>
> snprintf(number, sizeof number - 1, "%d", nr_expected_fds);
> - execl(exe, exe, number, (char *)NULL);
> + execl(exe_path, exe, number, (char *)NULL);
> assert(0 && "execing fd leak checker failed");
> }
>
> --
> 2.17.1
>
> _______________________________________________
> wayland-devel mailing list
> wayland-devel at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/wayland-devel
>
More information about the wayland-devel
mailing list