[meta-intel] [PATCH 1/3] dpdk-dev-libibverbs: fix do_fetch failure in case of multilib

[Bruce Ashfield]

On Wed, Jan 3, 2018 at 8:56 PM, ChenQi <Qi.Chen at windriver.com> wrote:
> On 01/04/2018 03:02 AM, Cal Sullivan wrote:
>>
>> In actually testing this patch, I noticed that I was getting a fetcher
>> warning. After a little investigation I found that
>> https://github.com/Mellanox/dpdk-dev-libibverbs no longer exists and we're
>> only getting this thanks to the YP mirroring. Do you know of any other
>> location we could point to?
>>
>> Thanks,
>> Cal
>>
>
> Hi Cal,
>
> I don't know any other location. It seems that this project no longer
> exists.
> Could you directly point the SRC_URI to yocto download location?
>
> There's a libibverbs recipe in meta-cloud-services. When I started to solve
> this conflict problem, there were two choices in front of me.
> 1) Use 'virtual/libibverbs' approach.
> 2) Drop one recipe, put the other in a common local (e.g. meta-oe layer),
> and make meta-dpdk and meta-cloud-services depend on that layer.
>
> I finally chose the first one, because
> 1) it would definitely bring in no regression problem. Users could choose
> libibverbs or dpdk-dev-libibverbs as they like.
> 2) I was not sure why dpdk-dev-libibverbs recipe was created in the first
> place.
>
> Anyway, for this SRC_URI problem, if we couldn't be sure that
> dpdk-dev-libibverbs is not necessary, let's just point the SRC_URI to yocto
> download location; otherwise, maybe the second approach above could be used.

I'm going to have to carry a different recipe in meta-cloud-services
regardless, due
to version and feature requirements that we have in the system level use cases.

So any solution that involves dropping the recipe there and moving things to a
single location, won't fly.

Cheers,

Bruce

>
> Best Regards,
> Chen Qi
>
>
>
>> On 01/02/2018 05:27 PM, Chen Qi wrote:
>>>
>>> Fix to correctly set SRC_URI and S to avoid do_fetch failure in case of
>>> multilib.
>>>
>>> Signed-off-by: Chen Qi <Qi.Chen at windriver.com>
>>> ---
>>> .../dpdk-dev-libibverbs/dpdk-dev-libibverbs_1.2.1-3.4-2.0.0.0.bb | 8
>>> ++++----
>>>   1 file changed, 4 insertions(+), 4 deletions(-)
>>>
>>> diff --git
>>> a/recipes-extended/dpdk-dev-libibverbs/dpdk-dev-libibverbs_1.2.1-3.4-2.0.0.0.bb
>>> b/recipes-extended/dpdk-dev-libibverbs/dpdk-dev-libibverbs_1.2.1-3.4-2.0.0.0.bb
>>> index e40c63b..9118494 100644
>>> ---
>>> a/recipes-extended/dpdk-dev-libibverbs/dpdk-dev-libibverbs_1.2.1-3.4-2.0.0.0.bb
>>> +++
>>> b/recipes-extended/dpdk-dev-libibverbs/dpdk-dev-libibverbs_1.2.1-3.4-2.0.0.0.bb
>>> @@ -3,7 +3,7 @@ HOMEPAGE =
>>> "https://github.com/Mellanox/dpdk-dev-libibverbs"
>>>   LICENSE = "GPLv2"
>>>   LIC_FILES_CHKSUM =
>>> "file://COPYING;md5=7c557f27dd795ba77cc419dddc656b51"
>>>   -SRC_URI =
>>> "https://github.com/Mellanox/dpdk-dev-libibverbs/archive/libibverbs-${PV}.tar.gz;name=${PN}
>>> \
>>> +SRC_URI =
>>> "https://github.com/Mellanox/dpdk-dev-libibverbs/archive/libibverbs-${PV}.tar.gz
>>> \
>>>              file://init_c.patch \
>>>              file://0001-Fix-build-with-clang.patch \
>>> file://0002-typecast-enum-to-int-before-comparison.patch \
>>> @@ -11,8 +11,8 @@ SRC_URI =
>>> "https://github.com/Mellanox/dpdk-dev-libibverbs/archive/libibverbs-${
>>>              file://0004-Fix-clang-warnings.patch \
>>>              "
>>>   -SRC_URI[dpdk-dev-libibverbs.md5sum] =
>>> "65234ee278eb437a7069326f37cd4d86"
>>> -SRC_URI[dpdk-dev-libibverbs.sha256sum] =
>>> "a6471515556cb8d10ad471bb7efb8cf760b248a28aceb57d4534d50d572f56cd"
>>> +SRC_URI[md5sum] = "65234ee278eb437a7069326f37cd4d86"
>>> +SRC_URI[sha256sum] =
>>> "a6471515556cb8d10ad471bb7efb8cf760b248a28aceb57d4534d50d572f56cd"
>>>     # A machine needs to enable this using:
>>>   # COMPATIBLE_MACHINE_pn-dpdk-dev-libibverbs = "<machine name>"
>>> @@ -20,7 +20,7 @@ SRC_URI[dpdk-dev-libibverbs.sha256sum] =
>>> "a6471515556cb8d10ad471bb7efb8cf760b248
>>>   COMPATIBLE_MACHINE = "null"
>>>   COMPATIBLE_HOST_libc-musl_class-target = "null"
>>>   -S = "${WORKDIR}/${PN}-libibverbs-${PV}"
>>> +S = "${WORKDIR}/dpdk-dev-libibverbs-libibverbs-${PV}"
>>>   COMPATIBLE_HOST = '(i.86|x86_64).*-linux'
>>>   DEPENDS = "libnl"
>>
>>
>>
>
> --
> _______________________________________________
> meta-intel mailing list
> meta-intel at yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-intel



-- 
"Thou shalt not follow the NULL pointer, for chaos and madness await
thee at its end"