[Buildroot] [PATCH] package/python: clarify that this refers to the deprecated 2.7 series

[Buildroot] [PATCH] package/python: clarify that this refers to the deprecated 2.7 series

[Peter Korsgaard]

Python 2.7 is EOL, so people should use the python3 package instead if
possible.  Make it a bit more obvious that 'python' is not the right package
to use by explicitly mentioning that this if about python 2.7 and that it is
deprecated.

Signed-off-by: Peter Korsgaard <peter@korsgaard.com>
---
 package/python/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/python/Config.in b/package/python/Config.in
index bded00c9e8..ab6950cae8 100644
--- a/package/python/Config.in
+++ b/package/python/Config.in
@@ -3,7 +3,7 @@ comment "python needs a toolchain w/ wchar, threads, dynamic library"
 	depends on !BR2_USE_WCHAR || !BR2_TOOLCHAIN_HAS_THREADS || BR2_STATIC_LIBS
 
 config BR2_PACKAGE_PYTHON
-	bool "python"
+	bool "python2.7 (deprecated)"
 	depends on BR2_USE_WCHAR
 	# uses fork()
 	depends on BR2_USE_MMU
-- 
2.20.1

[Buildroot] [PATCH] package/python: clarify that this refers to the deprecated 2.7 series

[Peter Seiderer]

Hello Peter,

On Mon,  8 Feb 2021 14:24:11 +0100, Peter Korsgaard <peter@korsgaard.com> wrote:

> Python 2.7 is EOL, so people should use the python3 package instead if
> possible.  Make it a bit more obvious that 'python' is not the right package
> to use by explicitly mentioning that this if about python 2.7 and that it is

s/this if about/this is about/?

Reviewed-by: Peter Seiderer <ps.report@gmx.net>

Regards,
Peter

> deprecated.
>
> Signed-off-by: Peter Korsgaard <peter@korsgaard.com>
> ---
>  package/python/Config.in | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/package/python/Config.in b/package/python/Config.in
> index bded00c9e8..ab6950cae8 100644
> --- a/package/python/Config.in
> +++ b/package/python/Config.in
> @@ -3,7 +3,7 @@ comment "python needs a toolchain w/ wchar, threads, dynamic library"
>  	depends on !BR2_USE_WCHAR || !BR2_TOOLCHAIN_HAS_THREADS || BR2_STATIC_LIBS
>
>  config BR2_PACKAGE_PYTHON
> -	bool "python"
> +	bool "python2.7 (deprecated)"
>  	depends on BR2_USE_WCHAR
>  	# uses fork()
>  	depends on BR2_USE_MMU

[Buildroot] [PATCH] package/python: clarify that this refers to the deprecated 2.7 series

[Peter Korsgaard]

>>>>> "Peter" == Peter Korsgaard <peter@korsgaard.com> writes:

 > Python 2.7 is EOL, so people should use the python3 package instead if
 > possible.  Make it a bit more obvious that 'python' is not the right package
 > to use by explicitly mentioning that this if about python 2.7 and that it is
 > deprecated.

 > Signed-off-by: Peter Korsgaard <peter@korsgaard.com>

Committed after fixing s/if/is/ in the commit message as pointed out by
Peter S., thanks.

 > ---
 >  package/python/Config.in | 2 +-
 >  1 file changed, 1 insertion(+), 1 deletion(-)

 > diff --git a/package/python/Config.in b/package/python/Config.in
 > index bded00c9e8..ab6950cae8 100644
 > --- a/package/python/Config.in
 > +++ b/package/python/Config.in
 > @@ -3,7 +3,7 @@ comment "python needs a toolchain w/ wchar, threads, dynamic library"
 >  	depends on !BR2_USE_WCHAR || !BR2_TOOLCHAIN_HAS_THREADS || BR2_STATIC_LIBS
 
 >  config BR2_PACKAGE_PYTHON
 > -	bool "python"
 > +	bool "python2.7 (deprecated)"
 >  	depends on BR2_USE_WCHAR
 >  	# uses fork()
 >  	depends on BR2_USE_MMU
 > -- 
 > 2.20.1


-- 
Bye, Peter Korsgaard