From patchwork Mon May 15 14:06:26 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Trevor Gamblin X-Patchwork-Id: 23964 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 aws-us-west-2-korg-lkml-1.web.codeaurora.org (localhost.localdomain [127.0.0.1]) by smtp.lore.kernel.org (Postfix) with ESMTP id F1541C77B7D for ; Mon, 15 May 2023 14:06:32 +0000 (UTC) Received: from mail-yb1-f182.google.com (mail-yb1-f182.google.com [209.85.219.182]) by mx.groups.io with SMTP id smtpd.web10.89560.1684159590078009024 for ; Mon, 15 May 2023 07:06:30 -0700 Authentication-Results: mx.groups.io; dkim=fail reason="signature has expired" header.i=@baylibre-com.20221208.gappssmtp.com header.s=20221208 header.b=DpaCOVLL; spf=pass (domain: baylibre.com, ip: 209.85.219.182, mailfrom: tgamblin@baylibre.com) Received: by mail-yb1-f182.google.com with SMTP id 3f1490d57ef6-ba67d34350aso8203600276.3 for ; Mon, 15 May 2023 07:06:29 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=baylibre-com.20221208.gappssmtp.com; s=20221208; t=1684159589; x=1686751589; h=content-transfer-encoding:mime-version:message-id:date:subject:to :from:from:to:cc:subject:date:message-id:reply-to; bh=10q/o3NOgfV48qf9TV4fAlQTYfgbaWEMNIVtfG2ilwo=; b=DpaCOVLL7WtH7W4T4Ikz8gccduYHJ8Aqx1Jv0/DG4iHHbYIkl0XCOE0rQmxBNl/FY2 resMzveu9oaYSUHBff6kZN1u68ag1//NFIByYFP2ZVJEzzz6UYARH6+JLdpC9wvgL1fr 0vc3kqDsx9oKSyvlwgM9JeZGWweG2fsRN94na7CJ4eXbxkjyaTuXnBiCDuz5aiTqIn92 PUHrzHZa2mvmVOAUURqO8GplcO1zL4wuZWQ3D6w9oqEPxz4lAZ+VaDkqQKo7XM3x26H+ 8LOZQciH5rr51u3yWwJOToSu//+SLBnWb3WqeXs9dZQhnD/V+iZTVp1PqnX6WSYstH6G Z4YQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1684159589; x=1686751589; h=content-transfer-encoding:mime-version:message-id:date:subject:to :from:x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=10q/o3NOgfV48qf9TV4fAlQTYfgbaWEMNIVtfG2ilwo=; b=kPO/DBwtaIQnKwQlo5mwCQu6h4XOWfQQh598L2tz7fiCSVjz9ddbXwnN4l0OGmmH7A 6E55LfqwPk3xaaj2zzIGuoIkMMTImqYZ+66fvp6uhceSSNOMzdy3jUfBIHUc2eUJ64UH nThT6oH6kbq70MQZuDhhdzUq8yJRI5r9izP6+SaijVRvjDSliOEhndc8GPn0GQ8CvIRa eGvLResGwgLoHMMXIbnR5j8JRmLgZk1gHGQ20L9od0EF3cQBLLyakHozLNL3JxV3bIMe 6CNJbSVAgyKDAaKxSpCptCrFFTj81Q4wrTV+TU8VxBNul4iTQwqiAug5/eSNaeegYyDx Ub4w== X-Gm-Message-State: AC+VfDxtKlNzHpz050oiGti6dsPvY/s5bP7peENDpXxKhmYtkUKKV2cL dftsxPNQqNIPlVoVe2TCp/nlVIQp7aEgu+MnOxlJqA== X-Google-Smtp-Source: ACHHUZ760WiAw71j7w78C3DCe1yGTBRrhdGTcRUYt7+oFJleZciJSTniMwqRMKClminFJ6B9exuP4g== X-Received: by 2002:a25:fc0a:0:b0:b9e:8f7b:5c36 with SMTP id v10-20020a25fc0a000000b00b9e8f7b5c36mr28674744ybd.17.1684159588783; Mon, 15 May 2023 07:06:28 -0700 (PDT) Received: from megalith.ht.home (cpe64777de11593-cm64777de11590.cpe.net.cable.rogers.com. [174.115.224.47]) by smtp.gmail.com with ESMTPSA id w7-20020ae9e507000000b0074df5d787f2sm2403397qkf.89.2023.05.15.07.06.28 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 15 May 2023 07:06:28 -0700 (PDT) From: Trevor Gamblin To: openembedded-devel@lists.openembedded.org Subject: [meta-python][PATCH 1/2] python3-pylint: upgrade 2.17.3 -> 2.17.4 Date: Mon, 15 May 2023 10:06:26 -0400 Message-Id: <20230515140627.2103801-1-tgamblin@baylibre.com> X-Mailer: git-send-email 2.40.1 MIME-Version: 1.0 List-Id: X-Webhook-Received: from li982-79.members.linode.com [45.33.32.79] by aws-us-west-2-korg-lkml-1.web.codeaurora.org with HTTPS for ; Mon, 15 May 2023 14:06:32 -0000 X-Groupsio-URL: https://lists.openembedded.org/g/openembedded-devel/message/102604 This upgrade fixes most of the ptest errors that have been seen with pylint, which were caused by a tomlkit issue. Changelog (see also https://github.com/pylint-dev/pylint/releases/tag/v2.17.4): False Positives Fixed - Fix a false positive for bad-dunder-name when there is a user-defined __index__ method. Closes #8613 Other Bug Fixes - pyreverse: added escaping of vertical bar character in annotation labels produced by DOT printer to ensure it is not treated as field separator of record-based nodes. Closes #8603 - Fixed a crash when generating a configuration file: tomlkit.exceptions.TOMLKitError: Can't add a table to a dotted key caused by tomlkit v0.11.8. Closes #8632 Signed-off-by: Trevor Gamblin --- .../{python3-pylint_2.17.3.bb => python3-pylint_2.17.4.bb} | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) rename meta-python/recipes-devtools/python/{python3-pylint_2.17.3.bb => python3-pylint_2.17.4.bb} (96%) diff --git a/meta-python/recipes-devtools/python/python3-pylint_2.17.3.bb b/meta-python/recipes-devtools/python/python3-pylint_2.17.4.bb similarity index 96% rename from meta-python/recipes-devtools/python/python3-pylint_2.17.3.bb rename to meta-python/recipes-devtools/python/python3-pylint_2.17.4.bb index 4a082066b..c20d91259 100644 --- a/meta-python/recipes-devtools/python/python3-pylint_2.17.3.bb +++ b/meta-python/recipes-devtools/python/python3-pylint_2.17.4.bb @@ -8,7 +8,7 @@ SRC_URI += " \ file://run-ptest \ file://0001-pyproject.toml-Remove-hard-coded-dependencies.patch \ " -SRCREV ?= "389e14c36819cb87190fd412d3f366a3283f0078" +SRCREV ?= "fc34a4b6abe56f3ac07ca15d846b1c1955545f85" inherit python_setuptools_build_meta ptest From patchwork Mon May 15 14:06:27 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Trevor Gamblin X-Patchwork-Id: 23963 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 aws-us-west-2-korg-lkml-1.web.codeaurora.org (localhost.localdomain [127.0.0.1]) by smtp.lore.kernel.org (Postfix) with ESMTP id F15BEC7EE22 for ; Mon, 15 May 2023 14:06:32 +0000 (UTC) Received: from mail-qt1-f179.google.com (mail-qt1-f179.google.com [209.85.160.179]) by mx.groups.io with SMTP id smtpd.web10.89561.1684159590817901112 for ; Mon, 15 May 2023 07:06:30 -0700 Authentication-Results: mx.groups.io; dkim=fail reason="signature has expired" header.i=@baylibre-com.20221208.gappssmtp.com header.s=20221208 header.b=EhkSgiH2; spf=pass (domain: baylibre.com, ip: 209.85.160.179, mailfrom: tgamblin@baylibre.com) Received: by mail-qt1-f179.google.com with SMTP id d75a77b69052e-3f529239e4aso11229641cf.0 for ; Mon, 15 May 2023 07:06:30 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=baylibre-com.20221208.gappssmtp.com; s=20221208; t=1684159589; x=1686751589; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:to:from:from:to:cc:subject:date:message-id :reply-to; bh=BqxEa0PNuAawl5fG8jbnNki2o1Dkb6gWnOHZ9teKchk=; b=EhkSgiH2RIcyt6TeznibU9D1lG/7cabT+nbc2ulJae/E7CthAG56c07YpuwJPLpwOx U4tzrNRA1+ds7PgdvsxApwJ+sLsUyCIRyqTAoo+XG/RyNqKE/TLMnZp8m6rAwzrZL1Rb 3A/obQxJ84SSV3kRcQ8jYVPXbvv0OvKlY8+35uGKZqge9ClYHYuaX5WPE+qWNBlVY46+ aBCVxqFaN+YhaxfdBdQz2FpZyJjEh0dQYwXX7J6TZ6lwVaf6ZGGnnw6ZPsuKK8re+8tG QP6AppKLPKFvT6FD0AECBrTxCGlgZcye0c5kg/WuRRx7Q9Pm+Chr715q3EYfRH60FPqG tzJQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1684159589; x=1686751589; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=BqxEa0PNuAawl5fG8jbnNki2o1Dkb6gWnOHZ9teKchk=; b=bnuUpNT2ng/vag4U+bNg+jO0sNluGZ1EfUS4Foecco9dxijUbg4B3qVUVyNi9yrBNP 49VFCmH9ndQy22wWWx+lw5E/+MmFTPxXP6IvWqZtG735FYw4uoaj4JooGc+63/jhfCeO qWO75juI5LqwBdfAOGcdN5OuocJA7coeswzYQjvXKRFTRX9tz7lEl1U9/X7Bgyb5jIji durjrhITJARtt/2A9sCyRhpH13VHQYg05/Kvz6GJZWVJIc38QDYpm+A4QVKFBTlkhxjy Xn4dE7RuU7iWjJ5RcDpZdpS96EvgXEXkvdaN2OGZlOPnFSZlKZSAHempum5YB27yYdti g+1w== X-Gm-Message-State: AC+VfDydHgXjBkLsoOzZkGk+Mv6euhsANFRY/v/Fx6a06NC3CU5iHEw1 Cnx0hzyp9kM/jJ5ts4S/UFcgwORm7Nk6zyRGr8M6ew== X-Google-Smtp-Source: ACHHUZ6zZDXjTYXnWnmD1Vjozj9JJabX1TVATw36qrCQEDj9idpR6/0Ns0nr9kOlkv3p6aGHMV0qCg== X-Received: by 2002:a05:622a:2c2:b0:3ef:6c09:edcc with SMTP id a2-20020a05622a02c200b003ef6c09edccmr58479907qtx.22.1684159589284; Mon, 15 May 2023 07:06:29 -0700 (PDT) Received: from megalith.ht.home (cpe64777de11593-cm64777de11590.cpe.net.cable.rogers.com. [174.115.224.47]) by smtp.gmail.com with ESMTPSA id w7-20020ae9e507000000b0074df5d787f2sm2403397qkf.89.2023.05.15.07.06.28 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 15 May 2023 07:06:29 -0700 (PDT) From: Trevor Gamblin To: openembedded-devel@lists.openembedded.org Subject: [meta-python][PATCH 2/2] python3-pylint: omit failing pickle test Date: Mon, 15 May 2023 10:06:27 -0400 Message-Id: <20230515140627.2103801-2-tgamblin@baylibre.com> X-Mailer: git-send-email 2.40.1 In-Reply-To: <20230515140627.2103801-1-tgamblin@baylibre.com> References: <20230515140627.2103801-1-tgamblin@baylibre.com> MIME-Version: 1.0 List-Id: X-Webhook-Received: from li982-79.members.linode.com [45.33.32.79] by aws-us-west-2-korg-lkml-1.web.codeaurora.org with HTTPS for ; Mon, 15 May 2023 14:06:32 -0000 X-Groupsio-URL: https://lists.openembedded.org/g/openembedded-devel/message/102605 There is one test in the pylint suite involving pickling non-pickleable plugins that fails on target (but not host). Disable this test for ptest runs. Signed-off-by: Trevor Gamblin --- ...001-pylint-remove-plugin-pickle-test.patch | 53 +++++++++++++++++++ .../python/python3-pylint_2.17.4.bb | 1 + 2 files changed, 54 insertions(+) create mode 100644 meta-python/recipes-devtools/python/python3-pylint/0001-pylint-remove-plugin-pickle-test.patch diff --git a/meta-python/recipes-devtools/python/python3-pylint/0001-pylint-remove-plugin-pickle-test.patch b/meta-python/recipes-devtools/python/python3-pylint/0001-pylint-remove-plugin-pickle-test.patch new file mode 100644 index 000000000..a7637a5ca --- /dev/null +++ b/meta-python/recipes-devtools/python/python3-pylint/0001-pylint-remove-plugin-pickle-test.patch @@ -0,0 +1,53 @@ +From 341e19dd69d847f84859e85cabff5846e7da24c4 Mon Sep 17 00:00:00 2001 +From: Trevor Gamblin +Date: Mon, 15 May 2023 09:26:04 -0400 +Subject: [PATCH] pylint: remove plugin pickle test + +Upstream-Status: Inappropriate (oe-specific) + +Upstream uses an assertion in the removed test to state that the test +needs changing if the plugin loading succeeds. The same failure wasn't +seen when running the tests on the host with pytest, so disable it by +removing the test from the suite. + +Signed-off-by: Trevor Gamblin +--- + tests/test_check_parallel.py | 22 ---------------------- + 1 file changed, 22 deletions(-) + +diff --git a/tests/test_check_parallel.py b/tests/test_check_parallel.py +index d56502eaf..34fec140f 100644 +--- a/tests/test_check_parallel.py ++++ b/tests/test_check_parallel.py +@@ -257,28 +257,6 @@ class TestCheckParallelFramework: + assert stats.statement == 18 + assert stats.warning == 0 + +- def test_linter_with_unpickleable_plugins_is_pickleable(self) -> None: +- """The linter needs to be pickle-able in order to be passed between workers""" +- linter = PyLinter(reporter=Reporter()) +- # We load an extension that we know is not pickle-safe +- linter.load_plugin_modules(["pylint.extensions.overlapping_exceptions"]) +- try: +- dill.dumps(linter) +- raise AssertionError( +- "Plugins loaded were pickle-safe! This test needs altering" +- ) +- except (KeyError, TypeError, PickleError, NotImplementedError): +- pass +- +- # And expect this call to make it pickle-able +- linter.load_plugin_configuration() +- try: +- dill.dumps(linter) +- except KeyError as exc: +- raise AssertionError( +- "Cannot pickle linter when using non-pickleable plugin" +- ) from exc +- + def test_worker_check_sequential_checker(self) -> None: + """Same as test_worker_check_single_file_no_checkers with SequentialTestChecker.""" + linter = PyLinter(reporter=Reporter()) +-- +2.40.1 + diff --git a/meta-python/recipes-devtools/python/python3-pylint_2.17.4.bb b/meta-python/recipes-devtools/python/python3-pylint_2.17.4.bb index c20d91259..96fce2e04 100644 --- a/meta-python/recipes-devtools/python/python3-pylint_2.17.4.bb +++ b/meta-python/recipes-devtools/python/python3-pylint_2.17.4.bb @@ -7,6 +7,7 @@ SRC_URI += " \ git://github.com/pylint-dev/pylint;branch=main;protocol=https \ file://run-ptest \ file://0001-pyproject.toml-Remove-hard-coded-dependencies.patch \ + file://0001-pylint-remove-plugin-pickle-test.patch \ " SRCREV ?= "fc34a4b6abe56f3ac07ca15d846b1c1955545f85"