python: make 'python' install everything instead of just the interpretter

Message ID 20190620145325.8654-1-ross.burton@intel.com
State Accepted
Commit 778a6b4bf6aa9d0941d52ee05e14b061b659b3df
Headers show
Series
  • python: make 'python' install everything instead of just the interpretter
Related show

Commit Message

Ross Burton June 20, 2019, 2:53 p.m.
Follow the python3 behaviour, and common sense, by making 'python' install
python-modules instead of python-core.  This means a user installing python gets
all of Python, instead of just a fraction of the library.

[ YOCTO #13402 ]

Signed-off-by: Ross Burton <ross.burton@intel.com>

---
 meta/recipes-devtools/python/python_2.7.16.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

-- 
2.11.0

-- 
_______________________________________________
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Comments

Alejandro Enedino Hernandez Samaniego June 24, 2019, 8:20 p.m. | #1
Hey Ross,

On 6/20/19 7:53 AM, Ross Burton wrote:
> Follow the python3 behaviour, and common sense, by making 'python' install

> python-modules instead of python-core.  This means a user installing python gets

> all of Python, instead of just a fraction of the library.

>

> [ YOCTO #13402 ]

>

> Signed-off-by: Ross Burton <ross.burton@intel.com>

> ---

>   meta/recipes-devtools/python/python_2.7.16.bb | 2 +-

>   1 file changed, 1 insertion(+), 1 deletion(-)

>

> diff --git a/meta/recipes-devtools/python/python_2.7.16.bb b/meta/recipes-devtools/python/python_2.7.16.bb

> index d70342fe3a6..5f387b8af11 100644

> --- a/meta/recipes-devtools/python/python_2.7.16.bb

> +++ b/meta/recipes-devtools/python/python_2.7.16.bb

> @@ -162,7 +162,7 @@ py_package_preprocess () {

>   PACKAGES_remove = "${PN}"

>   

>   # manual dependency additions

> -RPROVIDES_${PN}-core = "${PN}"

> +RPROVIDES_${PN}-modules = "${PN}"


Is it just me or we already had this?, maybe we had this only for python3


Alejandro


>   RRECOMMENDS_${PN}-core_append_class-nativesdk = " nativesdk-python-modules"

>   RRECOMMENDS_${PN}-crypt = "openssl"

>   

-- 
_______________________________________________
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core
Ross Burton June 24, 2019, 9:02 p.m. | #2
On Mon, 24 Jun 2019 at 21:20, Alejandro Enedino Hernandez Samaniego
<alejandro.enedino.hernandez-samaniego@xilinx.com> wrote:
 > >   # manual dependency additions

> > -RPROVIDES_${PN}-core = "${PN}"

> > +RPROVIDES_${PN}-modules = "${PN}"

>

> Is it just me or we already had this?, maybe we had this only for python3


It was only for Py3!  Swore I already fixed this too...

Ross
-- 
_______________________________________________
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Patch

diff --git a/meta/recipes-devtools/python/python_2.7.16.bb b/meta/recipes-devtools/python/python_2.7.16.bb
index d70342fe3a6..5f387b8af11 100644
--- a/meta/recipes-devtools/python/python_2.7.16.bb
+++ b/meta/recipes-devtools/python/python_2.7.16.bb
@@ -162,7 +162,7 @@  py_package_preprocess () {
 PACKAGES_remove = "${PN}"
 
 # manual dependency additions
-RPROVIDES_${PN}-core = "${PN}"
+RPROVIDES_${PN}-modules = "${PN}"
 RRECOMMENDS_${PN}-core_append_class-nativesdk = " nativesdk-python-modules"
 RRECOMMENDS_${PN}-crypt = "openssl"