On Tue, Oct 24, 2017 at 12:53:06PM +0200, Lukasz Fiedorowicz wrote:
> Lists in intel-ci directory are kept in all lower case but the subtest
> names are mix of lower and upper case.
> Piglit is able to handle this but not every CI is using piglit. Changing
> condition to ignore subtest names case.


This fails to match --run-subtest "*pipe-b" to foo-bar-pipe-B


-- 
Petri Latvala



> 
> Signed-off-by: Lukasz Fiedorowicz <[email protected]>
> ---
>  lib/igt_core.c | 7 ++++---
>  1 file changed, 4 insertions(+), 3 deletions(-)
> 
> diff --git a/lib/igt_core.c b/lib/igt_core.c
> index 538a447..743e82b 100644
> --- a/lib/igt_core.c
> +++ b/lib/igt_core.c
> @@ -965,10 +965,11 @@ bool __igt_run_subtest(const char *subtest_name)
>       }
>  
>       if (run_single_subtest) {
> -             if (uwildmat(subtest_name, run_single_subtest) == 0)
> +             if (uwildmat(subtest_name, run_single_subtest)
> +        || strcasecmp(subtest_name, run_single_subtest) == 0)
> +        run_single_subtest_found = true;
> +             else
>                       return false;
> -             else
> -                     run_single_subtest_found = true;
>       }
>  
>       if (skip_subtests_henceforth) {
> -- 
> 2.9.5
> 
> _______________________________________________
> Intel-gfx mailing list
> [email protected]
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
_______________________________________________
Intel-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to