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 04/07/2023 13:28, Tvrtko Ursulin wrote:
> 
> 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.

# time ninja
[1/448] Generating lib/version.h with a custom command
fatal: not a git repository (or any of the parent directories): .git
[6/6] Generating docs/testplan/i915_tests.rst with a custom command

real    0m24.363s
user    0m6.530s
sys     0m20.968s

24 seconds.. I just changed one i915 test. :(

Regards,

Tvrtko

> 
>> 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