From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 24B83C4829E for ; Fri, 9 Feb 2024 11:51:16 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id B325810F34F; Fri, 9 Feb 2024 11:51:15 +0000 (UTC) Authentication-Results: gabe.freedesktop.org; dkim=fail reason="signature verification failed" (2048-bit key; unprotected) header.d=intel.com header.i=@intel.com header.b="gvbk8v8h"; dkim-atps=neutral Received: from mgamail.intel.com (mgamail.intel.com [192.198.163.18]) by gabe.freedesktop.org (Postfix) with ESMTPS id 35F7310EB72 for ; Fri, 9 Feb 2024 11:51:11 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1707479471; x=1739015471; h=from:to:subject:date:message-id:in-reply-to:references: mime-version:content-transfer-encoding:sender; bh=1wplqa5gNvXwnuQ6hOH5G3h7o9C7Gd/ZRawPGgzaiXk=; b=gvbk8v8hahJAm3/YsqiX4jb0t3CFcEk6gCn4KP812ZXzggTjGC2kO4jS blb86EUildGilkexupCKuF1YzLR/RJPT4wKAkxwLstY7qD7U9gy90jQWn VvAKNl8GumEUZzxTvC1Pz5maItSlxfJXysLH6csO3CrG6N9OM/K6zQBIo 4yY63wXTvz8jWe+5kWmcfiRjVKzCKFyjWo0n9lbFTMSgMr4CNJKdnet0U B5tz9IRMGf7OnpYeCtlnRamvyKmioICxSBju7oUoyMtOs7eEysLVxKF+2 DYG1Ae6htnMGmh0MOQEOxNzZG3Yu48UvU1gGmeMxtJkDREf6DQSfLMmNw A==; X-IronPort-AV: E=McAfee;i="6600,9927,10978"; a="1294069" X-IronPort-AV: E=Sophos;i="6.05,256,1701158400"; d="scan'208";a="1294069" Received: from orsmga001.jf.intel.com ([10.7.209.18]) by fmvoesa112.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 09 Feb 2024 03:51:07 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6600,9927,10978"; a="825116727" X-IronPort-AV: E=Sophos;i="6.05,256,1701158400"; d="scan'208";a="825116727" Received: from linux.intel.com ([10.54.29.200]) by orsmga001.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 09 Feb 2024 03:51:07 -0800 Received: from linux.intel.com (maurocar-mobl2.ger.corp.intel.com [10.245.246.44]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by linux.intel.com (Postfix) with ESMTPS id E4F31580DE9 for ; Fri, 9 Feb 2024 03:51:06 -0800 (PST) Received: from maurocar by linux.intel.com with local (Exim 4.97.1) (envelope-from ) id 1rYPPM-00000001CFc-3VUC for igt-dev@lists.freedesktop.org; Fri, 09 Feb 2024 12:51:04 +0100 From: Mauro Carvalho Chehab To: igt-dev@lists.freedesktop.org Subject: [PATCH i-g-t v2 2/8] scripts/igt_doc.py: better handle issues with empty testlists Date: Fri, 9 Feb 2024 12:49:51 +0100 Message-ID: <20240209115101.285319-3-mauro.chehab@linux.intel.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20240209115101.285319-1-mauro.chehab@linux.intel.com> References: <20240209115101.285319-1-mauro.chehab@linux.intel.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-BeenThere: igt-dev@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Development mailing list for IGT GPU Tools List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" From: Mauro Carvalho Chehab The warning message of not creating an empty testlist is harmless, and it should only be used when not all GPUs are listed. So, change the logic to better check for "other" testlists before printing a message. Signed-off-by: Mauro Carvalho Chehab --- scripts/igt_doc.py | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/scripts/igt_doc.py b/scripts/igt_doc.py index 6a1cfc2b5a56..a6cbeee067fe 100755 --- a/scripts/igt_doc.py +++ b/scripts/igt_doc.py @@ -218,16 +218,14 @@ class IntelciTestlist: for testlist, subtests in names.items(): if testlist == "": - if not subtests: - continue - testlist = "other" else: testlist = re.sub(r"[\W_]+", "-", testlist).lower() testlist = re.sub(r"_+", "_", testlist) if not subtests: - sys.stderr.write(f"Warning: empty testlist: {testlist}\n") + if testlist != "other": + sys.stderr.write(f"Not creating empty testlist: {dname}/{testlist}\n") continue fname = os.path.join(dname, testlist) + ".testlist" -- 2.43.0