From nobody Wed Jan 29 00:08:46 2025 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=pass(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1734052615; cv=none; d=zohomail.com; s=zohoarc; b=KM+wLY9KM5+NCNW70X8w+sCz7QHd3nhVf+wOc5lm6M6qvTsiWicUWWtaCimAbftQZ99fSoHNjzQCT3r0UeLq3wvlysQjxIenI1kkm+ZIIYhaor90Yd+WAR6neSvx4nbTFrpPngtD9l3XEr3RLnp927gNNZXuEbTyKs3Ht885i/c= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1734052615; h=Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:Subject:To:To:Message-Id:Reply-To; bh=MPtM0t9j3+VUBJU/haXesLXFHdebw1mTjI3ZE0OFdKY=; b=hMjXQfdX26LeLLrSCax049oUGmzAYccFiW1R8aWwwXj9s60voCcEP2yTZ8ixueaQ7KDmHRHSfnf4BFrGiuAj3L1EAr11WSwAg/TmfXyrBGwpQ9jVjT4R9bx/cVqPLmqb0FM4sO1aVSLJlfNYbLSLq/Oxen+EllB65355e9Sshxs= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=pass header.from= (p=none dis=none) Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1734052615594893.1199572838683; Thu, 12 Dec 2024 17:16:55 -0800 (PST) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1tLuGz-0000JS-FU; Thu, 12 Dec 2024 20:15:17 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1tLuGy-0000Du-6E for qemu-devel@nongnu.org; Thu, 12 Dec 2024 20:15:16 -0500 Received: from us-smtp-delivery-124.mimecast.com ([170.10.129.124]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1tLuGs-0000uz-Lu for qemu-devel@nongnu.org; Thu, 12 Dec 2024 20:15:15 -0500 Received: from mx-prod-mc-03.mail-002.prod.us-west-2.aws.redhat.com (ec2-54-186-198-63.us-west-2.compute.amazonaws.com [54.186.198.63]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-15-GtaoVxW-OPqGPgA96HvLKw-1; Thu, 12 Dec 2024 20:15:08 -0500 Received: from mx-prod-int-02.mail-002.prod.us-west-2.aws.redhat.com (mx-prod-int-02.mail-002.prod.us-west-2.aws.redhat.com [10.30.177.15]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by mx-prod-mc-03.mail-002.prod.us-west-2.aws.redhat.com (Postfix) with ESMTPS id 3C0CA1955F3A; Fri, 13 Dec 2024 01:15:07 +0000 (UTC) Received: from jsnow-thinkpadp16vgen1.westford.csb (unknown [10.22.88.22]) by mx-prod-int-02.mail-002.prod.us-west-2.aws.redhat.com (Postfix) with ESMTP id 43B891956089; Fri, 13 Dec 2024 01:15:05 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1734052510; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=MPtM0t9j3+VUBJU/haXesLXFHdebw1mTjI3ZE0OFdKY=; b=LAgH5ryN+nGPhFIWK3DQeJOygIPBOxcNE6gsMNj9M6M9tz8NsFCTU+i86WFyBb88EWrdf6 uYZDnBdM8IMmEN8nRgnuOf0JSfDB8qaUa2TjAXUhNiO90fQjzhKp8atQY+rtjL+u6uxlXH /A9gwgQSGKEjcwx4ZkzouMqonvAqGv8= X-MC-Unique: GtaoVxW-OPqGPgA96HvLKw-1 X-Mimecast-MFC-AGG-ID: GtaoVxW-OPqGPgA96HvLKw From: John Snow To: qemu-devel@nongnu.org Cc: Michael Roth , Markus Armbruster , Peter Maydell , =?UTF-8?q?Marc-Andr=C3=A9=20Lureau?= , Victor Toso de Carvalho , Paolo Bonzini , John Snow Subject: [RFC PATCH v2 30/35] docs/qapi-domain: implement error context reporting fix Date: Thu, 12 Dec 2024 20:12:59 -0500 Message-ID: <20241213011307.2942030-31-jsnow@redhat.com> In-Reply-To: <20241213011307.2942030-1-jsnow@redhat.com> References: <20241213011307.2942030-1-jsnow@redhat.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-Scanned-By: MIMEDefang 3.0 on 10.30.177.15 Received-SPF: pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Received-SPF: pass client-ip=170.10.129.124; envelope-from=jsnow@redhat.com; helo=us-smtp-delivery-124.mimecast.com X-Spam_score_int: -25 X-Spam_score: -2.6 X-Spam_bar: -- X-Spam_report: (-2.6 / 5.0 requ) BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.496, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H2=-0.001, RCVD_IN_VALIDITY_RPBL_BLOCKED=0.001, RCVD_IN_VALIDITY_SAFE_BLOCKED=0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: qemu-devel-bounces+importer=patchew.org@nongnu.org X-ZohoMail-DKIM: pass (identity @redhat.com) X-ZM-MESSAGEID: 1734052617441116600 Content-Type: text/plain; charset="utf-8" Sphinx 5.3.0 to Sphinx 6.2.0 has a bug where nested content in an ObjectDescription content block has its error position reported incorrectly due to an oversight when they added nested section support to this directive. (This bug is present in Sphinx's own Python and C domains; test it yourself by creating a py:func directive and creating a syntax error in the directive's content block.) To avoid overriding and re-implementing the entirety of the run() method, a workaround is employed where we parse the content block ourselves in before_content(), then null the content block to make Sphinx's own parsing a no-op. Then, in transform_content (which occurs after Sphinx's nested parse), we simply swap our own parsed content tree back in for Sphinx's. It appears a little tricky, but it's the nicest solution I can find. Signed-off-by: John Snow --- docs/sphinx/qapi-domain.py | 24 ++++++++++++++++++++++++ 1 file changed, 24 insertions(+) diff --git a/docs/sphinx/qapi-domain.py b/docs/sphinx/qapi-domain.py index 8dbf0ee5e70..7cbf12d93f7 100644 --- a/docs/sphinx/qapi-domain.py +++ b/docs/sphinx/qapi-domain.py @@ -21,8 +21,10 @@ =20 from docutils import nodes from docutils.parsers.rst import directives +from docutils.statemachine import StringList =20 from compat import keyword_node, nested_parse, space_node +import sphinx from sphinx import addnodes from sphinx.addnodes import desc_signature, pending_xref from sphinx.directives import ObjectDescription @@ -509,7 +511,29 @@ def _validate_field(self, field: nodes.field) -> None: ) logger.warning(msg, location=3Dfield) =20 + def before_content(self) -> None: + # Work around a sphinx bug and parse the content ourselves. + self._temp_content =3D self.content + self._temp_offset =3D self.content_offset + self._temp_node =3D None + + if (5, 3, 0) <=3D sphinx.version_info[:3] < (6, 2, 0): + self._temp_node =3D addnodes.desc_content() + self.state.nested_parse( + self.content, self.content_offset, self._temp_node + ) + # Sphinx will try to parse the content block itself, + # Give it nothingness to parse instead. + self.content =3D StringList() + self.content_offset =3D 0 + def transform_content(self, contentnode: addnodes.desc_content) -> Non= e: + # Sphinx workaround: Inject our parsed content and restore state. + if self._temp_node: + contentnode +=3D self._temp_node.children + self.content =3D self._temp_content + self.content_offset =3D self._temp_offset + self._add_infopips(contentnode) self._merge_adjoining_field_lists(contentnode) =20 --=20 2.47.0