From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 74FFA45D54; Wed, 20 Nov 2024 16:54:47 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 1C99A40285; Wed, 20 Nov 2024 16:54:47 +0100 (CET) Received: from fhigh-b7-smtp.messagingengine.com (fhigh-b7-smtp.messagingengine.com [202.12.124.158]) by mails.dpdk.org (Postfix) with ESMTP id C4D784021E for ; Wed, 20 Nov 2024 16:54:45 +0100 (CET) Received: from phl-compute-06.internal (phl-compute-06.phl.internal [10.202.2.46]) by mailfhigh.stl.internal (Postfix) with ESMTP id D0492254014F; Wed, 20 Nov 2024 10:54:44 -0500 (EST) Received: from phl-mailfrontend-01 ([10.202.2.162]) by phl-compute-06.internal (MEProxy); Wed, 20 Nov 2024 10:54:45 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=monjalon.net; h= cc:cc:content-transfer-encoding:content-type:content-type:date :date:from:from:in-reply-to:in-reply-to:message-id:mime-version :references:reply-to:subject:subject:to:to; s=fm1; t=1732118084; x=1732204484; bh=LXXOSTDQ40f0WQsVRBCCV4Ze63tUM1NAO6I9D7bJW14=; b= POmzSep4/bs7mGeiO/8m/r2EJQfxOCInZV/1mv8iq1r/HLcGABLsen1C/GyoQpZa PweFN5sSsjJqgwZk0fwlXZ/897PO8P8vPfccwbFTEHcqxnaEM9yFwE38TC08Mwio AhOvYc48ixIMEYNLc3FJSUEnKQ1x9Xl9tAviXvfT9sEypN1xFbLS5speK3qS4+cd dkzwjqTKjXSzOIBf710q9lvqdWGcRw9j2THoe3D5eHQTVZwRCFsYAAgEClfZznje NDuW3+/j7+/AszOMm86/e4mQl6oOSpDREUu/BLbV3/waKgQThDvOWKx6b68z21an JhJ/ozM9zMG9GHj3G6FvkA== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:cc:content-transfer-encoding :content-type:content-type:date:date:feedback-id:feedback-id :from:from:in-reply-to:in-reply-to:message-id:mime-version :references:reply-to:subject:subject:to:to:x-me-proxy :x-me-sender:x-me-sender:x-sasl-enc; s=fm1; t=1732118084; x= 1732204484; bh=LXXOSTDQ40f0WQsVRBCCV4Ze63tUM1NAO6I9D7bJW14=; b=A gHs1kyldbLVXzVvnY+weY7LY2BCR6UjioPvtLIpMdD9uhOs0zpzAcg0WKbttGfyL Ag0XmRm1/A3MAch81h1OXKSoHkyQo9KRXTIv+U9fu7LV7Eh7RIQDs6amTTv+A1RU N0+xnMtVtMyYnKpPb2FVNXpZTbcraCizCzgc5p/RhhK9dWVT9R3Tl78olR289OOF rJT2sIRsquJN8GA81jwX/J5AowtxrYrxusSqrmF4gPHhnIvmy539CkXQZDsqtPvN aXI7WKKr5IhkMgHjOgp8DVt9+0JCiL7p7FWPsWnTb7cabEZT6BDcO8YbwEWNpqA4 k42b/cFYwT3fDuy/SGDCw== X-ME-Sender: X-ME-Received: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgeefuddrfeeggdekudcutefuodetggdotefrodftvf curfhrohhfihhlvgemucfhrghsthforghilhdpggftfghnshhusghstghrihgsvgdpuffr tefokffrpgfnqfghnecuuegrihhlohhuthemuceftddtnecusecvtfgvtghiphhivghnth hsucdlqddutddtmdenucfjughrpefhvfevufffkfgjfhgggfgtsehtufertddttdejnecu hfhrohhmpefvhhhomhgrshcuofhonhhjrghlohhnuceothhhohhmrghssehmohhnjhgrlh honhdrnhgvtheqnecuggftrfgrthhtvghrnhepjeduveehieevuddutdevfffgtdegkeeu veejffejgedtgeegkefgvdeugfefkeejnecuvehluhhsthgvrhfuihiivgeptdenucfrrg hrrghmpehmrghilhhfrhhomhepthhhohhmrghssehmohhnjhgrlhhonhdrnhgvthdpnhgs pghrtghpthhtohepgedpmhhouggvpehsmhhtphhouhhtpdhrtghpthhtoheplhhutggrrd hvihiiiigrrhhrohesrghrmhdrtghomhdprhgtphhtthhopeguvghvseguphgukhdrohhr ghdprhgtphhtthhopehprghulhdrshiitgiivghprghnvghksegrrhhmrdgtohhmpdhrtg hpthhtohepphhrohgssgesihholhdruhhnhhdrvgguuh X-ME-Proxy: Feedback-ID: i47234305:Fastmail Received: by mail.messagingengine.com (Postfix) with ESMTPA; Wed, 20 Nov 2024 10:54:43 -0500 (EST) From: Thomas Monjalon To: Luca Vizzarro Cc: dev@dpdk.org, Paul Szczepanek , Patrick Robb Subject: Re: [PATCH] dts: fix API doc build warnings Date: Wed, 20 Nov 2024 16:54:42 +0100 Message-ID: <2371296.bcXerOTE6V@thomas> In-Reply-To: <5896563.1B3tZ46Xf9@thomas> References: <20241120144101.1826555-1-luca.vizzarro@arm.com> <5896563.1B3tZ46Xf9@thomas> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="utf-8" X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org 20/11/2024 16:05, Thomas Monjalon: > 20/11/2024 15:41, Luca Vizzarro: > > Sphinx-autodoc issues warnings about some symbols due to bad autodoc > > "automock"ing when optional dependencies are missing. > > > > The current meson setup allows to treat warnings as errors, halting the > > process and exiting early. Given these warnings should be ignored, this > > patch removes this behavior. > > So the patch is not fixing warnings, it allows them. > I will change the title. > > > Until optional dependencies are enforced, > > warnings for DTS API docs will be checked manually as part of the DTS > > approval process. > > No, optional dependencies should never be enforced. > > [...] > > -if get_option('werror') > > - extra_sphinx_args += '-W' > > -endif It makes the CI issue disappear. Applied with this modified message, thanks for fixing quickly: doc: allow warnings in Sphinx for DTS Sphinx-autodoc issues warnings about some symbols due to bad autodoc "automock"ing when optional dependencies are missing: Warning, treated as error: Failed to get a method signature for framework.config.TestSuiteConfig.convert_from_string: )> is not a callable object The current meson setup allows to treat warnings as errors, halting the process and exiting early. Given these warnings should be ignored, this patch removes this behavior. Until optional dependencies are detected, warnings for DTS API docs will be checked manually as part of the DTS approval process. Fixes: 6597fa4a30ad ("dts: add autodoc Pydantic") Signed-off-by: Luca Vizzarro