[meta-python,v2] python3-pylint: Update to 3.1.0

From: Dan McGregor <dan.mcgregor@vecima.com>

PyLint needs to be comapatible with Astroid. It was recently updated
to version 3.1.0, so follow suit with pylint.

Signed-off-by: Dan McGregor <dan.mcgregor@vecima.com>
---
 ...1-Adjust-test-expectations-for-ptest.patch | 44 +++++++++++++++++++
 ...ylint_3.0.3.bb => python3-pylint_3.1.0.bb} |  5 ++-
 2 files changed, 47 insertions(+), 2 deletions(-)
 create mode 100644 meta-python/recipes-devtools/python/python3-pylint/0001-Adjust-test-expectations-for-ptest.patch
 rename meta-python/recipes-devtools/python/{python3-pylint_3.0.3.bb => python3-pylint_3.1.0.bb} (91%)

Hm. looks like v1 was committed. I'll rebase this change and submit a
new version.

On Wed, 13 Mar 2024 at 14:53, Dan McGregor <danismostlikely@gmail.com> wrote:
>
> From: Dan McGregor <dan.mcgregor@vecima.com>
>
> PyLint needs to be comapatible with Astroid. It was recently updated
> to version 3.1.0, so follow suit with pylint.
>
> Signed-off-by: Dan McGregor <dan.mcgregor@vecima.com>
> ---
>  ...1-Adjust-test-expectations-for-ptest.patch | 44 +++++++++++++++++++
>  ...ylint_3.0.3.bb => python3-pylint_3.1.0.bb} |  5 ++-
>  2 files changed, 47 insertions(+), 2 deletions(-)
>  create mode 100644 meta-python/recipes-devtools/python/python3-pylint/0001-Adjust-test-expectations-for-ptest.patch
>  rename meta-python/recipes-devtools/python/{python3-pylint_3.0.3.bb => python3-pylint_3.1.0.bb} (91%)
>
> diff --git a/meta-python/recipes-devtools/python/python3-pylint/0001-Adjust-test-expectations-for-ptest.patch b/meta-python/recipes-devtools/python/python3-pylint/0001-Adjust-test-expectations-for-ptest.patch
> new file mode 100644
> index 00000000000..8edecbf1bd1
> --- /dev/null
> +++ b/meta-python/recipes-devtools/python/python3-pylint/0001-Adjust-test-expectations-for-ptest.patch
> @@ -0,0 +1,44 @@
> +From f3a213cf4922c122e554277ea2031c0c54cd3fe5 Mon Sep 17 00:00:00 2001
> +From: Dan McGregor <dan.mcgregor@vecima.com>
> +Date: Wed, 13 Mar 2024 14:08:25 -0600
> +Subject: [PATCH] Adjust test expectations for ptest
> +
> +Because of the way we install ptests, some first party imports
> +became third party. Reflect that in the test case.
> +
> +Upstream-Status: Inappropriate (embedded specific)
> +Signed-off-by: Dan McGregor <dan.mcgregor@vecima.com>
> +---
> + tests/functional/w/wrong_import_order.txt | 22 +++++++++++-----------
> + 1 file changed, 11 insertions(+), 11 deletions(-)
> +
> +diff --git a/tests/functional/w/wrong_import_order.txt b/tests/functional/w/wrong_import_order.txt
> +index 068d2140d..9594ff9e0 100644
> +--- a/tests/functional/w/wrong_import_order.txt
> ++++ b/tests/functional/w/wrong_import_order.txt
> +@@ -3,14 +3,14 @@ wrong-import-order:14:0:14:10::"standard import ""sys"" should be placed before
> + wrong-import-order:15:0:15:15::"standard import ""datetime"" should be placed before third party imports ""six"", ""astroid.are_exclusive""":UNDEFINED
> + wrong-import-order:18:0:18:22::"third party import ""totally_missing"" should be placed before local import ""package.Class""":UNDEFINED
> + wrong-import-order:20:0:20:14::"third party import ""astroid"" should be placed before local imports ""package.Class"", "".package""":UNDEFINED
> +-wrong-import-order:22:0:22:22::"first party import ""pylint.checkers"" should be placed before local imports ""package.Class"", "".package"", "".package2""":UNDEFINED
> +-wrong-import-order:23:0:23:25::"first party import ""pylint.config"" should be placed before local imports ""package.Class"", "".package"", "".package2""":UNDEFINED
> +-wrong-import-order:24:0:24:17::"first party import ""pylint.sys"" should be placed before local imports ""package.Class"", "".package"", "".package2""":UNDEFINED
> +-wrong-import-order:25:0:25:28::"first party import ""pylint.pyreverse"" should be placed before local imports ""package.Class"", "".package"", "".package2""":UNDEFINED
> +-wrong-import-order:30:0:30:40::"third party import ""six.moves.urllib.parse.quote"" should be placed before first party imports ""pylint.checkers"", ""pylint.config"", ""pylint.sys"", ""pylint.pyreverse"" and local imports ""package.Class"", "".package"", "".package2"" (...) ""package3.Class3"", "".package4"", ""package4.Class4""":UNDEFINED
> +-wrong-import-order:31:0:31:23::"first party import ""pylint.constants"" should be placed before local imports ""package.Class"", "".package"", "".package2"" (...) ""package3.Class3"", "".package4"", ""package4.Class4""":UNDEFINED
> +-wrong-import-order:32:0:32:19::"standard import ""re"" should be placed before third party imports ""six"", ""astroid.are_exclusive"", ""unused_import"", ""totally_missing"", ""astroid"", ""six.moves.urllib.parse.quote"", first party imports ""pylint.checkers"", ""pylint.config"", ""pylint.sys"", ""pylint.pyreverse"", ""pylint.constants"", and local imports ""package.Class"", "".package"", "".package2"" (...) ""package3.Class3"", "".package4"", ""package4.Class4""":UNDEFINED
> +-wrong-import-order:32:0:32:19::"third party import ""requests"" should be placed before first party imports ""pylint.checkers"", ""pylint.config"", ""pylint.sys"", ""pylint.pyreverse"", ""pylint.constants"" and local imports ""package.Class"", "".package"", "".package2"" (...) ""package3.Class3"", "".package4"", ""package4.Class4""":UNDEFINED
> +-wrong-import-order:33:0:33:24::"first party import ""pylint.exceptions"" should be placed before local imports ""package.Class"", "".package"", "".package2"" (...) ""package3.Class3"", "".package4"", ""package4.Class4""":UNDEFINED
> +-wrong-import-order:34:0:34:21::"first party import ""pylint.message"" should be placed before local imports ""package.Class"", "".package"", "".package2"" (...) ""package3.Class3"", "".package4"", ""package4.Class4""":UNDEFINED
> +-wrong-import-order:35:0:35:11::"standard import ""time"" should be placed before third party imports ""six"", ""astroid.are_exclusive"", ""unused_import"" (...) ""astroid"", ""six.moves.urllib.parse.quote"", ""requests"", first party imports ""pylint.checkers"", ""pylint.config"", ""pylint.sys"" (...) ""pylint.constants"", ""pylint.exceptions"", ""pylint.message"", and local imports ""package.Class"", "".package"", "".package2"" (...) ""package3.Class3"", "".package4"", ""package4.Class4""":UNDEFINED
> ++wrong-import-order:22:0:22:22::"third party import ""pylint.checkers"" should be placed before local imports ""package.Class"", "".package"", "".package2""":UNDEFINED
> ++wrong-import-order:23:0:23:25::"third party import ""pylint.config"" should be placed before local imports ""package.Class"", "".package"", "".package2""":UNDEFINED
> ++wrong-import-order:24:0:24:17::"third party import ""pylint.sys"" should be placed before local imports ""package.Class"", "".package"", "".package2""":UNDEFINED
> ++wrong-import-order:25:0:25:28::"third party import ""pylint.pyreverse"" should be placed before local imports ""package.Class"", "".package"", "".package2""":UNDEFINED
> ++wrong-import-order:30:0:30:40::"third party import ""six.moves.urllib.parse.quote"" should be placed before local imports ""package.Class"", "".package"", "".package2"" (...) ""package3.Class3"", "".package4"", ""package4.Class4""":UNDEFINED
> ++wrong-import-order:31:0:31:23::"third party import ""pylint.constants"" should be placed before local imports ""package.Class"", "".package"", "".package2"" (...) ""package3.Class3"", "".package4"", ""package4.Class4""":UNDEFINED
> ++wrong-import-order:32:0:32:19::"standard import ""re"" should be placed before third party imports ""six"", ""astroid.are_exclusive"", ""unused_import"" (...) ""pylint.pyreverse"", ""six.moves.urllib.parse.quote"", ""pylint.constants"" and local imports ""package.Class"", "".package"", "".package2"" (...) ""package3.Class3"", "".package4"", ""package4.Class4""":UNDEFINED
> ++wrong-import-order:32:0:32:19::"third party import ""requests"" should be placed before local imports ""package.Class"", "".package"", "".package2"" (...) ""package3.Class3"", "".package4"", ""package4.Class4""":UNDEFINED
> ++wrong-import-order:33:0:33:24::"third party import ""pylint.exceptions"" should be placed before local imports ""package.Class"", "".package"", "".package2"" (...) ""package3.Class3"", "".package4"", ""package4.Class4""":UNDEFINED
> ++wrong-import-order:34:0:34:21::"third party import ""pylint.message"" should be placed before local imports ""package.Class"", "".package"", "".package2"" (...) ""package3.Class3"", "".package4"", ""package4.Class4""":UNDEFINED
> ++wrong-import-order:35:0:35:11::"standard import ""time"" should be placed before third party imports ""six"", ""astroid.are_exclusive"", ""unused_import"" (...) ""requests"", ""pylint.exceptions"", ""pylint.message"" and local imports ""package.Class"", "".package"", "".package2"" (...) ""package3.Class3"", "".package4"", ""package4.Class4""":UNDEFINED
> diff --git a/meta-python/recipes-devtools/python/python3-pylint_3.0.3.bb b/meta-python/recipes-devtools/python/python3-pylint_3.1.0.bb
> similarity index 91%
> rename from meta-python/recipes-devtools/python/python3-pylint_3.0.3.bb
> rename to meta-python/recipes-devtools/python/python3-pylint_3.1.0.bb
> index eb4b9e897ff..12f4f908af7 100644
> --- a/meta-python/recipes-devtools/python/python3-pylint_3.0.3.bb
> +++ b/meta-python/recipes-devtools/python/python3-pylint_3.1.0.bb
> @@ -3,10 +3,11 @@ HOMEPAGE= "http://www.pylint.org/"
>  LICENSE = "GPL-2.0-only"
>  LIC_FILES_CHKSUM = "file://LICENSE;md5=c107cf754550e65755c42985a5d4e9c9"
>
> -SRC_URI += "git://github.com/pylint-dev/pylint;branch=maintenance/3.0.x;protocol=https \
> +SRC_URI += "git://github.com/pylint-dev/pylint;branch=maintenance/3.1.x;protocol=https \
> +           file://0001-Adjust-test-expectations-for-ptest.patch \
>             file://run-ptest \
>             "
> -SRCREV = "1a5ffc1f447b77071ffe18a9c6836c09147ee2ed"
> +SRCREV = "053c2c3a25f97f043e10cdd41d2609e495a68f57"
>
>  inherit python_setuptools_build_meta ptest
>
> --
> 2.41.0
>

On Wed, Mar 13, 2024 at 2:04 PM Dan McGregor <danismostlikely@gmail.com> wrote:
>
> Hm. looks like v1 was committed. I'll rebase this change and submit a
> new version.

yes since this upgrade was reducing number of ptest failures down to
1, it was an improvement on existing
situation regardless. So I accepted it. Thanks for fixing it, and yes
please rebase it on top of latest master
and resend.

>
> On Wed, 13 Mar 2024 at 14:53, Dan McGregor <danismostlikely@gmail.com> wrote:
> >
> > From: Dan McGregor <dan.mcgregor@vecima.com>
> >
> > PyLint needs to be comapatible with Astroid. It was recently updated
> > to version 3.1.0, so follow suit with pylint.
> >
> > Signed-off-by: Dan McGregor <dan.mcgregor@vecima.com>
> > ---
> >  ...1-Adjust-test-expectations-for-ptest.patch | 44 +++++++++++++++++++
> >  ...ylint_3.0.3.bb => python3-pylint_3.1.0.bb} |  5 ++-
> >  2 files changed, 47 insertions(+), 2 deletions(-)
> >  create mode 100644 meta-python/recipes-devtools/python/python3-pylint/0001-Adjust-test-expectations-for-ptest.patch
> >  rename meta-python/recipes-devtools/python/{python3-pylint_3.0.3.bb => python3-pylint_3.1.0.bb} (91%)
> >
> > diff --git a/meta-python/recipes-devtools/python/python3-pylint/0001-Adjust-test-expectations-for-ptest.patch b/meta-python/recipes-devtools/python/python3-pylint/0001-Adjust-test-expectations-for-ptest.patch
> > new file mode 100644
> > index 00000000000..8edecbf1bd1
> > --- /dev/null
> > +++ b/meta-python/recipes-devtools/python/python3-pylint/0001-Adjust-test-expectations-for-ptest.patch
> > @@ -0,0 +1,44 @@
> > +From f3a213cf4922c122e554277ea2031c0c54cd3fe5 Mon Sep 17 00:00:00 2001
> > +From: Dan McGregor <dan.mcgregor@vecima.com>
> > +Date: Wed, 13 Mar 2024 14:08:25 -0600
> > +Subject: [PATCH] Adjust test expectations for ptest
> > +
> > +Because of the way we install ptests, some first party imports
> > +became third party. Reflect that in the test case.
> > +
> > +Upstream-Status: Inappropriate (embedded specific)
> > +Signed-off-by: Dan McGregor <dan.mcgregor@vecima.com>
> > +---
> > + tests/functional/w/wrong_import_order.txt | 22 +++++++++++-----------
> > + 1 file changed, 11 insertions(+), 11 deletions(-)
> > +
> > +diff --git a/tests/functional/w/wrong_import_order.txt b/tests/functional/w/wrong_import_order.txt
> > +index 068d2140d..9594ff9e0 100644
> > +--- a/tests/functional/w/wrong_import_order.txt
> > ++++ b/tests/functional/w/wrong_import_order.txt
> > +@@ -3,14 +3,14 @@ wrong-import-order:14:0:14:10::"standard import ""sys"" should be placed before
> > + wrong-import-order:15:0:15:15::"standard import ""datetime"" should be placed before third party imports ""six"", ""astroid.are_exclusive""":UNDEFINED
> > + wrong-import-order:18:0:18:22::"third party import ""totally_missing"" should be placed before local import ""package.Class""":UNDEFINED
> > + wrong-import-order:20:0:20:14::"third party import ""astroid"" should be placed before local imports ""package.Class"", "".package""":UNDEFINED
> > +-wrong-import-order:22:0:22:22::"first party import ""pylint.checkers"" should be placed before local imports ""package.Class"", "".package"", "".package2""":UNDEFINED
> > +-wrong-import-order:23:0:23:25::"first party import ""pylint.config"" should be placed before local imports ""package.Class"", "".package"", "".package2""":UNDEFINED
> > +-wrong-import-order:24:0:24:17::"first party import ""pylint.sys"" should be placed before local imports ""package.Class"", "".package"", "".package2""":UNDEFINED
> > +-wrong-import-order:25:0:25:28::"first party import ""pylint.pyreverse"" should be placed before local imports ""package.Class"", "".package"", "".package2""":UNDEFINED
> > +-wrong-import-order:30:0:30:40::"third party import ""six.moves.urllib.parse.quote"" should be placed before first party imports ""pylint.checkers"", ""pylint.config"", ""pylint.sys"", ""pylint.pyreverse"" and local imports ""package.Class"", "".package"", "".package2"" (...) ""package3.Class3"", "".package4"", ""package4.Class4""":UNDEFINED
> > +-wrong-import-order:31:0:31:23::"first party import ""pylint.constants"" should be placed before local imports ""package.Class"", "".package"", "".package2"" (...) ""package3.Class3"", "".package4"", ""package4.Class4""":UNDEFINED
> > +-wrong-import-order:32:0:32:19::"standard import ""re"" should be placed before third party imports ""six"", ""astroid.are_exclusive"", ""unused_import"", ""totally_missing"", ""astroid"", ""six.moves.urllib.parse.quote"", first party imports ""pylint.checkers"", ""pylint.config"", ""pylint.sys"", ""pylint.pyreverse"", ""pylint.constants"", and local imports ""package.Class"", "".package"", "".package2"" (...) ""package3.Class3"", "".package4"", ""package4.Class4""":UNDEFINED
> > +-wrong-import-order:32:0:32:19::"third party import ""requests"" should be placed before first party imports ""pylint.checkers"", ""pylint.config"", ""pylint.sys"", ""pylint.pyreverse"", ""pylint.constants"" and local imports ""package.Class"", "".package"", "".package2"" (...) ""package3.Class3"", "".package4"", ""package4.Class4""":UNDEFINED
> > +-wrong-import-order:33:0:33:24::"first party import ""pylint.exceptions"" should be placed before local imports ""package.Class"", "".package"", "".package2"" (...) ""package3.Class3"", "".package4"", ""package4.Class4""":UNDEFINED
> > +-wrong-import-order:34:0:34:21::"first party import ""pylint.message"" should be placed before local imports ""package.Class"", "".package"", "".package2"" (...) ""package3.Class3"", "".package4"", ""package4.Class4""":UNDEFINED
> > +-wrong-import-order:35:0:35:11::"standard import ""time"" should be placed before third party imports ""six"", ""astroid.are_exclusive"", ""unused_import"" (...) ""astroid"", ""six.moves.urllib.parse.quote"", ""requests"", first party imports ""pylint.checkers"", ""pylint.config"", ""pylint.sys"" (...) ""pylint.constants"", ""pylint.exceptions"", ""pylint.message"", and local imports ""package.Class"", "".package"", "".package2"" (...) ""package3.Class3"", "".package4"", ""package4.Class4""":UNDEFINED
> > ++wrong-import-order:22:0:22:22::"third party import ""pylint.checkers"" should be placed before local imports ""package.Class"", "".package"", "".package2""":UNDEFINED
> > ++wrong-import-order:23:0:23:25::"third party import ""pylint.config"" should be placed before local imports ""package.Class"", "".package"", "".package2""":UNDEFINED
> > ++wrong-import-order:24:0:24:17::"third party import ""pylint.sys"" should be placed before local imports ""package.Class"", "".package"", "".package2""":UNDEFINED
> > ++wrong-import-order:25:0:25:28::"third party import ""pylint.pyreverse"" should be placed before local imports ""package.Class"", "".package"", "".package2""":UNDEFINED
> > ++wrong-import-order:30:0:30:40::"third party import ""six.moves.urllib.parse.quote"" should be placed before local imports ""package.Class"", "".package"", "".package2"" (...) ""package3.Class3"", "".package4"", ""package4.Class4""":UNDEFINED
> > ++wrong-import-order:31:0:31:23::"third party import ""pylint.constants"" should be placed before local imports ""package.Class"", "".package"", "".package2"" (...) ""package3.Class3"", "".package4"", ""package4.Class4""":UNDEFINED
> > ++wrong-import-order:32:0:32:19::"standard import ""re"" should be placed before third party imports ""six"", ""astroid.are_exclusive"", ""unused_import"" (...) ""pylint.pyreverse"", ""six.moves.urllib.parse.quote"", ""pylint.constants"" and local imports ""package.Class"", "".package"", "".package2"" (...) ""package3.Class3"", "".package4"", ""package4.Class4""":UNDEFINED
> > ++wrong-import-order:32:0:32:19::"third party import ""requests"" should be placed before local imports ""package.Class"", "".package"", "".package2"" (...) ""package3.Class3"", "".package4"", ""package4.Class4""":UNDEFINED
> > ++wrong-import-order:33:0:33:24::"third party import ""pylint.exceptions"" should be placed before local imports ""package.Class"", "".package"", "".package2"" (...) ""package3.Class3"", "".package4"", ""package4.Class4""":UNDEFINED
> > ++wrong-import-order:34:0:34:21::"third party import ""pylint.message"" should be placed before local imports ""package.Class"", "".package"", "".package2"" (...) ""package3.Class3"", "".package4"", ""package4.Class4""":UNDEFINED
> > ++wrong-import-order:35:0:35:11::"standard import ""time"" should be placed before third party imports ""six"", ""astroid.are_exclusive"", ""unused_import"" (...) ""requests"", ""pylint.exceptions"", ""pylint.message"" and local imports ""package.Class"", "".package"", "".package2"" (...) ""package3.Class3"", "".package4"", ""package4.Class4""":UNDEFINED
> > diff --git a/meta-python/recipes-devtools/python/python3-pylint_3.0.3.bb b/meta-python/recipes-devtools/python/python3-pylint_3.1.0.bb
> > similarity index 91%
> > rename from meta-python/recipes-devtools/python/python3-pylint_3.0.3.bb
> > rename to meta-python/recipes-devtools/python/python3-pylint_3.1.0.bb
> > index eb4b9e897ff..12f4f908af7 100644
> > --- a/meta-python/recipes-devtools/python/python3-pylint_3.0.3.bb
> > +++ b/meta-python/recipes-devtools/python/python3-pylint_3.1.0.bb
> > @@ -3,10 +3,11 @@ HOMEPAGE= "http://www.pylint.org/"
> >  LICENSE = "GPL-2.0-only"
> >  LIC_FILES_CHKSUM = "file://LICENSE;md5=c107cf754550e65755c42985a5d4e9c9"
> >
> > -SRC_URI += "git://github.com/pylint-dev/pylint;branch=maintenance/3.0.x;protocol=https \
> > +SRC_URI += "git://github.com/pylint-dev/pylint;branch=maintenance/3.1.x;protocol=https \
> > +           file://0001-Adjust-test-expectations-for-ptest.patch \
> >             file://run-ptest \
> >             "
> > -SRCREV = "1a5ffc1f447b77071ffe18a9c6836c09147ee2ed"
> > +SRCREV = "053c2c3a25f97f043e10cdd41d2609e495a68f57"
> >
> >  inherit python_setuptools_build_meta ptest
> >
> > --
> > 2.41.0
> >
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#109296): https://lists.openembedded.org/g/openembedded-devel/message/109296
> Mute This Topic: https://lists.openembedded.org/mt/104914544/1997914
> Group Owner: openembedded-devel+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub [raj.khem@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>