diff mbox series

docs: poky.yaml.in: remove pylint3 from Ubuntu/Debian host dependencies

Message ID 20221107-upstream-pylint-removal-v1-0-014ecbdb50c7@theobroma-systems.com
State New
Headers show
Series docs: poky.yaml.in: remove pylint3 from Ubuntu/Debian host dependencies | expand

Commit Message

Quentin Schulz Nov. 7, 2022, 4:07 p.m. UTC
From: Quentin Schulz <quentin.schulz@theobroma-systems.com>

It is not actually needed outside of the autobuilder and since the
package got renamed in Ubuntu 20.04 and later to pylint, it was giving
the users an incorrect command line to run and would fail all the time,
better remove it instead of trying to handle Debian/recent Ubuntu
differences in the documentation.

Moreover, the other supported host distributions didn't have pylint
listed as a dependency, proving that it's indeed not needed.

Cc: Quentin Schulz <foss+yocto@0leil.net>
Signed-off-by: Quentin Schulz <quentin.schulz@theobroma-systems.com>
---
---
 documentation/poky.yaml.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)


---
base-commit: dce50679242d39f133e0cde5c8483b5e69f3eb54
change-id: 20221107-upstream-pylint-removal-00f7219e14f7

Best regards,

Comments

Michael Opdenacker Nov. 8, 2022, 11:11 a.m. UTC | #1
Hi Quentin

On 11/7/22 17:07, Quentin Schulz wrote:
> From: Quentin Schulz <quentin.schulz@theobroma-systems.com>
>
> It is not actually needed outside of the autobuilder and since the
> package got renamed in Ubuntu 20.04 and later to pylint, it was giving
> the users an incorrect command line to run and would fail all the time,
> better remove it instead of trying to handle Debian/recent Ubuntu
> differences in the documentation.
>
> Moreover, the other supported host distributions didn't have pylint
> listed as a dependency, proving that it's indeed not needed.
>
> Cc: Quentin Schulz <foss+yocto@0leil.net>
> Signed-off-by: Quentin Schulz <quentin.schulz@theobroma-systems.com>
> ---
> ---
>   documentation/poky.yaml.in | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/documentation/poky.yaml.in b/documentation/poky.yaml.in
> index 5639f3ca7..2d6affbca 100644
> --- a/documentation/poky.yaml.in
> +++ b/documentation/poky.yaml.in
> @@ -13,7 +13,7 @@ YOCTO_RELEASE_DL_URL : "&YOCTO_DL_URL;/releases/yocto/yocto-&DISTRO;"
>   UBUNTU_HOST_PACKAGES_ESSENTIAL : "gawk wget git diffstat unzip texinfo gcc \
>        build-essential chrpath socat cpio python3 python3-pip python3-pexpect \
>        xz-utils debianutils iputils-ping python3-git python3-jinja2 libegl1-mesa libsdl1.2-dev \
> -     pylint3 xterm python3-subunit mesa-common-dev zstd liblz4-tool"
> +     xterm python3-subunit mesa-common-dev zstd liblz4-tool"
>   FEDORA_HOST_PACKAGES_ESSENTIAL : "gawk make wget tar bzip2 gzip python3 unzip perl patch \
>        diffutils diffstat git cpp gcc gcc-c++ glibc-devel texinfo chrpath \
>        ccache perl-Data-Dumper perl-Text-ParseWords perl-Thread-Queue perl-bignum socat \
>
> ---
> base-commit: dce50679242d39f133e0cde5c8483b5e69f3eb54
> change-id: 20221107-upstream-pylint-removal-00f7219e14f7

Reviewed-by: Michael Opdenacker <michael.opdenacker@bootlin.com>
... and merged into master-next.

Thanks for the patch!
Cheers
Michael.
diff mbox series

Patch

diff --git a/documentation/poky.yaml.in b/documentation/poky.yaml.in
index 5639f3ca7..2d6affbca 100644
--- a/documentation/poky.yaml.in
+++ b/documentation/poky.yaml.in
@@ -13,7 +13,7 @@  YOCTO_RELEASE_DL_URL : "&YOCTO_DL_URL;/releases/yocto/yocto-&DISTRO;"
 UBUNTU_HOST_PACKAGES_ESSENTIAL : "gawk wget git diffstat unzip texinfo gcc \
      build-essential chrpath socat cpio python3 python3-pip python3-pexpect \
      xz-utils debianutils iputils-ping python3-git python3-jinja2 libegl1-mesa libsdl1.2-dev \
-     pylint3 xterm python3-subunit mesa-common-dev zstd liblz4-tool"
+     xterm python3-subunit mesa-common-dev zstd liblz4-tool"
 FEDORA_HOST_PACKAGES_ESSENTIAL : "gawk make wget tar bzip2 gzip python3 unzip perl patch \
      diffutils diffstat git cpp gcc gcc-c++ glibc-devel texinfo chrpath \
      ccache perl-Data-Dumper perl-Text-ParseWords perl-Thread-Queue perl-bignum socat \