[oe] [meta-python][rocko][PATCH 2/2] python-twisted: restore RDEPENDS on -news for py2

[Derek Straka]

Yeah.  Agreed.  These commits are me cleaning up the RDEPENDS to work with
minimal python installs.  Meta-python has been pretty bad about setting the
dependencies.  I just missed the news package.  Thanks.

On Dec 15, 2017 2:22 PM, "Mark Asselstine" <mark.asselstine at windriver.com>
wrote:

> No problem. Unfortunately your commit log was thin on information so I
> had to make some assumptions as to what you were doing in the commit.
> I could have easily misinterpreted.
>
> On Fri, Dec 15, 2017 at 2:19 PM, Derek Straka <derek at asterius.io> wrote:
> > Sure.  I wasn't clear and even then I misread the 1/2 patch and thought I
> > removed news from core and not the base package RDEPENDS.
> >
> > On Dec 15, 2017 2:08 PM, "Mark Asselstine" <
> mark.asselstine at windriver.com>
> > wrote:
> >
> > The RDEPENDS are not for core, these are RDEPENDS_${PN}, ie. if you
> > don't know what parts you want you install ${PN} and get everything.
> > We can straighten this up though if my thoughts are incorrect. I have
> > to send a V2 anyway as I was hasty and missed the cherrypick of a
> > bunch of dependencies you added for automat and others.
> >
> > Mark
> >
> > On Fri, Dec 15, 2017 at 1:32 PM, Derek Straka <derek at asterius.io> wrote:
> >> Is it actually required for core?  I thought I went through and verified
> >> that news was a stand alone module.
> >>
> >> On Dec 15, 2017 12:46 PM, "Mark Asselstine" <
> mark.asselstine at windriver.com
> >>
> >> wrote:
> >>
> >>> Although this plugin has not yet been ported to py3 it is available in
> >>> py2. The previous commit isn't clear about what is being addressed but
> >>> I assume it is:
> >>>
> >>> nothing provides python3-twisted-news needed by
> > python3-twisted-17.9.0-r0.
> >>> corei7_64
> >>>
> >>> For py2 this package is valid and so we should maintain the RDEPENDS,
> >>> restore this RDEPENDS for py2.
> >>>
> >>> Signed-off-by: Mark Asselstine <mark.asselstine at windriver.com>
> >>> ---
> >>>  meta-python/recipes-devtools/python/python-twisted_17.9.0.bb | 6
> ++++++
> >>>  1 file changed, 6 insertions(+)
> >>>
> >>> diff --git a/meta-python/recipes-devtools/python/python-
> twisted_17.9.0.bb
> >>> b/meta-python/recipes-devtools/python/python-twisted_17.9.0.bb
> >>> index 2be1709..7fc8c65 100644
> >>> --- a/meta-python/recipes-devtools/python/python-twisted_17.9.0.bb
> >>> +++ b/meta-python/recipes-devtools/python/python-twisted_17.9.0.bb
> >>> @@ -2,3 +2,9 @@ inherit pypi setuptools
> >>>  require python-twisted.inc
> >>>
> >>>  RDEPENDS_${PN}-core += "${PYTHON_PN}-contextlib"
> >>> +
> >>> +# Not yet ported to py3 yet available in py2
> >>> +# See src/twisted/python/_setup.py -- notPortedModules
> >>> +RDEPENDS_${PN} += "\
> >>> +    ${PN}-news \
> >>> +    "
> >>> --
> >>> 2.7.4
> >>>
> >>> --
> >>> _______________________________________________
> >>> Openembedded-devel mailing list
> >>> Openembedded-devel at lists.openembedded.org
> >>> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
> >>>
> >> --
> >> _______________________________________________
> >> Openembedded-devel mailing list
> >> Openembedded-devel at lists.openembedded.org
> >> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
> > --
> > _______________________________________________
> > Openembedded-devel mailing list
> > Openembedded-devel at lists.openembedded.org
> > http://lists.openembedded.org/mailman/listinfo/openembedded-devel
>