[PATCH] jailhouse_git: update to inherit python3 classes

[PATCH] jailhouse_git: update to inherit python3 classes

[Peter Griffin]

This avoids the parsing errors, now that python2
has been removed.

Signed-off-by: Peter Griffin <peter.griffin@linaro.org>
---
 recipes-kernel/jailhouse/jailhouse_git.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/recipes-kernel/jailhouse/jailhouse_git.bb b/recipes-kernel/jailhouse/jailhouse_git.bb
index 4186dc3..89833e6 100644
--- a/recipes-kernel/jailhouse/jailhouse_git.bb
+++ b/recipes-kernel/jailhouse/jailhouse_git.bb
@@ -29,7 +29,7 @@ RDEPENDS_${PN} += "\
 S = "${WORKDIR}/git"
 
 require jailhouse-arch.inc
-inherit module pythonnative bash-completion deploy setuptools
+inherit module python3native bash-completion deploy setuptools3
 
 PACKAGE_ARCH = "${MACHINE_ARCH}"
 COMPATIBLE_MACHINE = "(ti-soc)"
-- 
2.7.4

Re: [PATCH] jailhouse_git: update to inherit python3 classes

[Denys Dmytriyenko]

NAK

On Wed, Jan 29, 2020 at 11:15:55PM +0000, Peter Griffin wrote:
> This avoids the parsing errors, now that python2
> has been removed.
> 
> Signed-off-by: Peter Griffin <peter.griffin@linaro.org>
> ---
>  recipes-kernel/jailhouse/jailhouse_git.bb | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/recipes-kernel/jailhouse/jailhouse_git.bb b/recipes-kernel/jailhouse/jailhouse_git.bb
> index 4186dc3..89833e6 100644
> --- a/recipes-kernel/jailhouse/jailhouse_git.bb
> +++ b/recipes-kernel/jailhouse/jailhouse_git.bb
> @@ -29,7 +29,7 @@ RDEPENDS_${PN} += "\
>  S = "${WORKDIR}/git"
>  
>  require jailhouse-arch.inc
> -inherit module pythonnative bash-completion deploy setuptools
> +inherit module python3native bash-completion deploy setuptools3
>  
>  PACKAGE_ARCH = "${MACHINE_ARCH}"
>  COMPATIBLE_MACHINE = "(ti-soc)"
> -- 
> 2.7.4
> 

Re: [meta-ti] [PATCH] jailhouse_git: update to inherit python3 classes

[Denys Dmytriyenko]

Let me elaborate:

1. Your patch is incomplete - it fixes parsing issues, but not build issues
2. There was a more complete patch submitted earlier in the day to address that:
https://lists.yoctoproject.org/g/meta-ti/message/12625

-- 
Denys


On Wed, Jan 29, 2020 at 06:19:23PM -0500, Denys Dmytriyenko via Lists.Yoctoproject.Org wrote:
> NAK
> 
> On Wed, Jan 29, 2020 at 11:15:55PM +0000, Peter Griffin wrote:
> > This avoids the parsing errors, now that python2
> > has been removed.
> > 
> > Signed-off-by: Peter Griffin <peter.griffin@linaro.org>
> > ---
> >  recipes-kernel/jailhouse/jailhouse_git.bb | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/recipes-kernel/jailhouse/jailhouse_git.bb b/recipes-kernel/jailhouse/jailhouse_git.bb
> > index 4186dc3..89833e6 100644
> > --- a/recipes-kernel/jailhouse/jailhouse_git.bb
> > +++ b/recipes-kernel/jailhouse/jailhouse_git.bb
> > @@ -29,7 +29,7 @@ RDEPENDS_${PN} += "\
> >  S = "${WORKDIR}/git"
> >  
> >  require jailhouse-arch.inc
> > -inherit module pythonnative bash-completion deploy setuptools
> > +inherit module python3native bash-completion deploy setuptools3
> >  
> >  PACKAGE_ARCH = "${MACHINE_ARCH}"
> >  COMPATIBLE_MACHINE = "(ti-soc)"
> > -- 
> > 2.7.4
> > 

> 

Re: [meta-ti] [PATCH] jailhouse_git: update to inherit python3 classes

[Peter Griffin]

[-- Attachment #1: Type: text/plain, Size: 1906 bytes --]

Hi Denys,

On Thu, 30 Jan 2020 at 03:34, Denys Dmytriyenko <denys@ti.com> wrote:

> Let me elaborate:
>
> 1. Your patch is incomplete - it fixes parsing issues, but not build issues
>

Thanks for the extra info, yes that is correct I didn't build any TI
platform,
but I needed to fix the parse issues to build the non meta-ti based platform
I was working on (or disable your layer). I should have made that clearer
in the commit message.

2. There was a more complete patch submitted earlier in the day to address
> that:
> https://lists.yoctoproject.org/g/meta-ti/message/12625


Ok great. I was trying to check your mailing list but seems I was using a
old link from the README here
https://www.yoctoproject.org/pipermail/meta-ti/.

Peter.

>
>
> --
> Denys
>
>
> On Wed, Jan 29, 2020 at 06:19:23PM -0500, Denys Dmytriyenko via
> Lists.Yoctoproject.Org wrote:
> > NAK
> >
> > On Wed, Jan 29, 2020 at 11:15:55PM +0000, Peter Griffin wrote:
> > > This avoids the parsing errors, now that python2
> > > has been removed.
> > >
> > > Signed-off-by: Peter Griffin <peter.griffin@linaro.org>
> > > ---
> > >  recipes-kernel/jailhouse/jailhouse_git.bb | 2 +-
> > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > >
> > > diff --git a/recipes-kernel/jailhouse/jailhouse_git.bb
> b/recipes-kernel/jailhouse/jailhouse_git.bb
> > > index 4186dc3..89833e6 100644
> > > --- a/recipes-kernel/jailhouse/jailhouse_git.bb
> > > +++ b/recipes-kernel/jailhouse/jailhouse_git.bb
> > > @@ -29,7 +29,7 @@ RDEPENDS_${PN} += "\
> > >  S = "${WORKDIR}/git"
> > >
> > >  require jailhouse-arch.inc
> > > -inherit module pythonnative bash-completion deploy setuptools
> > > +inherit module python3native bash-completion deploy setuptools3
> > >
> > >  PACKAGE_ARCH = "${MACHINE_ARCH}"
> > >  COMPATIBLE_MACHINE = "(ti-soc)"
> > > --
> > > 2.7.4
> > >
>
> > 
>
>

[-- Attachment #2: Type: text/html, Size: 3535 bytes --]

Re: [meta-ti] [PATCH] jailhouse_git: update to inherit python3 classes

[Denys Dmytriyenko]

On Thu, Jan 30, 2020 at 09:14:47AM +0000, Peter Griffin wrote:
> Hi Denys,
> 
> On Thu, 30 Jan 2020 at 03:34, Denys Dmytriyenko <denys@ti.com> wrote:
> 
> > Let me elaborate:
> >
> > 1. Your patch is incomplete - it fixes parsing issues, but not build issues
> >
> 
> Thanks for the extra info, yes that is correct I didn't build any TI
> platform,
> but I needed to fix the parse issues to build the non meta-ti based platform
> I was working on (or disable your layer). I should have made that clearer
> in the commit message.
> 
> 2. There was a more complete patch submitted earlier in the day to address
> > that:
> > https://lists.yoctoproject.org/g/meta-ti/message/12625
> 
> 
> Ok great. I was trying to check your mailing list but seems I was using a
> old link from the README here
> https://www.yoctoproject.org/pipermail/meta-ti/.

Ah, good point! I need to update that. Thanks.


> Peter.
> 
> >
> >
> > --
> > Denys
> >
> >
> > On Wed, Jan 29, 2020 at 06:19:23PM -0500, Denys Dmytriyenko via
> > Lists.Yoctoproject.Org wrote:
> > > NAK
> > >
> > > On Wed, Jan 29, 2020 at 11:15:55PM +0000, Peter Griffin wrote:
> > > > This avoids the parsing errors, now that python2
> > > > has been removed.
> > > >
> > > > Signed-off-by: Peter Griffin <peter.griffin@linaro.org>
> > > > ---
> > > >  recipes-kernel/jailhouse/jailhouse_git.bb | 2 +-
> > > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > > >
> > > > diff --git a/recipes-kernel/jailhouse/jailhouse_git.bb
> > b/recipes-kernel/jailhouse/jailhouse_git.bb
> > > > index 4186dc3..89833e6 100644
> > > > --- a/recipes-kernel/jailhouse/jailhouse_git.bb
> > > > +++ b/recipes-kernel/jailhouse/jailhouse_git.bb
> > > > @@ -29,7 +29,7 @@ RDEPENDS_${PN} += "\
> > > >  S = "${WORKDIR}/git"
> > > >
> > > >  require jailhouse-arch.inc
> > > > -inherit module pythonnative bash-completion deploy setuptools
> > > > +inherit module python3native bash-completion deploy setuptools3
> > > >
> > > >  PACKAGE_ARCH = "${MACHINE_ARCH}"
> > > >  COMPATIBLE_MACHINE = "(ti-soc)"
> > > > --
> > > > 2.7.4
> > > >
> >
> > > 
> >
> >