Re: [igt-dev] [PATCH i-g-t] Revert "testplan/kms: Make documentation is mandatory for all kms subtests"

["Bernatowicz, Marcin" <marcin.bernatowicz@linux.intel.com>]

On 8/18/2023 1:38 PM, Juha-Pekka Heikkila wrote:
> This change broke igt build when building without chamelium
> 
> This reverts commit 00bc4cf384382018ebe6f333e4d53849c883445b.
> 
> Signed-off-by: Juha-Pekka Heikkila <juhapekka.heikkila@gmail.com>
> ---
>   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 433464dc8..e838f2eb1 100644
> --- a/docs/testplan/meson.build
> +++ b/docs/testplan/meson.build
> @@ -31,13 +31,13 @@ xe_test_dict = {
>   if build_xe
>   	test_dict = {
>   		'i915_tests': { 'input': i915_test_config, 'extra_args': check_testlist },
> -		'kms_tests': { 'input': kms_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': check_testlist },
> -	      'kms_tests': { 'input': kms_test_config, 'extra_args': check_testlist }
> +	      'kms_tests': { 'input': kms_test_config, 'extra_args': [] }
>   	    }
>   endif
>   
LGTM, verified build does not fail on docker with chamelium disabled.

Acked-by: Marcin Bernatowicz <marcin.bernatowicz@linux.intel.com>