diff mbox series

[meta-oe] msgpack-c: upgrade 4.0.0 -> 6.0.0

Message ID 20231127133808.1396531-1-alperyasinak1@gmail.com
State Accepted
Headers show
Series [meta-oe] msgpack-c: upgrade 4.0.0 -> 6.0.0 | expand

Commit Message

alperak Nov. 27, 2023, 1:38 p.m. UTC
- GTest should be OFF by default. -> https://github.com/msgpack/msgpack-c/issues/1066

Changelog:

- Remove C++ requirement if test is disabled (#1055)
- Change CMake package name of C library to msgpack-c (#1053)
- Unified all C package, library, cmake, tarball name become msgpack-c.
- Add additional address sanitizer for CI. (#1023)
- Change CMake package name of C library to msgpackc (#1044, #1049)

Signed-off-by: alperak <alperyasinak1@gmail.com>
---
 .../msgpack/{msgpack-c_4.0.0.bb => msgpack-c_6.0.0.bb}      | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
 rename meta-oe/recipes-devtools/msgpack/{msgpack-c_4.0.0.bb => msgpack-c_6.0.0.bb} (86%)

Comments

Khem Raj Nov. 27, 2023, 6:05 p.m. UTC | #1
tmate fails to build with this upgrade

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

On Mon, Nov 27, 2023 at 5:38 AM Alper Ak <alperyasinak1@gmail.com> wrote:
>
> - GTest should be OFF by default. -> https://github.com/msgpack/msgpack-c/issues/1066
>
> Changelog:
>
> - Remove C++ requirement if test is disabled (#1055)
> - Change CMake package name of C library to msgpack-c (#1053)
> - Unified all C package, library, cmake, tarball name become msgpack-c.
> - Add additional address sanitizer for CI. (#1023)
> - Change CMake package name of C library to msgpackc (#1044, #1049)
>
> Signed-off-by: alperak <alperyasinak1@gmail.com>
> ---
>  .../msgpack/{msgpack-c_4.0.0.bb => msgpack-c_6.0.0.bb}      | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
>  rename meta-oe/recipes-devtools/msgpack/{msgpack-c_4.0.0.bb => msgpack-c_6.0.0.bb} (86%)
>
> diff --git a/meta-oe/recipes-devtools/msgpack/msgpack-c_4.0.0.bb b/meta-oe/recipes-devtools/msgpack/msgpack-c_6.0.0.bb
> similarity index 86%
> rename from meta-oe/recipes-devtools/msgpack/msgpack-c_4.0.0.bb
> rename to meta-oe/recipes-devtools/msgpack/msgpack-c_6.0.0.bb
> index c6a6ac467..40d997a52 100644
> --- a/meta-oe/recipes-devtools/msgpack/msgpack-c_4.0.0.bb
> +++ b/meta-oe/recipes-devtools/msgpack/msgpack-c_6.0.0.bb
> @@ -7,14 +7,14 @@ LIC_FILES_CHKSUM = "file://NOTICE;md5=7a858c074723608e08614061dc044352 \
>                      file://LICENSE_1_0.txt;md5=e4224ccaecb14d942c71d31bef20d78c \
>                     "
>
> -PV .= "+git${SRCPV}"
> -
>  SRC_URI = "git://github.com/msgpack/msgpack-c;branch=c_master;protocol=https \
>             "
> -SRCREV = "a9a48cea3a78ba661ee8096b5dab456361b0ff23"
> +SRCREV = "8160ede5e20fd3019a77eea46d9c72cf6163f802"
>
>  S = "${WORKDIR}/git"
>
>  inherit cmake pkgconfig
>
> +EXTRA_OECMAKE += "-DMSGPACK_BUILD_TESTS=off"
> +
>  BBCLASSEXTEND += "native nativesdk"
> --
> 2.25.1
>
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#107059): https://lists.openembedded.org/g/openembedded-devel/message/107059
> Mute This Topic: https://lists.openembedded.org/mt/102828614/1997914
> Group Owner: openembedded-devel+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub [raj.khem@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
alperak Nov. 28, 2023, 9:57 a.m. UTC | #2
I sent a patch request for tmate to fix this issue:

https://lists.openembedded.org/g/openembedded-devel/message/107095

Khem Raj <raj.khem@gmail.com>, 27 Kas 2023 Pzt, 21:06 tarihinde şunu yazdı:

> tmate fails to build with this upgrade
>
> https://errors.yoctoproject.org/Errors/Details/743897/
>
> On Mon, Nov 27, 2023 at 5:38 AM Alper Ak <alperyasinak1@gmail.com> wrote:
> >
> > - GTest should be OFF by default. ->
> https://github.com/msgpack/msgpack-c/issues/1066
> >
> > Changelog:
> >
> > - Remove C++ requirement if test is disabled (#1055)
> > - Change CMake package name of C library to msgpack-c (#1053)
> > - Unified all C package, library, cmake, tarball name become msgpack-c.
> > - Add additional address sanitizer for CI. (#1023)
> > - Change CMake package name of C library to msgpackc (#1044, #1049)
> >
> > Signed-off-by: alperak <alperyasinak1@gmail.com>
> > ---
> >  .../msgpack/{msgpack-c_4.0.0.bb => msgpack-c_6.0.0.bb}      | 6 +++---
> >  1 file changed, 3 insertions(+), 3 deletions(-)
> >  rename meta-oe/recipes-devtools/msgpack/{msgpack-c_4.0.0.bb =>
> msgpack-c_6.0.0.bb} (86%)
> >
> > diff --git a/meta-oe/recipes-devtools/msgpack/msgpack-c_4.0.0.bb
> b/meta-oe/recipes-devtools/msgpack/msgpack-c_6.0.0.bb
> > similarity index 86%
> > rename from meta-oe/recipes-devtools/msgpack/msgpack-c_4.0.0.bb
> > rename to meta-oe/recipes-devtools/msgpack/msgpack-c_6.0.0.bb
> > index c6a6ac467..40d997a52 100644
> > --- a/meta-oe/recipes-devtools/msgpack/msgpack-c_4.0.0.bb
> > +++ b/meta-oe/recipes-devtools/msgpack/msgpack-c_6.0.0.bb
> > @@ -7,14 +7,14 @@ LIC_FILES_CHKSUM =
> "file://NOTICE;md5=7a858c074723608e08614061dc044352 \
> >
> file://LICENSE_1_0.txt;md5=e4224ccaecb14d942c71d31bef20d78c \
> >                     "
> >
> > -PV .= "+git${SRCPV}"
> > -
> >  SRC_URI = "git://
> github.com/msgpack/msgpack-c;branch=c_master;protocol=https \
> >             "
> > -SRCREV = "a9a48cea3a78ba661ee8096b5dab456361b0ff23"
> > +SRCREV = "8160ede5e20fd3019a77eea46d9c72cf6163f802"
> >
> >  S = "${WORKDIR}/git"
> >
> >  inherit cmake pkgconfig
> >
> > +EXTRA_OECMAKE += "-DMSGPACK_BUILD_TESTS=off"
> > +
> >  BBCLASSEXTEND += "native nativesdk"
> > --
> > 2.25.1
> >
> >
> > -=-=-=-=-=-=-=-=-=-=-=-
> > Links: You receive all messages sent to this group.
> > View/Reply Online (#107059):
> https://lists.openembedded.org/g/openembedded-devel/message/107059
> > Mute This Topic: https://lists.openembedded.org/mt/102828614/1997914
> > Group Owner: openembedded-devel+owner@lists.openembedded.org
> > Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub [
> raj.khem@gmail.com]
> > -=-=-=-=-=-=-=-=-=-=-=-
> >
>
Khem Raj Nov. 28, 2023, 11:55 p.m. UTC | #3
On Mon, 27 Nov 2023 16:38:08 +0300, alperak wrote:
> - GTest should be OFF by default. -> https://github.com/msgpack/msgpack-c/issues/1066
> 
> Changelog:
> 
> - Remove C++ requirement if test is disabled (#1055)
> - Change CMake package name of C library to msgpack-c (#1053)
> - Unified all C package, library, cmake, tarball name become msgpack-c.
> - Add additional address sanitizer for CI. (#1023)
> - Change CMake package name of C library to msgpackc (#1044, #1049)
> 
> [...]

Applied, thanks!

[1/1] msgpack-c: upgrade 4.0.0 -> 6.0.0
      commit: fe0d5e65cd8d9fef9d4e3043dfb86f747c5f14fb

Best regards,
diff mbox series

Patch

diff --git a/meta-oe/recipes-devtools/msgpack/msgpack-c_4.0.0.bb b/meta-oe/recipes-devtools/msgpack/msgpack-c_6.0.0.bb
similarity index 86%
rename from meta-oe/recipes-devtools/msgpack/msgpack-c_4.0.0.bb
rename to meta-oe/recipes-devtools/msgpack/msgpack-c_6.0.0.bb
index c6a6ac467..40d997a52 100644
--- a/meta-oe/recipes-devtools/msgpack/msgpack-c_4.0.0.bb
+++ b/meta-oe/recipes-devtools/msgpack/msgpack-c_6.0.0.bb
@@ -7,14 +7,14 @@  LIC_FILES_CHKSUM = "file://NOTICE;md5=7a858c074723608e08614061dc044352 \
                     file://LICENSE_1_0.txt;md5=e4224ccaecb14d942c71d31bef20d78c \
                    "
 
-PV .= "+git${SRCPV}"
-
 SRC_URI = "git://github.com/msgpack/msgpack-c;branch=c_master;protocol=https \
            "
-SRCREV = "a9a48cea3a78ba661ee8096b5dab456361b0ff23"
+SRCREV = "8160ede5e20fd3019a77eea46d9c72cf6163f802"
 
 S = "${WORKDIR}/git"
 
 inherit cmake pkgconfig
 
+EXTRA_OECMAKE += "-DMSGPACK_BUILD_TESTS=off" 
+
 BBCLASSEXTEND += "native nativesdk"