diff mbox series

[meta-networking,v2] mctp: upgrade 1.1 -> 2.0

Message ID 20241017063626.2089926-1-chanh@os.amperecomputing.com
State Under Review
Headers show
Series [meta-networking,v2] mctp: upgrade 1.1 -> 2.0 | expand

Commit Message

Chanh Nguyen Oct. 17, 2024, 6:36 a.m. UTC
Signed-off-by: Chanh Nguyen <chanh@os.amperecomputing.com>
---
Change in v2:
  - Fix CI failed                              [Chanh]
---
 .../recipes-support/mctp/{mctp_1.1.bb => mctp_2.0.bb}       | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)
 rename meta-networking/recipes-support/mctp/{mctp_1.1.bb => mctp_2.0.bb} (93%)

Comments

Khem Raj Oct. 17, 2024, 8:46 p.m. UTC | #1
ok, now it fails for musl target see

https://errors.yoctoproject.org/Errors/Details/809802/

maybe you need to include linux/socket.h to get AF_MCTP address family
definition

On Wed, Oct 16, 2024 at 11:36 PM Chanh Nguyen
<chanh@os.amperecomputing.com> wrote:
>
> Signed-off-by: Chanh Nguyen <chanh@os.amperecomputing.com>
> ---
> Change in v2:
>   - Fix CI failed                              [Chanh]
> ---
>  .../recipes-support/mctp/{mctp_1.1.bb => mctp_2.0.bb}       | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)
>  rename meta-networking/recipes-support/mctp/{mctp_1.1.bb => mctp_2.0.bb} (93%)
>
> diff --git a/meta-networking/recipes-support/mctp/mctp_1.1.bb b/meta-networking/recipes-support/mctp/mctp_2.0.bb
> similarity index 93%
> rename from meta-networking/recipes-support/mctp/mctp_1.1.bb
> rename to meta-networking/recipes-support/mctp/mctp_2.0.bb
> index bf3961be4..62044e007 100644
> --- a/meta-networking/recipes-support/mctp/mctp_1.1.bb
> +++ b/meta-networking/recipes-support/mctp/mctp_2.0.bb
> @@ -5,7 +5,7 @@ LICENSE = "GPL-2.0-only"
>
>  LIC_FILES_CHKSUM = "file://LICENSE;md5=4cc91856b08b094b4f406a29dc61db21"
>
> -SRCREV = "a591ad885aa28f113cb45870b2fd5bce48378d6f"
> +SRCREV = "9986862f88fd6e6ca75e2a29236cf332d640e600"
>
>  SRC_URI = "git://github.com/CodeConstruct/mctp;branch=main;protocol=https"
>
> @@ -13,6 +13,10 @@ S = "${WORKDIR}/git"
>
>  inherit meson pkgconfig systemd
>
> +EXTRA_OEMESON = " \
> +    -Dtests=false \
> +"
> +
>  PACKAGECONFIG ??= " \
>      ${@bb.utils.filter('DISTRO_FEATURES', 'systemd', d)} \
>  "
> --
> 2.43.0
>
Chanh Nguyen Oct. 18, 2024, 7:18 a.m. UTC | #2
Hi Khem Raj,

Follow the error log, I tried to use the musl cross compiler 
(https://musl.cc/) to compiler https://github.com/CodeConstruct/mctp and 
report error to mctp maintainer. Jeremy fixed this error by commit 
https://github.com/CodeConstruct/mctp/commit/0629ec8976b1d56c17db51805f647bf35f4bfd3b 


Now I'll push the patch v3 with latest commit from mctp repo.

Khem Raj, How can I run CI on meta-openembedded manually? I would like 
to verify errors from CI failed before push patch. That will help us 
work on the patch quickly.

Thanks,
Chanh


On 18/10/2024 03:46, Khem Raj via lists.openembedded.org wrote:
> ok, now it fails for musl target see
> 
> https://errors.yoctoproject.org/Errors/Details/809802/
> 
> maybe you need to include linux/socket.h to get AF_MCTP address family
> definition
> 
> On Wed, Oct 16, 2024 at 11:36 PM Chanh Nguyen
> <chanh@os.amperecomputing.com> wrote:
>>
>> Signed-off-by: Chanh Nguyen <chanh@os.amperecomputing.com>
>> ---
>> Change in v2:
>>    - Fix CI failed                              [Chanh]
>> ---
>>   .../recipes-support/mctp/{mctp_1.1.bb => mctp_2.0.bb}       | 6 +++++-
>>   1 file changed, 5 insertions(+), 1 deletion(-)
>>   rename meta-networking/recipes-support/mctp/{mctp_1.1.bb => mctp_2.0.bb} (93%)
>>
>> diff --git a/meta-networking/recipes-support/mctp/mctp_1.1.bb b/meta-networking/recipes-support/mctp/mctp_2.0.bb
>> similarity index 93%
>> rename from meta-networking/recipes-support/mctp/mctp_1.1.bb
>> rename to meta-networking/recipes-support/mctp/mctp_2.0.bb
>> index bf3961be4..62044e007 100644
>> --- a/meta-networking/recipes-support/mctp/mctp_1.1.bb
>> +++ b/meta-networking/recipes-support/mctp/mctp_2.0.bb
>> @@ -5,7 +5,7 @@ LICENSE = "GPL-2.0-only"
>>
>>   LIC_FILES_CHKSUM = "file://LICENSE;md5=4cc91856b08b094b4f406a29dc61db21"
>>
>> -SRCREV = "a591ad885aa28f113cb45870b2fd5bce48378d6f"
>> +SRCREV = "9986862f88fd6e6ca75e2a29236cf332d640e600"
>>
>>   SRC_URI = "git://github.com/CodeConstruct/mctp;branch=main;protocol=https"
>>
>> @@ -13,6 +13,10 @@ S = "${WORKDIR}/git"
>>
>>   inherit meson pkgconfig systemd
>>
>> +EXTRA_OEMESON = " \
>> +    -Dtests=false \
>> +"
>> +
>>   PACKAGECONFIG ??= " \
>>       ${@bb.utils.filter('DISTRO_FEATURES', 'systemd', d)} \
>>   "
>> --
>> 2.43.0
>>
>>
>>
>> -=-=-=-=-=-=-=-=-=-=-=-
>> Links: You receive all messages sent to this group.
>> View/Reply Online (#112970): https://lists.openembedded.org/g/openembedded-devel/message/112970
>> Mute This Topic: https://lists.openembedded.org/mt/109059332/9027231
>> Group Owner: openembedded-devel+owner@lists.openembedded.org
>> Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub [chanh@os.amperecomputing.com]
>> -=-=-=-=-=-=-=-=-=-=-=-
>>
diff mbox series

Patch

diff --git a/meta-networking/recipes-support/mctp/mctp_1.1.bb b/meta-networking/recipes-support/mctp/mctp_2.0.bb
similarity index 93%
rename from meta-networking/recipes-support/mctp/mctp_1.1.bb
rename to meta-networking/recipes-support/mctp/mctp_2.0.bb
index bf3961be4..62044e007 100644
--- a/meta-networking/recipes-support/mctp/mctp_1.1.bb
+++ b/meta-networking/recipes-support/mctp/mctp_2.0.bb
@@ -5,7 +5,7 @@  LICENSE = "GPL-2.0-only"
 
 LIC_FILES_CHKSUM = "file://LICENSE;md5=4cc91856b08b094b4f406a29dc61db21"
 
-SRCREV = "a591ad885aa28f113cb45870b2fd5bce48378d6f"
+SRCREV = "9986862f88fd6e6ca75e2a29236cf332d640e600"
 
 SRC_URI = "git://github.com/CodeConstruct/mctp;branch=main;protocol=https"
 
@@ -13,6 +13,10 @@  S = "${WORKDIR}/git"
 
 inherit meson pkgconfig systemd
 
+EXTRA_OEMESON = " \
+    -Dtests=false \
+"
+
 PACKAGECONFIG ??= " \
     ${@bb.utils.filter('DISTRO_FEATURES', 'systemd', d)} \
 "