Message ID | 20230704102307.3654880-1-ross.burton@arm.com |
---|---|
State | Accepted, archived |
Commit | d0cc8bb5047fe843532bcba37ca48d94c9749daf |
Headers | show |
Series | libjpeg-turbo: upgrade to 3.0.0 | expand |
I wonder of this failure is related to this https://errors.yoctoproject.org/Errors/Details/711770/ On Tue, Jul 4, 2023 at 3:23 AM Ross Burton <ross.burton@arm.com> wrote: > > From: Ross Burton <ross.burton@arm.com> > > Remove 0001-libjpeg-turbo-fix-package_qa-error.patch, doesn't appear to > be needed anymore. > > Use the full upstream license for LICENSE, and change LIC_FILES_CHKSUM > to point at just the license overview in LICENSE.md. > > Signed-off-by: Ross Burton <ross.burton@arm.com> > --- > ...1-libjpeg-turbo-fix-package_qa-error.patch | 32 ------------------- > ...urbo_2.1.5.1.bb => libjpeg-turbo_3.0.0.bb} | 14 +++----- > 2 files changed, 5 insertions(+), 41 deletions(-) > delete mode 100644 meta/recipes-graphics/jpeg/files/0001-libjpeg-turbo-fix-package_qa-error.patch > rename meta/recipes-graphics/jpeg/{libjpeg-turbo_2.1.5.1.bb => libjpeg-turbo_3.0.0.bb} (83%) > > diff --git a/meta/recipes-graphics/jpeg/files/0001-libjpeg-turbo-fix-package_qa-error.patch b/meta/recipes-graphics/jpeg/files/0001-libjpeg-turbo-fix-package_qa-error.patch > deleted file mode 100644 > index fab5109f831..00000000000 > --- a/meta/recipes-graphics/jpeg/files/0001-libjpeg-turbo-fix-package_qa-error.patch > +++ /dev/null > @@ -1,32 +0,0 @@ > -From 5cf847b5bef8dc3f9f89bd09dd5af4e6603f393c Mon Sep 17 00:00:00 2001 > -From: Changqing Li <changqing.li@windriver.com> > -Date: Mon, 27 Aug 2018 16:10:55 +0800 > -Subject: [PATCH] libjpeg-turbo: fix package_qa error > - > -Fix package qa errors like below: > -libjpeg.so.62.3.0 contains probably-redundant RPATH /usr/lib [useless-rpaths] > -usr/bin/cjpeg contains probably-redundant RPATH /usr/lib > - > -Upstream-Status: Inappropriate [oe-specific] > - > -Signed-off-by: Changqing Li <changqing.li@windriver.com> > -Signed-off-by: Anuj Mittal <anuj.mittal@intel.com> > ---- > - CMakeLists.txt | 4 ---- > - 1 file changed, 4 deletions(-) > - > -diff --git a/CMakeLists.txt b/CMakeLists.txt > -index 2bc3458..ea3041e 100644 > ---- a/CMakeLists.txt > -+++ b/CMakeLists.txt > -@@ -191,10 +191,6 @@ endif() > - report_option(ENABLE_SHARED "Shared libraries") > - report_option(ENABLE_STATIC "Static libraries") > - > --if(ENABLE_SHARED) > -- set(CMAKE_INSTALL_RPATH ${CMAKE_INSTALL_FULL_LIBDIR}) > --endif() > -- > - if(WITH_JPEG8 OR WITH_JPEG7) > - set(WITH_ARITH_ENC 1) > - set(WITH_ARITH_DEC 1) > diff --git a/meta/recipes-graphics/jpeg/libjpeg-turbo_2.1.5.1.bb b/meta/recipes-graphics/jpeg/libjpeg-turbo_3.0.0.bb > similarity index 83% > rename from meta/recipes-graphics/jpeg/libjpeg-turbo_2.1.5.1.bb > rename to meta/recipes-graphics/jpeg/libjpeg-turbo_3.0.0.bb > index e086830c021..146d80008c5 100644 > --- a/meta/recipes-graphics/jpeg/libjpeg-turbo_2.1.5.1.bb > +++ b/meta/recipes-graphics/jpeg/libjpeg-turbo_3.0.0.bb > @@ -2,19 +2,15 @@ SUMMARY = "Hardware accelerated JPEG compression/decompression library" > DESCRIPTION = "libjpeg-turbo is a derivative of libjpeg that uses SIMD instructions (MMX, SSE2, NEON) to accelerate baseline JPEG compression and decompression" > HOMEPAGE = "http://libjpeg-turbo.org/" > > -LICENSE = "BSD-3-Clause" > -LIC_FILES_CHKSUM = "file://cdjpeg.h;endline=13;md5=8a61af33cc1c681cd5cc297150bbb5bd \ > - file://jpeglib.h;endline=16;md5=52b5eaade8d5b6a452a7693dfe52c084 \ > - file://djpeg.c;endline=11;md5=510b386442ab6a27ee241fc5669bc5ea \ > - " > +LICENSE = "IJG & BSD-3-Clause & Zlib" > +LIC_FILES_CHKSUM = "file://LICENSE.md;md5=2a8e0d8226a102f07ab63ed7fd6ce155" > + > DEPENDS:append:x86-64:class-target = " nasm-native" > DEPENDS:append:x86:class-target = " nasm-native" > > -SRC_URI = "${SOURCEFORGE_MIRROR}/${BPN}/${BPN}-${PV}.tar.gz \ > - file://0001-libjpeg-turbo-fix-package_qa-error.patch \ > - " > +SRC_URI = "${SOURCEFORGE_MIRROR}/${BPN}/${BPN}-${PV}.tar.gz" > > -SRC_URI[sha256sum] = "2fdc3feb6e9deb17adec9bafa3321419aa19f8f4e5dea7bf8486844ca22207bf" > +SRC_URI[sha256sum] = "c77c65fcce3d33417b2e90432e7a0eb05f59a7fff884022a9d931775d583bfaa" > UPSTREAM_CHECK_URI = "http://sourceforge.net/projects/libjpeg-turbo/files/" > UPSTREAM_CHECK_REGEX = "/libjpeg-turbo/files/(?P<pver>(\d+[\.\-_]*)+)/" > > -- > 2.34.1 > > > -=-=-=-=-=-=-=-=-=-=-=- > Links: You receive all messages sent to this group. > View/Reply Online (#183821): https://lists.openembedded.org/g/openembedded-core/message/183821 > Mute This Topic: https://lists.openembedded.org/mt/99943765/1997914 > Group Owner: openembedded-core+owner@lists.openembedded.org > Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [raj.khem@gmail.com] > -=-=-=-=-=-=-=-=-=-=-=- >
On 5 Jul 2023, at 15:48, Khem Raj via lists.openembedded.org <raj.khem=gmail.com@lists.openembedded.org> wrote: > > I wonder of this failure is related to this > https://errors.yoctoproject.org/Errors/Details/711770/ TOPDIR/build/tmp/work/cortexa72-cortexa53-crypto-mx8-yoe-linux/poppler/23.04.0-r0/recipe-sysroot/usr/include/jpeglib.h:968:52: error: unknown type name 'FILE' 968 | EXTERN(void) jpeg_stdio_dest(j_compress_ptr cinfo, FILE *outfile); | ^ Quite possibly, but poppler should be including stdio.h: “”" Applications using the JPEG library should include the header file jpeglib.h to obtain declarations of data types and routines. Before including jpeglib.h, include system headers that define at least the typedefs FILE and size_t. On ANSI-conforming systems, including <stdio.h> is sufficient; on older Unix systems, you may need <sys/types.h> to define size_t. “”” Looks like you want to backport https://gitlab.freedesktop.org/poppler/poppler/-/commit/2986f06c7cc9d64a506ebe861b8bf38f73386e86 to poppler. Ross
On Wed, Jul 5, 2023 at 8:19 AM Ross Burton <Ross.Burton@arm.com> wrote: > On 5 Jul 2023, at 15:48, Khem Raj via lists.openembedded.org <raj.khem= > gmail.com@lists.openembedded.org> wrote: > > > > I wonder of this failure is related to this > > https://errors.yoctoproject.org/Errors/Details/711770/ > > TOPDIR/build/tmp/work/cortexa72-cortexa53-crypto-mx8-yoe-linux/poppler/23.04.0-r0/recipe-sysroot/usr/include/jpeglib.h:968:52: > error: unknown type name 'FILE' > 968 | EXTERN(void) jpeg_stdio_dest(j_compress_ptr cinfo, FILE *outfile); > | ^ > > > Quite possibly, but poppler should be including stdio.h: > > “”" > Applications using the JPEG library should include the header file > jpeglib.h > to obtain declarations of data types and routines. Before including > jpeglib.h, include system headers that define at least the typedefs FILE > and > size_t. On ANSI-conforming systems, including <stdio.h> is sufficient; on > older Unix systems, you may need <sys/types.h> to define size_t. > “”” > > Looks like you want to backport > https://gitlab.freedesktop.org/poppler/poppler/-/commit/2986f06c7cc9d64a506ebe861b8bf38f73386e86 > to poppler. Right that seems to be the fix. So should I be expecting a patch :) > > > Ross
On 5 Jul 2023, at 17:42, Khem Raj <raj.khem@gmail.com> wrote: > Looks like you want to backport https://gitlab.freedesktop.org/poppler/poppler/-/commit/2986f06c7cc9d64a506ebe861b8bf38f73386e86 to poppler. > > Right that seems to be the fix. So should I be expecting a patch :) I suspect you’re actually also testing a new clang and that is to blame, but patch on the list :) Ross
diff --git a/meta/recipes-graphics/jpeg/files/0001-libjpeg-turbo-fix-package_qa-error.patch b/meta/recipes-graphics/jpeg/files/0001-libjpeg-turbo-fix-package_qa-error.patch deleted file mode 100644 index fab5109f831..00000000000 --- a/meta/recipes-graphics/jpeg/files/0001-libjpeg-turbo-fix-package_qa-error.patch +++ /dev/null @@ -1,32 +0,0 @@ -From 5cf847b5bef8dc3f9f89bd09dd5af4e6603f393c Mon Sep 17 00:00:00 2001 -From: Changqing Li <changqing.li@windriver.com> -Date: Mon, 27 Aug 2018 16:10:55 +0800 -Subject: [PATCH] libjpeg-turbo: fix package_qa error - -Fix package qa errors like below: -libjpeg.so.62.3.0 contains probably-redundant RPATH /usr/lib [useless-rpaths] -usr/bin/cjpeg contains probably-redundant RPATH /usr/lib - -Upstream-Status: Inappropriate [oe-specific] - -Signed-off-by: Changqing Li <changqing.li@windriver.com> -Signed-off-by: Anuj Mittal <anuj.mittal@intel.com> ---- - CMakeLists.txt | 4 ---- - 1 file changed, 4 deletions(-) - -diff --git a/CMakeLists.txt b/CMakeLists.txt -index 2bc3458..ea3041e 100644 ---- a/CMakeLists.txt -+++ b/CMakeLists.txt -@@ -191,10 +191,6 @@ endif() - report_option(ENABLE_SHARED "Shared libraries") - report_option(ENABLE_STATIC "Static libraries") - --if(ENABLE_SHARED) -- set(CMAKE_INSTALL_RPATH ${CMAKE_INSTALL_FULL_LIBDIR}) --endif() -- - if(WITH_JPEG8 OR WITH_JPEG7) - set(WITH_ARITH_ENC 1) - set(WITH_ARITH_DEC 1) diff --git a/meta/recipes-graphics/jpeg/libjpeg-turbo_2.1.5.1.bb b/meta/recipes-graphics/jpeg/libjpeg-turbo_3.0.0.bb similarity index 83% rename from meta/recipes-graphics/jpeg/libjpeg-turbo_2.1.5.1.bb rename to meta/recipes-graphics/jpeg/libjpeg-turbo_3.0.0.bb index e086830c021..146d80008c5 100644 --- a/meta/recipes-graphics/jpeg/libjpeg-turbo_2.1.5.1.bb +++ b/meta/recipes-graphics/jpeg/libjpeg-turbo_3.0.0.bb @@ -2,19 +2,15 @@ SUMMARY = "Hardware accelerated JPEG compression/decompression library" DESCRIPTION = "libjpeg-turbo is a derivative of libjpeg that uses SIMD instructions (MMX, SSE2, NEON) to accelerate baseline JPEG compression and decompression" HOMEPAGE = "http://libjpeg-turbo.org/" -LICENSE = "BSD-3-Clause" -LIC_FILES_CHKSUM = "file://cdjpeg.h;endline=13;md5=8a61af33cc1c681cd5cc297150bbb5bd \ - file://jpeglib.h;endline=16;md5=52b5eaade8d5b6a452a7693dfe52c084 \ - file://djpeg.c;endline=11;md5=510b386442ab6a27ee241fc5669bc5ea \ - " +LICENSE = "IJG & BSD-3-Clause & Zlib" +LIC_FILES_CHKSUM = "file://LICENSE.md;md5=2a8e0d8226a102f07ab63ed7fd6ce155" + DEPENDS:append:x86-64:class-target = " nasm-native" DEPENDS:append:x86:class-target = " nasm-native" -SRC_URI = "${SOURCEFORGE_MIRROR}/${BPN}/${BPN}-${PV}.tar.gz \ - file://0001-libjpeg-turbo-fix-package_qa-error.patch \ - " +SRC_URI = "${SOURCEFORGE_MIRROR}/${BPN}/${BPN}-${PV}.tar.gz" -SRC_URI[sha256sum] = "2fdc3feb6e9deb17adec9bafa3321419aa19f8f4e5dea7bf8486844ca22207bf" +SRC_URI[sha256sum] = "c77c65fcce3d33417b2e90432e7a0eb05f59a7fff884022a9d931775d583bfaa" UPSTREAM_CHECK_URI = "http://sourceforge.net/projects/libjpeg-turbo/files/" UPSTREAM_CHECK_REGEX = "/libjpeg-turbo/files/(?P<pver>(\d+[\.\-_]*)+)/"