From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by gabe.freedesktop.org (Postfix) with ESMTPS id E01F910E164 for ; Tue, 4 Jul 2023 12:28:38 +0000 (UTC) Message-ID: <843966cc-f42a-5ec8-34d5-80cb832b8ee1@linux.intel.com> Date: Tue, 4 Jul 2023 13:28:34 +0100 MIME-Version: 1.0 Content-Language: en-US References: <20230526064624.2886063-1-mauro.chehab@linux.intel.com> <20230526064624.2886063-3-mauro.chehab@linux.intel.com> From: Tvrtko Ursulin In-Reply-To: <20230526064624.2886063-3-mauro.chehab@linux.intel.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [igt-dev] [PATCH i-g-t 2/2] testplan/meson.build: make it check for missing i915 documentation List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" To: Mauro Carvalho Chehab , igt-dev@lists.freedesktop.org List-ID: On 26/05/2023 07:46, Mauro Carvalho Chehab wrote: > From: Mauro Carvalho Chehab > > 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 > --- > 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