Re: [PATCH 0/2] bind update req. new pkg in core

From: akuster808 <akuster808@gmail.com>
To: Khem Raj <raj.khem@gmail.com>
Cc: Patches and discussions about the oe-core layer
	<openembedded-core@lists.openembedded.org>
Subject: Re: [PATCH 0/2] bind update req. new pkg in core
Date: Mon, 21 May 2018 21:05:31 -0700	[thread overview]
Message-ID: <04088507-78ee-10e5-da79-e11d90384c2b@gmail.com> (raw)
In-Reply-To: <CAMKF1so+vefznkou750crbUntgOhzfJ_RXSn+E91bEb4HbjTUA@mail.gmail.com>

On 05/21/2018 07:25 PM, Khem Raj wrote:
> On Mon, May 21, 2018 at 9:30 PM, akuster808 <akuster808@gmail.com> wrote:
>>
>> On 05/21/2018 06:06 PM, Khem Raj wrote:
>>
>>
>> On Mon, May 21, 2018 at 3:44 PM Armin Kuster <akuster808@gmail.com> wrote:
>>> With this update, the python-ply package is required.
>>> I copied the one from meta-python to core. Once this hits master,
>>> I will send a patch to remove the same recipe from
>>> meta-python.
>>
>> Can this be turned on/off using packageconfig ?
>>
>> Yes we can and it is already in the PACKAGECONFIG options.
>>
>> If we do disable python, then the python binaries that are currently being
>> installed will not. Someone decided: dnssec-coverage dnssec-checkds  where
>> important to install in the current Bind solution.   A new one is being
>> added; dnssec-keymgr. If we don't want dnssec support, thats easy to
>> exclude.
>>
> these all seem to be additional utilities so probably should be
> packaged into a separate package
> may be bind-utils
there is already a bind-utils.  bind-dnssec-utils may be more appropriate.
it is the if/conditional when python is enabled to do all the right
things that is going to get ugly.

>
>> BTW, pthon-ply  exists in 4 other layers besides meta-python and  at
>> different versions.
> thats less than ideal, and probably should be consolidated, moving to
> oe-core probably will not
> solve that problem
>
>> let me know which way you want to go.
> I dont have any strong opinion either way is fine.

I will wait to see who else may want to weight in.

thank for the input.

- armin
>
>> - armin
>>
>>
>>
>>
>>
>> If so then it would be ideal
>>
>> If not then please send the removal patches for meta-python regardless so it
>> can be tested together
>>>
>>>
>>> Armin Kuster (2):
>>>   bind: update to 9.12.1
>>>   python3-ply: add package needed by bind 9.12 update
>>>
>>>  ...0001-build-use-pkg-config-to-find-libxml2.patch | 13 +++---
>>>  ...-gen.c-extend-DIRNAMESIZE-from-256-to-512.patch | 13 +++---
>>>  .../0001-lib-dns-gen.c-fix-too-long-error.patch    | 13 +++---
>>>  .../bind/bind/bind-confgen-build-unix.o-once.patch | 48
>>> ----------------------
>>>  ...-searching-for-json-headers-searches-sysr.patch | 13 +++---
>>>  .../bind/bind/dont-test-on-host.patch              | 17 --------
>>>  .../use-python3-and-fix-install-lib-path.patch     | 36 ----------------
>>>  .../bind/{bind_9.10.6.bb => bind_9.12.1.bb}        | 21 ++++------
>>>  meta/recipes-devtools/python/python-ply.inc        | 18 ++++++++
>>>  meta/recipes-devtools/python/python-ply_3.11.bb    |  2 +
>>>  10 files changed, 49 insertions(+), 145 deletions(-)
>>>  delete mode 100644
>>> meta/recipes-connectivity/bind/bind/bind-confgen-build-unix.o-once.patch
>>>  delete mode 100644
>>> meta/recipes-connectivity/bind/bind/dont-test-on-host.patch
>>>  delete mode 100644
>>> meta/recipes-connectivity/bind/bind/use-python3-and-fix-install-lib-path.patch
>>>  rename meta/recipes-connectivity/bind/{bind_9.10.6.bb => bind_9.12.1.bb}
>>> (86%)
>>>  create mode 100644 meta/recipes-devtools/python/python-ply.inc
>>>  create mode 100644 meta/recipes-devtools/python/python-ply_3.11.bb
>>>
>>> --
>>> 2.7.4
>>>
>>> --
>>> _______________________________________________
>>> Openembedded-core mailing list
>>> Openembedded-core@lists.openembedded.org
>>> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>>