| Message ID | 20220916172332.3096-1-Wolfgang.Meyer@gossenmetrawatt.com |
|---|---|
| State | New |
| Headers | show |
| Series | [meta-oe,1/4] fbida: Switch to git fetcher | expand |
Can you rebase this series on top of master-next and resend please. On Fri, Sep 16, 2022 at 10:23 AM Wolfgang Meyer <wolfgang.meyer@gossenmetrawatt.com> wrote: > > fbida has not seen any versioned releases for five years even though > has not stalled. Update to latest commit that builds with the legacy gnumake > style build system ([2019-01-24] ac9005bf0). > > Support for lirc, curl, sane, and cairo-gl has been dropped, the corresponding > packageconfigs and dependencies were removed from the recipe. Dependencies to > libinput and udev have been added. > > Signed-off-by: Wolfgang Meyer <Wolfgang.Meyer@gossenmetrawatt.com> > --- > .../fbida/{fbida_2.14.bb => fbida_git.bb} | 38 +++++++------------ > .../fbida/files/cairo-weak-detect.patch | 17 --------- > 2 files changed, 14 insertions(+), 41 deletions(-) > rename meta-oe/recipes-graphics/fbida/{fbida_2.14.bb => fbida_git.bb} (63%) > delete mode 100644 meta-oe/recipes-graphics/fbida/files/cairo-weak-detect.patch > > diff --git a/meta-oe/recipes-graphics/fbida/fbida_2.14.bb b/meta-oe/recipes-graphics/fbida/fbida_git.bb > similarity index 63% > rename from meta-oe/recipes-graphics/fbida/fbida_2.14.bb > rename to meta-oe/recipes-graphics/fbida/fbida_git.bb > index 3d65399cd..aadaaf57e 100644 > --- a/meta-oe/recipes-graphics/fbida/fbida_2.14.bb > +++ b/meta-oe/recipes-graphics/fbida/fbida_git.bb > @@ -1,23 +1,25 @@ > SUMMARY = "Framebuffer image and doc viewer tools" > DESCRIPTION = "The fbida project contains a few applications for viewing and editing images, \ > - with the main focus being photos." > -HOMEPAGE = "http://linux.bytesex.org/fbida/" > +with the main focus being photos." > +HOMEPAGE = "https://www.kraxel.org/blog/linux/fbida/" > AUTHOR = "Gerd Hoffmann" > SECTION = "utils" > > LICENSE = "GPL-2.0-only" > LIC_FILES_CHKSUM = "file://COPYING;md5=e8feb78a32950a909621bbb51f634b39" > > -DEPENDS = "virtual/libiconv jpeg fontconfig freetype libexif libdrm pixman poppler libepoxy cairo" > +DEPENDS = "virtual/libiconv jpeg fontconfig freetype libexif libdrm pixman udev libinput poppler libepoxy cairo" > > -SRC_URI = "https://www.kraxel.org/releases/fbida/fbida-${PV}.tar.gz \ > - file://0001-Avoid-using-host-path.patch \ > - file://fix-preprocessor.patch \ > - file://support-jpeg-turbo.patch \ > - file://cairo-weak-detect.patch \ > - file://fbida-gcc10.patch \ > - " > -SRC_URI[sha256sum] = "95b7c01556cb6ef9819f358b314ddfeb8a4cbe862b521a3ed62f03d163154438" > +PV = "2.14+git${SRCPV}" > +SRC_URI = "\ > + git://github.com/kraxel/fbida;protocol=https;branch=master \ > + file://0001-Avoid-using-host-path.patch \ > + file://fix-preprocessor.patch \ > + file://support-jpeg-turbo.patch \ > + file://fbida-gcc10.patch \ > +" > +SRCREV = "ac9005bf0bbf50f14dc1b368be5084c8e0510a5d" > +S = "${WORKDIR}/git" > > inherit pkgconfig features_check > > @@ -26,18 +28,12 @@ REQUIRED_DISTRO_FEATURES = "opengl" > > EXTRA_OEMAKE = "STRIP= 'srcdir=${S}' -f ${S}/GNUmakefile" > > -PACKAGECONFIG ??= "gif png curl" > -PACKAGECONFIG[curl] = ",,curl" > +PACKAGECONFIG ??= "gif png" > PACKAGECONFIG[gif] = ",,giflib" > PACKAGECONFIG[png] = ",,libpng" > PACKAGECONFIG[tiff] = ",,tiff" > PACKAGECONFIG[motif] = ",,libx11 libxext libxpm libxt openmotif" > PACKAGECONFIG[webp] = ",,libwebp" > -PACKAGECONFIG[lirc] = ",,lirc" > -# This can only be enabled when cairo has egl enabled in its packageconfig support too > -PACKAGECONFIG[egl] = ",," > - > -EXTRA_OEMAKE += ""${@bb.utils.contains('PACKAGECONFIG', 'egl', 'HAVE_CAIRO_GL=yes', 'HAVE_CAIRO_GL=no', d)}"" > > CFLAGS += "${@bb.utils.contains('DISTRO_FEATURES', 'x11', '', '-DEGL_NO_X11=1', d)}" > > @@ -48,9 +44,6 @@ do_compile() { > > # Be sure to respect preferences (force to "no") > # Also avoid issues when ${BUILD_ARCH} == ${HOST_ARCH} > - if [ -z "${@bb.utils.filter('PACKAGECONFIG', 'curl', d)}" ]; then > - sed -i -e '/^HAVE_LIBCURL/s/:=.*$/:= no/' ${S}/GNUmakefile > - fi > if [ -z "${@bb.utils.filter('PACKAGECONFIG', 'gif', d)}" ]; then > sed -i -e '/^HAVE_LIBGIF/s/:=.*$/:= no/' ${S}/GNUmakefile > fi > @@ -66,9 +59,6 @@ do_compile() { > if [ -z "${@bb.utils.filter('PACKAGECONFIG', 'webp', d)}" ]; then > sed -i -e '/^HAVE_LIBWEBP/s/:=.*$/:= no/' ${S}/GNUmakefile > fi > - if [ -z "${@bb.utils.filter('PACKAGECONFIG', 'lirc', d)}" ]; then > - sed -i -e '/^HAVE_LIBLIRC/s/:=.*$/:= no/' ${S}/GNUmakefile > - fi > > oe_runmake > } > diff --git a/meta-oe/recipes-graphics/fbida/files/cairo-weak-detect.patch b/meta-oe/recipes-graphics/fbida/files/cairo-weak-detect.patch > deleted file mode 100644 > index c7c579cc4..000000000 > --- a/meta-oe/recipes-graphics/fbida/files/cairo-weak-detect.patch > +++ /dev/null > @@ -1,17 +0,0 @@ > -OE configures cairo to disable EGL support, however fbida expects cairo with egl support > -this patch therefore makes it possible to set HAVE_CAIRO_GL via recipe > - > -Upstream-Status: Inappropriate [OE-Specific] > -Signed-off-by: Khem Raj <raj.khem@gmail.com> > - > ---- a/GNUmakefile > -+++ b/GNUmakefile > -@@ -67,7 +67,7 @@ define make-config > - LIB := $(LIB) > - RESDIR := $(call ac_resdir) > - HAVE_LINUX_FB_H := $(call ac_header,linux/fb.h) > --HAVE_CAIRO_GL := $(call ac_pkg_config,cairo-gl) > -+HAVE_CAIRO_GL ?= $(call ac_pkg_config,cairo-gl) > - HAVE_LIBPCD := $(call ac_lib,pcd_open,pcd) > - HAVE_LIBGIF := $(call ac_lib,DGifOpenFileName,gif) > - HAVE_LIBWEBP := $(call ac_pkg_config,libwebp) > -- > 2.37.3 > > > Gossen Metrawatt GmbH > > Südwestpark 15 > 90449 Nürnberg > Germany > > http://www.gossenmetrawatt.com > Amtsgericht Nürnberg HRB 24 602/USt-IdNr. DE815002769 > Geschäftsführer: Marcel Hutka, Matthias Wist > > Gossen Metrawatt GmbH is a member of GMC INSTRUMENTS-Group > > [https://www.gmc-instruments.de/media/mailfooter/mailfooter.jpg]<http://www.gossenmetrawatt.com> > > -=-=-=-=-=-=-=-=-=-=-=- > Links: You receive all messages sent to this group. > View/Reply Online (#98838): https://lists.openembedded.org/g/openembedded-devel/message/98838 > Mute This Topic: https://lists.openembedded.org/mt/93727697/1997914 > Group Owner: openembedded-devel+owner@lists.openembedded.org > Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub [raj.khem@gmail.com] > -=-=-=-=-=-=-=-=-=-=-=- >
Am Freitag, dem 16.09.2022 um 12:10 -0700 schrieb Khem Raj: > Can you rebase this series on top of master-next and resend please. > In order to get this right for the future: is this the default procedure when sending patches? > On Fri, Sep 16, 2022 at 10:23 AM Wolfgang Meyer > <wolfgang.meyer@gossenmetrawatt.com> wrote: > > fbida has not seen any versioned releases for five years even > > though > > has not stalled. Update to latest commit that builds with the > > legacy gnumake > > style build system ([2019-01-24] ac9005bf0). > > > > Support for lirc, curl, sane, and cairo-gl has been dropped, the > > corresponding > > packageconfigs and dependencies were removed from the recipe. > > Dependencies to > > libinput and udev have been added. > > > > Signed-off-by: Wolfgang Meyer <Wolfgang.Meyer@gossenmetrawatt.com> > > --- > > .../fbida/{fbida_2.14.bb => fbida_git.bb} | 38 +++++++------ > > ------ > > .../fbida/files/cairo-weak-detect.patch | 17 --------- > > 2 files changed, 14 insertions(+), 41 deletions(-) > > rename meta-oe/recipes-graphics/fbida/{fbida_2.14.bb => > > fbida_git.bb} (63%) > > delete mode 100644 meta-oe/recipes-graphics/fbida/files/cairo- > > weak-detect.patch > > > > diff --git a/meta-oe/recipes-graphics/fbida/fbida_2.14.bb b/meta- > > oe/recipes-graphics/fbida/fbida_git.bb > > similarity index 63% > > rename from meta-oe/recipes-graphics/fbida/fbida_2.14.bb > > rename to meta-oe/recipes-graphics/fbida/fbida_git.bb > > index 3d65399cd..aadaaf57e 100644 > > --- a/meta-oe/recipes-graphics/fbida/fbida_2.14.bb > > +++ b/meta-oe/recipes-graphics/fbida/fbida_git.bb > > @@ -1,23 +1,25 @@ > > SUMMARY = "Framebuffer image and doc viewer tools" > > DESCRIPTION = "The fbida project contains a few applications for > > viewing and editing images, \ > > - with the main focus being photos." > > -HOMEPAGE = "http://linux.bytesex.org/fbida/" > > +with the main focus being photos." > > +HOMEPAGE = "https://www.kraxel.org/blog/linux/fbida/" > > AUTHOR = "Gerd Hoffmann" > > SECTION = "utils" > > > > LICENSE = "GPL-2.0-only" > > LIC_FILES_CHKSUM = > > "file://COPYING;md5=e8feb78a32950a909621bbb51f634b39" > > > > -DEPENDS = "virtual/libiconv jpeg fontconfig freetype libexif > > libdrm pixman poppler libepoxy cairo" > > +DEPENDS = "virtual/libiconv jpeg fontconfig freetype libexif > > libdrm pixman udev libinput poppler libepoxy cairo" > > > > -SRC_URI = " > > https://www.kraxel.org/releases/fbida/fbida-${PV}.tar.gz \ > > - file://0001-Avoid-using-host-path.patch \ > > - file://fix-preprocessor.patch \ > > - file://support-jpeg-turbo.patch \ > > - file://cairo-weak-detect.patch \ > > - file://fbida-gcc10.patch \ > > - " > > -SRC_URI[sha256sum] = > > "95b7c01556cb6ef9819f358b314ddfeb8a4cbe862b521a3ed62f03d163154438" > > +PV = "2.14+git${SRCPV}" > > +SRC_URI = "\ > > + git://github.com/kraxel/fbida;protocol=https;branch=master > > \ > > + file://0001-Avoid-using-host- > > path.patch \ > > + file://fix- > > preprocessor.patch \ > > + file://support-jpeg- > > turbo.patch \ > > + file://fbida- > > gcc10.patch \ > > +" > > +SRCREV = "ac9005bf0bbf50f14dc1b368be5084c8e0510a5d" > > +S = "${WORKDIR}/git" > > > > inherit pkgconfig features_check > > > > @@ -26,18 +28,12 @@ REQUIRED_DISTRO_FEATURES = "opengl" > > > > EXTRA_OEMAKE = "STRIP= 'srcdir=${S}' -f ${S}/GNUmakefile" > > > > -PACKAGECONFIG ??= "gif png curl" > > -PACKAGECONFIG[curl] = ",,curl" > > +PACKAGECONFIG ??= "gif png" > > PACKAGECONFIG[gif] = ",,giflib" > > PACKAGECONFIG[png] = ",,libpng" > > PACKAGECONFIG[tiff] = ",,tiff" > > PACKAGECONFIG[motif] = ",,libx11 libxext libxpm libxt openmotif" > > PACKAGECONFIG[webp] = ",,libwebp" > > -PACKAGECONFIG[lirc] = ",,lirc" > > -# This can only be enabled when cairo has egl enabled in its > > packageconfig support too > > -PACKAGECONFIG[egl] = ",," > > - > > -EXTRA_OEMAKE += ""${@bb.utils.contains('PACKAGECONFIG', 'egl', > > 'HAVE_CAIRO_GL=yes', 'HAVE_CAIRO_GL=no', d)}"" > > > > CFLAGS += "${@bb.utils.contains('DISTRO_FEATURES', 'x11', '', '- > > DEGL_NO_X11=1', d)}" > > > > @@ -48,9 +44,6 @@ do_compile() { > > > > # Be sure to respect preferences (force to "no") > > # Also avoid issues when ${BUILD_ARCH} == ${HOST_ARCH} > > - if [ -z "${@bb.utils.filter('PACKAGECONFIG', 'curl', d)}" ]; > > then > > - sed -i -e '/^HAVE_LIBCURL/s/:=.*$/:= no/' ${S}/GNUmakefile > > - fi > > if [ -z "${@bb.utils.filter('PACKAGECONFIG', 'gif', d)}" ]; > > then > > sed -i -e '/^HAVE_LIBGIF/s/:=.*$/:= no/' ${S}/GNUmakefile > > fi > > @@ -66,9 +59,6 @@ do_compile() { > > if [ -z "${@bb.utils.filter('PACKAGECONFIG', 'webp', d)}" ]; > > then > > sed -i -e '/^HAVE_LIBWEBP/s/:=.*$/:= no/' ${S}/GNUmakefile > > fi > > - if [ -z "${@bb.utils.filter('PACKAGECONFIG', 'lirc', d)}" ]; > > then > > - sed -i -e '/^HAVE_LIBLIRC/s/:=.*$/:= no/' ${S}/GNUmakefile > > - fi > > > > oe_runmake > > } > > diff --git a/meta-oe/recipes-graphics/fbida/files/cairo-weak- > > detect.patch b/meta-oe/recipes-graphics/fbida/files/cairo-weak- > > detect.patch > > deleted file mode 100644 > > index c7c579cc4..000000000 > > --- a/meta-oe/recipes-graphics/fbida/files/cairo-weak-detect.patch > > +++ /dev/null > > @@ -1,17 +0,0 @@ > > -OE configures cairo to disable EGL support, however fbida expects > > cairo with egl support > > -this patch therefore makes it possible to set HAVE_CAIRO_GL via > > recipe > > - > > -Upstream-Status: Inappropriate [OE-Specific] > > -Signed-off-by: Khem Raj <raj.khem@gmail.com> > > - > > ---- a/GNUmakefile > > -+++ b/GNUmakefile > > -@@ -67,7 +67,7 @@ define make-config > > - LIB := $(LIB) > > - RESDIR := $(call ac_resdir) > > - HAVE_LINUX_FB_H := $(call ac_header,linux/fb.h) > > --HAVE_CAIRO_GL := $(call ac_pkg_config,cairo-gl) > > -+HAVE_CAIRO_GL ?= $(call ac_pkg_config,cairo-gl) > > - HAVE_LIBPCD := $(call ac_lib,pcd_open,pcd) > > - HAVE_LIBGIF := $(call ac_lib,DGifOpenFileName,gif) > > - HAVE_LIBWEBP := $(call ac_pkg_config,libwebp) > > -- > > 2.37.3 > > > > > > Gossen Metrawatt GmbH > > > > Südwestpark 15 > > 90449 Nürnberg > > Germany > > > > http://www.gossenmetrawatt.com > > Amtsgericht Nürnberg HRB 24 602/USt-IdNr. DE815002769 > > Geschäftsführer: Marcel Hutka, Matthias Wist > > > > Gossen Metrawatt GmbH is a member of GMC INSTRUMENTS-Group > > > > [ > > https://www.gmc-instruments.de/media/mailfooter/mailfooter.jpg]<http://www.gossenmetrawatt.com > > > > > > > -=-=-=-=-=-=-=-=-=-=-=- > > Links: You receive all messages sent to this group. > > View/Reply Online (#98838): > > https://lists.openembedded.org/g/openembedded-devel/message/98838 > > Mute This Topic: https://lists.openembedded.org/mt/93727697/1997914 > > Group Owner: openembedded-devel+owner@lists.openembedded.org > > Unsubscribe: > > https://lists.openembedded.org/g/openembedded-devel/unsub > > [raj.khem@gmail.com] > > -=-=-=-=-=-=-=-=-=-=-=- > > Gossen Metrawatt GmbH Südwestpark 15 90449 Nürnberg Germany http://www.gossenmetrawatt.com Amtsgericht Nürnberg HRB 24 602/USt-IdNr. DE815002769 Geschäftsführer: Marcel Hutka, Matthias Wist Gossen Metrawatt GmbH is a member of GMC INSTRUMENTS-Group [https://www.gmc-instruments.de/media/mailfooter/mailfooter.jpg]<http://www.gossenmetrawatt.com>
On Sat, Sep 17, 2022 at 1:48 AM Meyer, Wolfgang <Wolfgang.Meyer@gossenmetrawatt.com> wrote: > > Am Freitag, dem 16.09.2022 um 12:10 -0700 schrieb Khem Raj: > > Can you rebase this series on top of master-next and resend please. > > > > In order to get this right for the future: is this the default > procedure when sending patches? usually patches against master are fine, occasionally there are conflict where it make my life easier if I ask submitter to rebase on master-next master-next is the integration branch that I use for testing and staging submitted patches. > > > On Fri, Sep 16, 2022 at 10:23 AM Wolfgang Meyer > > <wolfgang.meyer@gossenmetrawatt.com> wrote: > > > fbida has not seen any versioned releases for five years even > > > though > > > has not stalled. Update to latest commit that builds with the > > > legacy gnumake > > > style build system ([2019-01-24] ac9005bf0). > > > > > > Support for lirc, curl, sane, and cairo-gl has been dropped, the > > > corresponding > > > packageconfigs and dependencies were removed from the recipe. > > > Dependencies to > > > libinput and udev have been added. > > > > > > Signed-off-by: Wolfgang Meyer <Wolfgang.Meyer@gossenmetrawatt.com> > > > --- > > > .../fbida/{fbida_2.14.bb => fbida_git.bb} | 38 +++++++------ > > > ------ > > > .../fbida/files/cairo-weak-detect.patch | 17 --------- > > > 2 files changed, 14 insertions(+), 41 deletions(-) > > > rename meta-oe/recipes-graphics/fbida/{fbida_2.14.bb => > > > fbida_git.bb} (63%) > > > delete mode 100644 meta-oe/recipes-graphics/fbida/files/cairo- > > > weak-detect.patch > > > > > > diff --git a/meta-oe/recipes-graphics/fbida/fbida_2.14.bb b/meta- > > > oe/recipes-graphics/fbida/fbida_git.bb > > > similarity index 63% > > > rename from meta-oe/recipes-graphics/fbida/fbida_2.14.bb > > > rename to meta-oe/recipes-graphics/fbida/fbida_git.bb > > > index 3d65399cd..aadaaf57e 100644 > > > --- a/meta-oe/recipes-graphics/fbida/fbida_2.14.bb > > > +++ b/meta-oe/recipes-graphics/fbida/fbida_git.bb > > > @@ -1,23 +1,25 @@ > > > SUMMARY = "Framebuffer image and doc viewer tools" > > > DESCRIPTION = "The fbida project contains a few applications for > > > viewing and editing images, \ > > > - with the main focus being photos." > > > -HOMEPAGE = "http://linux.bytesex.org/fbida/" > > > +with the main focus being photos." > > > +HOMEPAGE = "https://www.kraxel.org/blog/linux/fbida/" > > > AUTHOR = "Gerd Hoffmann" > > > SECTION = "utils" > > > > > > LICENSE = "GPL-2.0-only" > > > LIC_FILES_CHKSUM = > > > "file://COPYING;md5=e8feb78a32950a909621bbb51f634b39" > > > > > > -DEPENDS = "virtual/libiconv jpeg fontconfig freetype libexif > > > libdrm pixman poppler libepoxy cairo" > > > +DEPENDS = "virtual/libiconv jpeg fontconfig freetype libexif > > > libdrm pixman udev libinput poppler libepoxy cairo" > > > > > > -SRC_URI = " > > > https://www.kraxel.org/releases/fbida/fbida-${PV}.tar.gz \ > > > - file://0001-Avoid-using-host-path.patch \ > > > - file://fix-preprocessor.patch \ > > > - file://support-jpeg-turbo.patch \ > > > - file://cairo-weak-detect.patch \ > > > - file://fbida-gcc10.patch \ > > > - " > > > -SRC_URI[sha256sum] = > > > "95b7c01556cb6ef9819f358b314ddfeb8a4cbe862b521a3ed62f03d163154438" > > > +PV = "2.14+git${SRCPV}" > > > +SRC_URI = "\ > > > + git://github.com/kraxel/fbida;protocol=https;branch=master > > > \ > > > + file://0001-Avoid-using-host- > > > path.patch \ > > > + file://fix- > > > preprocessor.patch \ > > > + file://support-jpeg- > > > turbo.patch \ > > > + file://fbida- > > > gcc10.patch \ > > > +" > > > +SRCREV = "ac9005bf0bbf50f14dc1b368be5084c8e0510a5d" > > > +S = "${WORKDIR}/git" > > > > > > inherit pkgconfig features_check > > > > > > @@ -26,18 +28,12 @@ REQUIRED_DISTRO_FEATURES = "opengl" > > > > > > EXTRA_OEMAKE = "STRIP= 'srcdir=${S}' -f ${S}/GNUmakefile" > > > > > > -PACKAGECONFIG ??= "gif png curl" > > > -PACKAGECONFIG[curl] = ",,curl" > > > +PACKAGECONFIG ??= "gif png" > > > PACKAGECONFIG[gif] = ",,giflib" > > > PACKAGECONFIG[png] = ",,libpng" > > > PACKAGECONFIG[tiff] = ",,tiff" > > > PACKAGECONFIG[motif] = ",,libx11 libxext libxpm libxt openmotif" > > > PACKAGECONFIG[webp] = ",,libwebp" > > > -PACKAGECONFIG[lirc] = ",,lirc" > > > -# This can only be enabled when cairo has egl enabled in its > > > packageconfig support too > > > -PACKAGECONFIG[egl] = ",," > > > - > > > -EXTRA_OEMAKE += ""${@bb.utils.contains('PACKAGECONFIG', 'egl', > > > 'HAVE_CAIRO_GL=yes', 'HAVE_CAIRO_GL=no', d)}"" > > > > > > CFLAGS += "${@bb.utils.contains('DISTRO_FEATURES', 'x11', '', '- > > > DEGL_NO_X11=1', d)}" > > > > > > @@ -48,9 +44,6 @@ do_compile() { > > > > > > # Be sure to respect preferences (force to "no") > > > # Also avoid issues when ${BUILD_ARCH} == ${HOST_ARCH} > > > - if [ -z "${@bb.utils.filter('PACKAGECONFIG', 'curl', d)}" ]; > > > then > > > - sed -i -e '/^HAVE_LIBCURL/s/:=.*$/:= no/' ${S}/GNUmakefile > > > - fi > > > if [ -z "${@bb.utils.filter('PACKAGECONFIG', 'gif', d)}" ]; > > > then > > > sed -i -e '/^HAVE_LIBGIF/s/:=.*$/:= no/' ${S}/GNUmakefile > > > fi > > > @@ -66,9 +59,6 @@ do_compile() { > > > if [ -z "${@bb.utils.filter('PACKAGECONFIG', 'webp', d)}" ]; > > > then > > > sed -i -e '/^HAVE_LIBWEBP/s/:=.*$/:= no/' ${S}/GNUmakefile > > > fi > > > - if [ -z "${@bb.utils.filter('PACKAGECONFIG', 'lirc', d)}" ]; > > > then > > > - sed -i -e '/^HAVE_LIBLIRC/s/:=.*$/:= no/' ${S}/GNUmakefile > > > - fi > > > > > > oe_runmake > > > } > > > diff --git a/meta-oe/recipes-graphics/fbida/files/cairo-weak- > > > detect.patch b/meta-oe/recipes-graphics/fbida/files/cairo-weak- > > > detect.patch > > > deleted file mode 100644 > > > index c7c579cc4..000000000 > > > --- a/meta-oe/recipes-graphics/fbida/files/cairo-weak-detect.patch > > > +++ /dev/null > > > @@ -1,17 +0,0 @@ > > > -OE configures cairo to disable EGL support, however fbida expects > > > cairo with egl support > > > -this patch therefore makes it possible to set HAVE_CAIRO_GL via > > > recipe > > > - > > > -Upstream-Status: Inappropriate [OE-Specific] > > > -Signed-off-by: Khem Raj <raj.khem@gmail.com> > > > - > > > ---- a/GNUmakefile > > > -+++ b/GNUmakefile > > > -@@ -67,7 +67,7 @@ define make-config > > > - LIB := $(LIB) > > > - RESDIR := $(call ac_resdir) > > > - HAVE_LINUX_FB_H := $(call ac_header,linux/fb.h) > > > --HAVE_CAIRO_GL := $(call ac_pkg_config,cairo-gl) > > > -+HAVE_CAIRO_GL ?= $(call ac_pkg_config,cairo-gl) > > > - HAVE_LIBPCD := $(call ac_lib,pcd_open,pcd) > > > - HAVE_LIBGIF := $(call ac_lib,DGifOpenFileName,gif) > > > - HAVE_LIBWEBP := $(call ac_pkg_config,libwebp) > > > -- > > > 2.37.3 > > > > > > > > > Gossen Metrawatt GmbH > > > > > > Südwestpark 15 > > > 90449 Nürnberg > > > Germany > > > > > > http://www.gossenmetrawatt.com > > > Amtsgericht Nürnberg HRB 24 602/USt-IdNr. DE815002769 > > > Geschäftsführer: Marcel Hutka, Matthias Wist > > > > > > Gossen Metrawatt GmbH is a member of GMC INSTRUMENTS-Group > > > > > > [ > > > https://www.gmc-instruments.de/media/mailfooter/mailfooter.jpg]<http://www.gossenmetrawatt.com > > > > > > > > > > -=-=-=-=-=-=-=-=-=-=-=- > > > Links: You receive all messages sent to this group. > > > View/Reply Online (#98838): > > > https://lists.openembedded.org/g/openembedded-devel/message/98838 > > > Mute This Topic: https://lists.openembedded.org/mt/93727697/1997914 > > > Group Owner: openembedded-devel+owner@lists.openembedded.org > > > Unsubscribe: > > > https://lists.openembedded.org/g/openembedded-devel/unsub > > > [raj.khem@gmail.com] > > > -=-=-=-=-=-=-=-=-=-=-=- > > > > > Gossen Metrawatt GmbH > > Südwestpark 15 > 90449 Nürnberg > Germany > > http://www.gossenmetrawatt.com > Amtsgericht Nürnberg HRB 24 602/USt-IdNr. DE815002769 > Geschäftsführer: Marcel Hutka, Matthias Wist > > Gossen Metrawatt GmbH is a member of GMC INSTRUMENTS-Group > > [https://www.gmc-instruments.de/media/mailfooter/mailfooter.jpg]<http://www.gossenmetrawatt.com>
On Sat, Sep 17, 2022 at 7:55 AM Khem Raj <raj.khem@gmail.com> wrote: > > On Sat, Sep 17, 2022 at 1:48 AM Meyer, Wolfgang > <Wolfgang.Meyer@gossenmetrawatt.com> wrote: > > > > Am Freitag, dem 16.09.2022 um 12:10 -0700 schrieb Khem Raj: > > > Can you rebase this series on top of master-next and resend please. > > > > > > > In order to get this right for the future: is this the default > > procedure when sending patches? > > usually patches against master are fine, occasionally there are conflict > where it make my life easier if I ask submitter to rebase on master-next > master-next is the integration branch that I use for testing and staging > submitted patches. well the problem might be that these patches are somehow getting corrupted I tried to apply the v2 series and I get patching file meta-oe/recipes-graphics/fbida/fbida_git.bb (renamed from meta-oe/recipes-graphics/fbida/fbida_2.14.bb) patch: **** malformed patch at line 166: 9" Could you try applying these patches in a test sandbox after downloading from ml or patchework and see if it working for you ? https://patchwork.yoctoproject.org/project/oe/patch/20220917084459.17933-1-Wolfgang.Meyer@gossenmetrawatt.com/ > > > > > > On Fri, Sep 16, 2022 at 10:23 AM Wolfgang Meyer > > > <wolfgang.meyer@gossenmetrawatt.com> wrote: > > > > fbida has not seen any versioned releases for five years even > > > > though > > > > has not stalled. Update to latest commit that builds with the > > > > legacy gnumake > > > > style build system ([2019-01-24] ac9005bf0). > > > > > > > > Support for lirc, curl, sane, and cairo-gl has been dropped, the > > > > corresponding > > > > packageconfigs and dependencies were removed from the recipe. > > > > Dependencies to > > > > libinput and udev have been added. > > > > > > > > Signed-off-by: Wolfgang Meyer <Wolfgang.Meyer@gossenmetrawatt.com> > > > > --- > > > > .../fbida/{fbida_2.14.bb => fbida_git.bb} | 38 +++++++------ > > > > ------ > > > > .../fbida/files/cairo-weak-detect.patch | 17 --------- > > > > 2 files changed, 14 insertions(+), 41 deletions(-) > > > > rename meta-oe/recipes-graphics/fbida/{fbida_2.14.bb => > > > > fbida_git.bb} (63%) > > > > delete mode 100644 meta-oe/recipes-graphics/fbida/files/cairo- > > > > weak-detect.patch > > > > > > > > diff --git a/meta-oe/recipes-graphics/fbida/fbida_2.14.bb b/meta- > > > > oe/recipes-graphics/fbida/fbida_git.bb > > > > similarity index 63% > > > > rename from meta-oe/recipes-graphics/fbida/fbida_2.14.bb > > > > rename to meta-oe/recipes-graphics/fbida/fbida_git.bb > > > > index 3d65399cd..aadaaf57e 100644 > > > > --- a/meta-oe/recipes-graphics/fbida/fbida_2.14.bb > > > > +++ b/meta-oe/recipes-graphics/fbida/fbida_git.bb > > > > @@ -1,23 +1,25 @@ > > > > SUMMARY = "Framebuffer image and doc viewer tools" > > > > DESCRIPTION = "The fbida project contains a few applications for > > > > viewing and editing images, \ > > > > - with the main focus being photos." > > > > -HOMEPAGE = "http://linux.bytesex.org/fbida/" > > > > +with the main focus being photos." > > > > +HOMEPAGE = "https://www.kraxel.org/blog/linux/fbida/" > > > > AUTHOR = "Gerd Hoffmann" > > > > SECTION = "utils" > > > > > > > > LICENSE = "GPL-2.0-only" > > > > LIC_FILES_CHKSUM = > > > > "file://COPYING;md5=e8feb78a32950a909621bbb51f634b39" > > > > > > > > -DEPENDS = "virtual/libiconv jpeg fontconfig freetype libexif > > > > libdrm pixman poppler libepoxy cairo" > > > > +DEPENDS = "virtual/libiconv jpeg fontconfig freetype libexif > > > > libdrm pixman udev libinput poppler libepoxy cairo" > > > > > > > > -SRC_URI = " > > > > https://www.kraxel.org/releases/fbida/fbida-${PV}.tar.gz \ > > > > - file://0001-Avoid-using-host-path.patch \ > > > > - file://fix-preprocessor.patch \ > > > > - file://support-jpeg-turbo.patch \ > > > > - file://cairo-weak-detect.patch \ > > > > - file://fbida-gcc10.patch \ > > > > - " > > > > -SRC_URI[sha256sum] = > > > > "95b7c01556cb6ef9819f358b314ddfeb8a4cbe862b521a3ed62f03d163154438" > > > > +PV = "2.14+git${SRCPV}" > > > > +SRC_URI = "\ > > > > + git://github.com/kraxel/fbida;protocol=https;branch=master > > > > \ > > > > + file://0001-Avoid-using-host- > > > > path.patch \ > > > > + file://fix- > > > > preprocessor.patch \ > > > > + file://support-jpeg- > > > > turbo.patch \ > > > > + file://fbida- > > > > gcc10.patch \ > > > > +" > > > > +SRCREV = "ac9005bf0bbf50f14dc1b368be5084c8e0510a5d" > > > > +S = "${WORKDIR}/git" > > > > > > > > inherit pkgconfig features_check > > > > > > > > @@ -26,18 +28,12 @@ REQUIRED_DISTRO_FEATURES = "opengl" > > > > > > > > EXTRA_OEMAKE = "STRIP= 'srcdir=${S}' -f ${S}/GNUmakefile" > > > > > > > > -PACKAGECONFIG ??= "gif png curl" > > > > -PACKAGECONFIG[curl] = ",,curl" > > > > +PACKAGECONFIG ??= "gif png" > > > > PACKAGECONFIG[gif] = ",,giflib" > > > > PACKAGECONFIG[png] = ",,libpng" > > > > PACKAGECONFIG[tiff] = ",,tiff" > > > > PACKAGECONFIG[motif] = ",,libx11 libxext libxpm libxt openmotif" > > > > PACKAGECONFIG[webp] = ",,libwebp" > > > > -PACKAGECONFIG[lirc] = ",,lirc" > > > > -# This can only be enabled when cairo has egl enabled in its > > > > packageconfig support too > > > > -PACKAGECONFIG[egl] = ",," > > > > - > > > > -EXTRA_OEMAKE += ""${@bb.utils.contains('PACKAGECONFIG', 'egl', > > > > 'HAVE_CAIRO_GL=yes', 'HAVE_CAIRO_GL=no', d)}"" > > > > > > > > CFLAGS += "${@bb.utils.contains('DISTRO_FEATURES', 'x11', '', '- > > > > DEGL_NO_X11=1', d)}" > > > > > > > > @@ -48,9 +44,6 @@ do_compile() { > > > > > > > > # Be sure to respect preferences (force to "no") > > > > # Also avoid issues when ${BUILD_ARCH} == ${HOST_ARCH} > > > > - if [ -z "${@bb.utils.filter('PACKAGECONFIG', 'curl', d)}" ]; > > > > then > > > > - sed -i -e '/^HAVE_LIBCURL/s/:=.*$/:= no/' ${S}/GNUmakefile > > > > - fi > > > > if [ -z "${@bb.utils.filter('PACKAGECONFIG', 'gif', d)}" ]; > > > > then > > > > sed -i -e '/^HAVE_LIBGIF/s/:=.*$/:= no/' ${S}/GNUmakefile > > > > fi > > > > @@ -66,9 +59,6 @@ do_compile() { > > > > if [ -z "${@bb.utils.filter('PACKAGECONFIG', 'webp', d)}" ]; > > > > then > > > > sed -i -e '/^HAVE_LIBWEBP/s/:=.*$/:= no/' ${S}/GNUmakefile > > > > fi > > > > - if [ -z "${@bb.utils.filter('PACKAGECONFIG', 'lirc', d)}" ]; > > > > then > > > > - sed -i -e '/^HAVE_LIBLIRC/s/:=.*$/:= no/' ${S}/GNUmakefile > > > > - fi > > > > > > > > oe_runmake > > > > } > > > > diff --git a/meta-oe/recipes-graphics/fbida/files/cairo-weak- > > > > detect.patch b/meta-oe/recipes-graphics/fbida/files/cairo-weak- > > > > detect.patch > > > > deleted file mode 100644 > > > > index c7c579cc4..000000000 > > > > --- a/meta-oe/recipes-graphics/fbida/files/cairo-weak-detect.patch > > > > +++ /dev/null > > > > @@ -1,17 +0,0 @@ > > > > -OE configures cairo to disable EGL support, however fbida expects > > > > cairo with egl support > > > > -this patch therefore makes it possible to set HAVE_CAIRO_GL via > > > > recipe > > > > - > > > > -Upstream-Status: Inappropriate [OE-Specific] > > > > -Signed-off-by: Khem Raj <raj.khem@gmail.com> > > > > - > > > > ---- a/GNUmakefile > > > > -+++ b/GNUmakefile > > > > -@@ -67,7 +67,7 @@ define make-config > > > > - LIB := $(LIB) > > > > - RESDIR := $(call ac_resdir) > > > > - HAVE_LINUX_FB_H := $(call ac_header,linux/fb.h) > > > > --HAVE_CAIRO_GL := $(call ac_pkg_config,cairo-gl) > > > > -+HAVE_CAIRO_GL ?= $(call ac_pkg_config,cairo-gl) > > > > - HAVE_LIBPCD := $(call ac_lib,pcd_open,pcd) > > > > - HAVE_LIBGIF := $(call ac_lib,DGifOpenFileName,gif) > > > > - HAVE_LIBWEBP := $(call ac_pkg_config,libwebp) > > > > -- > > > > 2.37.3 > > > > > > > > > > > > Gossen Metrawatt GmbH > > > > > > > > Südwestpark 15 > > > > 90449 Nürnberg > > > > Germany > > > > > > > > http://www.gossenmetrawatt.com > > > > Amtsgericht Nürnberg HRB 24 602/USt-IdNr. DE815002769 > > > > Geschäftsführer: Marcel Hutka, Matthias Wist > > > > > > > > Gossen Metrawatt GmbH is a member of GMC INSTRUMENTS-Group > > > > > > > > [ > > > > https://www.gmc-instruments.de/media/mailfooter/mailfooter.jpg]<http://www.gossenmetrawatt.com > > > > > > > > > > > > > -=-=-=-=-=-=-=-=-=-=-=- > > > > Links: You receive all messages sent to this group. > > > > View/Reply Online (#98838): > > > > https://lists.openembedded.org/g/openembedded-devel/message/98838 > > > > Mute This Topic: https://lists.openembedded.org/mt/93727697/1997914 > > > > Group Owner: openembedded-devel+owner@lists.openembedded.org > > > > Unsubscribe: > > > > https://lists.openembedded.org/g/openembedded-devel/unsub > > > > [raj.khem@gmail.com] > > > > -=-=-=-=-=-=-=-=-=-=-=- > > > > > > > > Gossen Metrawatt GmbH > > > > Südwestpark 15 > > 90449 Nürnberg > > Germany > > > > http://www.gossenmetrawatt.com > > Amtsgericht Nürnberg HRB 24 602/USt-IdNr. DE815002769 > > Geschäftsführer: Marcel Hutka, Matthias Wist > > > > Gossen Metrawatt GmbH is a member of GMC INSTRUMENTS-Group > > > > [https://www.gmc-instruments.de/media/mailfooter/mailfooter.jpg]<http://www.gossenmetrawatt.com>
Am Samstag, dem 17.09.2022 um 08:10 -0700 schrieb Khem Raj: > On Sat, Sep 17, 2022 at 7:55 AM Khem Raj <raj.khem@gmail.com> wrote: > > On Sat, Sep 17, 2022 at 1:48 AM Meyer, Wolfgang > > <Wolfgang.Meyer@gossenmetrawatt.com> wrote: > > > Am Freitag, dem 16.09.2022 um 12:10 -0700 schrieb Khem Raj: > > > > Can you rebase this series on top of master-next and resend > > > > please. > > > > > > > > > > In order to get this right for the future: is this the default > > > procedure when sending patches? > > > > usually patches against master are fine, occasionally there are > > conflict > > where it make my life easier if I ask submitter to rebase on > > master-next > > master-next is the integration branch that I use for testing and > > staging > > submitted patches. > > well the problem might be that these patches are somehow getting > corrupted > I tried to apply the v2 series and I get > > patching file meta-oe/recipes-graphics/fbida/fbida_git.bb (renamed > from meta-oe/recipes-graphics/fbida/fbida_2.14.bb) > patch: **** malformed patch at line 166: 9" > > Could you try applying these patches in a test sandbox after > downloading from ml or patchework and see if it working for you ? > https://patchwork.yoctoproject.org/project/oe/patch/20220917084459.17933-1-Wolfgang.Meyer@gossenmetrawatt.com/ I can reproduce the problem with the patches from patchwork. It seems that our mailserver is changing the content of the sent mails with respect to the whitespace, rendering the patches unusable. I asked the company that supports our Yocto/openembedded related project to submit these patches via their infrastructure. Then they can hopefully been integrated successfully. > > > > > On Fri, Sep 16, 2022 at 10:23 AM Wolfgang Meyer > > > > <wolfgang.meyer@gossenmetrawatt.com> wrote: > > > > > fbida has not seen any versioned releases for five years even > > > > > though > > > > > has not stalled. Update to latest commit that builds with the > > > > > legacy gnumake > > > > > style build system ([2019-01-24] ac9005bf0). > > > > > > > > > > Support for lirc, curl, sane, and cairo-gl has been dropped, > > > > > the > > > > > corresponding > > > > > packageconfigs and dependencies were removed from the recipe. > > > > > Dependencies to > > > > > libinput and udev have been added. > > > > > > > > > > Signed-off-by: Wolfgang Meyer < > > > > > Wolfgang.Meyer@gossenmetrawatt.com> > > > > > --- > > > > > .../fbida/{fbida_2.14.bb => fbida_git.bb} | 38 +++++++ > > > > > ------ > > > > > ------ > > > > > .../fbida/files/cairo-weak-detect.patch | 17 --------- > > > > > 2 files changed, 14 insertions(+), 41 deletions(-) > > > > > rename meta-oe/recipes-graphics/fbida/{fbida_2.14.bb => > > > > > fbida_git.bb} (63%) > > > > > delete mode 100644 meta-oe/recipes- > > > > > graphics/fbida/files/cairo- > > > > > weak-detect.patch > > > > > > > > > > diff --git a/meta-oe/recipes-graphics/fbida/fbida_2.14.bb > > > > > b/meta- > > > > > oe/recipes-graphics/fbida/fbida_git.bb > > > > > similarity index 63% > > > > > rename from meta-oe/recipes-graphics/fbida/fbida_2.14.bb > > > > > rename to meta-oe/recipes-graphics/fbida/fbida_git.bb > > > > > index 3d65399cd..aadaaf57e 100644 > > > > > --- a/meta-oe/recipes-graphics/fbida/fbida_2.14.bb > > > > > +++ b/meta-oe/recipes-graphics/fbida/fbida_git.bb > > > > > @@ -1,23 +1,25 @@ > > > > > SUMMARY = "Framebuffer image and doc viewer tools" > > > > > DESCRIPTION = "The fbida project contains a few applications > > > > > for > > > > > viewing and editing images, \ > > > > > - with the main focus being photos." > > > > > -HOMEPAGE = "http://linux.bytesex.org/fbida/" > > > > > +with the main focus being photos." > > > > > +HOMEPAGE = "https://www.kraxel.org/blog/linux/fbida/" > > > > > AUTHOR = "Gerd Hoffmann" > > > > > SECTION = "utils" > > > > > > > > > > LICENSE = "GPL-2.0-only" > > > > > LIC_FILES_CHKSUM = > > > > > "file://COPYING;md5=e8feb78a32950a909621bbb51f634b39" > > > > > > > > > > -DEPENDS = "virtual/libiconv jpeg fontconfig freetype libexif > > > > > libdrm pixman poppler libepoxy cairo" > > > > > +DEPENDS = "virtual/libiconv jpeg fontconfig freetype libexif > > > > > libdrm pixman udev libinput poppler libepoxy cairo" > > > > > > > > > > -SRC_URI = " > > > > > https://www.kraxel.org/releases/fbida/fbida-${PV}.tar.gz \ > > > > > - file://0001-Avoid-using-host-path.patch \ > > > > > - file://fix-preprocessor.patch \ > > > > > - file://support-jpeg-turbo.patch \ > > > > > - file://cairo-weak-detect.patch \ > > > > > - file://fbida-gcc10.patch \ > > > > > - " > > > > > -SRC_URI[sha256sum] = > > > > > "95b7c01556cb6ef9819f358b314ddfeb8a4cbe862b521a3ed62f03d16315 > > > > > 4438" > > > > > +PV = "2.14+git${SRCPV}" > > > > > +SRC_URI = "\ > > > > > + git://github.com/kraxel/fbida;protocol=https;branch= > > > > > master > > > > > \ > > > > > + file://0001-Avoid-using-host- > > > > > path.patch \ > > > > > + file://fix- > > > > > preprocessor.patch \ > > > > > + file://support-jpeg- > > > > > turbo.patch \ > > > > > + file://fbida- > > > > > gcc10.patch \ > > > > > +" > > > > > +SRCREV = "ac9005bf0bbf50f14dc1b368be5084c8e0510a5d" > > > > > +S = "${WORKDIR}/git" > > > > > > > > > > inherit pkgconfig features_check > > > > > > > > > > @@ -26,18 +28,12 @@ REQUIRED_DISTRO_FEATURES = "opengl" > > > > > > > > > > EXTRA_OEMAKE = "STRIP= 'srcdir=${S}' -f ${S}/GNUmakefile" > > > > > > > > > > -PACKAGECONFIG ??= "gif png curl" > > > > > -PACKAGECONFIG[curl] = ",,curl" > > > > > +PACKAGECONFIG ??= "gif png" > > > > > PACKAGECONFIG[gif] = ",,giflib" > > > > > PACKAGECONFIG[png] = ",,libpng" > > > > > PACKAGECONFIG[tiff] = ",,tiff" > > > > > PACKAGECONFIG[motif] = ",,libx11 libxext libxpm libxt > > > > > openmotif" > > > > > PACKAGECONFIG[webp] = ",,libwebp" > > > > > -PACKAGECONFIG[lirc] = ",,lirc" > > > > > -# This can only be enabled when cairo has egl enabled in its > > > > > packageconfig support too > > > > > -PACKAGECONFIG[egl] = ",," > > > > > - > > > > > -EXTRA_OEMAKE += ""${@bb.utils.contains('PACKAGECONFIG', > > > > > 'egl', > > > > > 'HAVE_CAIRO_GL=yes', 'HAVE_CAIRO_GL=no', d)}"" > > > > > > > > > > CFLAGS += "${@bb.utils.contains('DISTRO_FEATURES', 'x11', > > > > > '', '- > > > > > DEGL_NO_X11=1', d)}" > > > > > > > > > > @@ -48,9 +44,6 @@ do_compile() { > > > > > > > > > > # Be sure to respect preferences (force to "no") > > > > > # Also avoid issues when ${BUILD_ARCH} == ${HOST_ARCH} > > > > > - if [ -z "${@bb.utils.filter('PACKAGECONFIG', 'curl', > > > > > d)}" ]; > > > > > then > > > > > - sed -i -e '/^HAVE_LIBCURL/s/:=.*$/:= no/' > > > > > ${S}/GNUmakefile > > > > > - fi > > > > > if [ -z "${@bb.utils.filter('PACKAGECONFIG', 'gif', d)}" > > > > > ]; > > > > > then > > > > > sed -i -e '/^HAVE_LIBGIF/s/:=.*$/:= no/' > > > > > ${S}/GNUmakefile > > > > > fi > > > > > @@ -66,9 +59,6 @@ do_compile() { > > > > > if [ -z "${@bb.utils.filter('PACKAGECONFIG', 'webp', > > > > > d)}" ]; > > > > > then > > > > > sed -i -e '/^HAVE_LIBWEBP/s/:=.*$/:= no/' > > > > > ${S}/GNUmakefile > > > > > fi > > > > > - if [ -z "${@bb.utils.filter('PACKAGECONFIG', 'lirc', > > > > > d)}" ]; > > > > > then > > > > > - sed -i -e '/^HAVE_LIBLIRC/s/:=.*$/:= no/' > > > > > ${S}/GNUmakefile > > > > > - fi > > > > > > > > > > oe_runmake > > > > > } > > > > > diff --git a/meta-oe/recipes-graphics/fbida/files/cairo-weak- > > > > > detect.patch b/meta-oe/recipes-graphics/fbida/files/cairo- > > > > > weak- > > > > > detect.patch > > > > > deleted file mode 100644 > > > > > index c7c579cc4..000000000 > > > > > --- a/meta-oe/recipes-graphics/fbida/files/cairo-weak- > > > > > detect.patch > > > > > +++ /dev/null > > > > > @@ -1,17 +0,0 @@ > > > > > -OE configures cairo to disable EGL support, however fbida > > > > > expects > > > > > cairo with egl support > > > > > -this patch therefore makes it possible to set HAVE_CAIRO_GL > > > > > via > > > > > recipe > > > > > - > > > > > -Upstream-Status: Inappropriate [OE-Specific] > > > > > -Signed-off-by: Khem Raj <raj.khem@gmail.com> > > > > > - > > > > > ---- a/GNUmakefile > > > > > -+++ b/GNUmakefile > > > > > -@@ -67,7 +67,7 @@ define make-config > > > > > - LIB := $(LIB) > > > > > - RESDIR := $(call ac_resdir) > > > > > - HAVE_LINUX_FB_H := $(call ac_header,linux/fb.h) > > > > > --HAVE_CAIRO_GL := $(call ac_pkg_config,cairo-gl) > > > > > -+HAVE_CAIRO_GL ?= $(call ac_pkg_config,cairo-gl) > > > > > - HAVE_LIBPCD := $(call ac_lib,pcd_open,pcd) > > > > > - HAVE_LIBGIF := $(call ac_lib,DGifOpenFileName,gif) > > > > > - HAVE_LIBWEBP := $(call ac_pkg_config,libwebp) > > > > > -- > > > > > 2.37.3 > > > > > > > > > > > > > > > Gossen Metrawatt GmbH > > > > > > > > > > Südwestpark 15 > > > > > 90449 Nürnberg > > > > > Germany > > > > > > > > > > http://www.gossenmetrawatt.com > > > > > Amtsgericht Nürnberg HRB 24 602/USt-IdNr. DE815002769 > > > > > Geschäftsführer: Marcel Hutka, Matthias Wist > > > > > > > > > > Gossen Metrawatt GmbH is a member of GMC INSTRUMENTS-Group > > > > > > > > > > [ > > > > > https://www.gmc-instruments.de/media/mailfooter/mailfooter.jpg]<http://www.gossenmetrawatt.com > > > > > > > > > > > > > > > > Gossen Metrawatt GmbH > > > > > > Südwestpark 15 > > > 90449 Nürnberg > > > Germany > > > > > > http://www.gossenmetrawatt.com > > > Amtsgericht Nürnberg HRB 24 602/USt-IdNr. DE815002769 > > > Geschäftsführer: Marcel Hutka, Matthias Wist > > > > > > Gossen Metrawatt GmbH is a member of GMC INSTRUMENTS-Group > > > > > > [ > > > https://www.gmc-instruments.de/media/mailfooter/mailfooter.jpg]<http://www.gossenmetrawatt.com > > > > > > -=-=-=-=-=-=-=-=-=-=-=- > Links: You receive all messages sent to this group. > View/Reply Online (#98850): > https://lists.openembedded.org/g/openembedded-devel/message/98850 > Mute This Topic: https://lists.openembedded.org/mt/93727697/6951566 > Group Owner: openembedded-devel+owner@lists.openembedded.org > Unsubscribe: > https://lists.openembedded.org/g/openembedded-devel/unsub > [wolfgang.meyer@gossenmetrawatt.com] > -=-=-=-=-=-=-=-=-=-=-=- > Gossen Metrawatt GmbH Südwestpark 15 90449 Nürnberg Germany http://www.gossenmetrawatt.com Amtsgericht Nürnberg HRB 24 602/USt-IdNr. DE815002769 Geschäftsführer: Marcel Hutka, Matthias Wist Gossen Metrawatt GmbH is a member of GMC INSTRUMENTS-Group [https://www.gmc-instruments.de/media/mailfooter/mailfooter.jpg]<http://www.gossenmetrawatt.com>
diff --git a/meta-oe/recipes-graphics/fbida/fbida_2.14.bb b/meta-oe/recipes-graphics/fbida/fbida_git.bb similarity index 63% rename from meta-oe/recipes-graphics/fbida/fbida_2.14.bb rename to meta-oe/recipes-graphics/fbida/fbida_git.bb index 3d65399cd..aadaaf57e 100644 --- a/meta-oe/recipes-graphics/fbida/fbida_2.14.bb +++ b/meta-oe/recipes-graphics/fbida/fbida_git.bb @@ -1,23 +1,25 @@ SUMMARY = "Framebuffer image and doc viewer tools" DESCRIPTION = "The fbida project contains a few applications for viewing and editing images, \ - with the main focus being photos." -HOMEPAGE = "http://linux.bytesex.org/fbida/" +with the main focus being photos." +HOMEPAGE = "https://www.kraxel.org/blog/linux/fbida/" AUTHOR = "Gerd Hoffmann" SECTION = "utils" LICENSE = "GPL-2.0-only" LIC_FILES_CHKSUM = "file://COPYING;md5=e8feb78a32950a909621bbb51f634b39" -DEPENDS = "virtual/libiconv jpeg fontconfig freetype libexif libdrm pixman poppler libepoxy cairo" +DEPENDS = "virtual/libiconv jpeg fontconfig freetype libexif libdrm pixman udev libinput poppler libepoxy cairo" -SRC_URI = "https://www.kraxel.org/releases/fbida/fbida-${PV}.tar.gz \ - file://0001-Avoid-using-host-path.patch \ - file://fix-preprocessor.patch \ - file://support-jpeg-turbo.patch \ - file://cairo-weak-detect.patch \ - file://fbida-gcc10.patch \ - " -SRC_URI[sha256sum] = "95b7c01556cb6ef9819f358b314ddfeb8a4cbe862b521a3ed62f03d163154438" +PV = "2.14+git${SRCPV}" +SRC_URI = "\ + git://github.com/kraxel/fbida;protocol=https;branch=master \ + file://0001-Avoid-using-host-path.patch \ + file://fix-preprocessor.patch \ + file://support-jpeg-turbo.patch \ + file://fbida-gcc10.patch \ +" +SRCREV = "ac9005bf0bbf50f14dc1b368be5084c8e0510a5d" +S = "${WORKDIR}/git" inherit pkgconfig features_check @@ -26,18 +28,12 @@ REQUIRED_DISTRO_FEATURES = "opengl" EXTRA_OEMAKE = "STRIP= 'srcdir=${S}' -f ${S}/GNUmakefile" -PACKAGECONFIG ??= "gif png curl" -PACKAGECONFIG[curl] = ",,curl" +PACKAGECONFIG ??= "gif png" PACKAGECONFIG[gif] = ",,giflib" PACKAGECONFIG[png] = ",,libpng" PACKAGECONFIG[tiff] = ",,tiff" PACKAGECONFIG[motif] = ",,libx11 libxext libxpm libxt openmotif" PACKAGECONFIG[webp] = ",,libwebp" -PACKAGECONFIG[lirc] = ",,lirc" -# This can only be enabled when cairo has egl enabled in its packageconfig support too -PACKAGECONFIG[egl] = ",," - -EXTRA_OEMAKE += ""${@bb.utils.contains('PACKAGECONFIG', 'egl', 'HAVE_CAIRO_GL=yes', 'HAVE_CAIRO_GL=no', d)}"" CFLAGS += "${@bb.utils.contains('DISTRO_FEATURES', 'x11', '', '-DEGL_NO_X11=1', d)}" @@ -48,9 +44,6 @@ do_compile() { # Be sure to respect preferences (force to "no") # Also avoid issues when ${BUILD_ARCH} == ${HOST_ARCH} - if [ -z "${@bb.utils.filter('PACKAGECONFIG', 'curl', d)}" ]; then - sed -i -e '/^HAVE_LIBCURL/s/:=.*$/:= no/' ${S}/GNUmakefile - fi if [ -z "${@bb.utils.filter('PACKAGECONFIG', 'gif', d)}" ]; then sed -i -e '/^HAVE_LIBGIF/s/:=.*$/:= no/' ${S}/GNUmakefile fi @@ -66,9 +59,6 @@ do_compile() { if [ -z "${@bb.utils.filter('PACKAGECONFIG', 'webp', d)}" ]; then sed -i -e '/^HAVE_LIBWEBP/s/:=.*$/:= no/' ${S}/GNUmakefile fi - if [ -z "${@bb.utils.filter('PACKAGECONFIG', 'lirc', d)}" ]; then - sed -i -e '/^HAVE_LIBLIRC/s/:=.*$/:= no/' ${S}/GNUmakefile - fi oe_runmake } diff --git a/meta-oe/recipes-graphics/fbida/files/cairo-weak-detect.patch b/meta-oe/recipes-graphics/fbida/files/cairo-weak-detect.patch deleted file mode 100644 index c7c579cc4..000000000 --- a/meta-oe/recipes-graphics/fbida/files/cairo-weak-detect.patch +++ /dev/null @@ -1,17 +0,0 @@ -OE configures cairo to disable EGL support, however fbida expects cairo with egl support -this patch therefore makes it possible to set HAVE_CAIRO_GL via recipe - -Upstream-Status: Inappropriate [OE-Specific] -Signed-off-by: Khem Raj <raj.khem@gmail.com> - ---- a/GNUmakefile -+++ b/GNUmakefile -@@ -67,7 +67,7 @@ define make-config - LIB := $(LIB) - RESDIR := $(call ac_resdir) - HAVE_LINUX_FB_H := $(call ac_header,linux/fb.h) --HAVE_CAIRO_GL := $(call ac_pkg_config,cairo-gl) -+HAVE_CAIRO_GL ?= $(call ac_pkg_config,cairo-gl) - HAVE_LIBPCD := $(call ac_lib,pcd_open,pcd) - HAVE_LIBGIF := $(call ac_lib,DGifOpenFileName,gif) - HAVE_LIBWEBP := $(call ac_pkg_config,libwebp)
fbida has not seen any versioned releases for five years even though has not stalled. Update to latest commit that builds with the legacy gnumake style build system ([2019-01-24] ac9005bf0). Support for lirc, curl, sane, and cairo-gl has been dropped, the corresponding packageconfigs and dependencies were removed from the recipe. Dependencies to libinput and udev have been added. Signed-off-by: Wolfgang Meyer <Wolfgang.Meyer@gossenmetrawatt.com> --- .../fbida/{fbida_2.14.bb => fbida_git.bb} | 38 +++++++------------ .../fbida/files/cairo-weak-detect.patch | 17 --------- 2 files changed, 14 insertions(+), 41 deletions(-) rename meta-oe/recipes-graphics/fbida/{fbida_2.14.bb => fbida_git.bb} (63%) delete mode 100644 meta-oe/recipes-graphics/fbida/files/cairo-weak-detect.patch -- 2.37.3 Gossen Metrawatt GmbH Südwestpark 15 90449 Nürnberg Germany http://www.gossenmetrawatt.com Amtsgericht Nürnberg HRB 24 602/USt-IdNr. DE815002769 Geschäftsführer: Marcel Hutka, Matthias Wist Gossen Metrawatt GmbH is a member of GMC INSTRUMENTS-Group [https://www.gmc-instruments.de/media/mailfooter/mailfooter.jpg]<http://www.gossenmetrawatt.com>