Re: [igt-dev] [PATCH i-g-t 2/2] testplan/meson.build: make it check for missing i915 documentation

[Tvrtko Ursulin <tvrtko.ursulin@linux.intel.com>]

On 26/05/2023 07:46, Mauro Carvalho Chehab wrote:
> From: Mauro Carvalho Chehab <mchehab@kernel.org>
> 
> Now that i915 is fully documented, check it at build time.

This step seems to be slow as molasses and it also rebuilds the Xe test 
plan when I touch an i915 test.

What is the way to disable it all when configuring the build?

Regards,

Tvrtko

P.S. I also find the "now that i915 is fully documented" statement a bit 
of a chuckle, since random two tests I happened to open haven't really 
been documented - it rather looks to be a bit of a charade.

I wouldn't care really apart from it significantly slowing down the 
development workflow.

> Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
> ---
>   docs/testplan/meson.build | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/docs/testplan/meson.build b/docs/testplan/meson.build
> index 6cd44be5ccd3..bf73dd32a15f 100644
> --- a/docs/testplan/meson.build
> +++ b/docs/testplan/meson.build
> @@ -30,13 +30,13 @@ xe_test_dict = {
>   
>   if build_xe
>   	test_dict = {
> -		'i915_tests': { 'input': i915_test_config, 'extra_args': [] },
> +		'i915_tests': { 'input': i915_test_config, 'extra_args': check_testlist },
>   		'kms_tests': { 'input': kms_test_config, 'extra_args': [] },
>   		'xe_tests': { 'input': xe_test_config, 'extra_args': check_testlist }
>   	    }
>   else
>   	test_dict = {
> -	      'i915_tests': { 'input': i915_test_config, 'extra_args': [] },
> +	      'i915_tests': { 'input': i915_test_config, 'extra_args': check_testlist },
>   	      'kms_tests': { 'input': kms_test_config, 'extra_args': [] }
>   	    }
>   endif