[igt-dev] [PATCH i-g-t 6/7] runner/job_list: use case-insensitive regular expressions

Kamil Konieczny kamil.konieczny at linux.intel.com
Wed Oct 25 07:56:02 UTC 2023


Hi Mauro,
On 2023-10-24 at 16:11:09 +0200, Mauro Carvalho Chehab wrote:
> From: Mauro Carvalho Chehab <mchehab at kernel.org>
> 
> There are some KMS tests that are case-sensitive. Eventually,
> those could be blocklisted with a different case, causing troubles
> for CI runs.
> 
> As it makes no sense to have the same test name with different
> cases, handle regular expressions in a case-insensitive way.
> 
> Signed-off-by: Mauro Carvalho Chehab <mchehab at kernel.org>

Reviewed-by: Kamil Konieczny <kamil.konieczny at linux.intel.com>

> ---
>  runner/settings.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/runner/settings.c b/runner/settings.c
> index 788957948b4f..42d8137f18e9 100644
> --- a/runner/settings.c
> +++ b/runner/settings.c
> @@ -323,7 +323,7 @@ static bool add_regex(struct regex_list *list, char *new)
>  	GRegex *regex;
>  	GError *error = NULL;
>  
> -	regex = g_regex_new(new, G_REGEX_OPTIMIZE, 0, &error);
> +	regex = g_regex_new(new, G_REGEX_CASELESS | G_REGEX_OPTIMIZE, 0, &error);
>  	if (error) {
>  		usage(stderr, "Invalid regex '%s': %s", new, error->message);
>  		g_error_free(error);
> -- 
> 2.41.0
> 


More information about the igt-dev mailing list