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

[Derek Straka]

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