diff mbox series

[v2,2/3] libdisplay-info: add recipe

Message ID 20241123193527.1435268-3-hiagofranco@gmail.com
State Accepted, archived
Commit aa3fe5a5c00085f5455a66c1033f4124e76239ec
Headers show
Series weston: upgrade and add libdisplay-info recipe | expand

Commit Message

Hiago De Franco Nov. 23, 2024, 7:35 p.m. UTC 
From: Hiago De Franco <hiago.franco@toradex.com>

libdisplay-info is a EDID and DisplayID library. Goals:

- Provide a set of high-level, easy-to-use, opinionated functions as
  well as low-level functions to access detailed information.
- Simplicity and correctness over performance and resource usage.
- Well-tested and fuzzed.

Since commit a16598b038b2 ("backend-drm: make libdisplay-info
mandatory") from Weston, this library is now mandatory in order to
build weston 14.0.1 and newer.

Signed-off-by: Hiago De Franco <hiago.franco@toradex.com>
---
 meta/conf/distro/include/maintainers.inc         |  1 +
 .../libdisplay-info/libdisplay-info_0.2.0.bb     | 16 ++++++++++++++++
 2 files changed, 17 insertions(+)
 create mode 100644 meta/recipes-support/libdisplay-info/libdisplay-info_0.2.0.bb

Comments

Ross Burton Nov. 25, 2024, 2:18 p.m. UTC | #1 
On 23 Nov 2024, at 19:35, Hiago De Franco via lists.openembedded.org <hiagofranco=gmail.com@lists.openembedded.org> wrote:
> 
> From: Hiago De Franco <hiago.franco@toradex.com>
> 
> libdisplay-info is a EDID and DisplayID library. Goals:
> 
> - Provide a set of high-level, easy-to-use, opinionated functions as
>  well as low-level functions to access detailed information.
> - Simplicity and correctness over performance and resource usage.
> - Well-tested and fuzzed.
> 
> Since commit a16598b038b2 ("backend-drm: make libdisplay-info
> mandatory") from Weston, this library is now mandatory in order to
> build weston 14.0.1 and newer.
> 
> Signed-off-by: Hiago De Franco <hiago.franco@toradex.com>

This puts a small tool into the same package as the library, so library package renaming doesn’t happen.  Can you inherit lib_package so the small tool (di-edid-decode) goes into libdisplay-info-bin?

Thanks,
Ross
Hiago De Franco Nov. 25, 2024, 11:57 p.m. UTC | #2 
On Mon, Nov 25, 2024 at 02:18:09PM +0000, Ross Burton wrote:
> On 23 Nov 2024, at 19:35, Hiago De Franco via lists.openembedded.org <hiagofranco=gmail.com@lists.openembedded.org> wrote:
> > 
> > From: Hiago De Franco <hiago.franco@toradex.com>
> > 
> > libdisplay-info is a EDID and DisplayID library. Goals:
> > 
> > - Provide a set of high-level, easy-to-use, opinionated functions as
> >  well as low-level functions to access detailed information.
> > - Simplicity and correctness over performance and resource usage.
> > - Well-tested and fuzzed.
> > 
> > Since commit a16598b038b2 ("backend-drm: make libdisplay-info
> > mandatory") from Weston, this library is now mandatory in order to
> > build weston 14.0.1 and newer.
> > 
> > Signed-off-by: Hiago De Franco <hiago.franco@toradex.com>
> 
> This puts a small tool into the same package as the library, so library package renaming doesn’t happen.  Can you inherit lib_package so the small tool (di-edid-decode) goes into libdisplay-info-bin?

Sure, thanks. I will prepare a v3.

> 
> Thanks,
> Ross
> 

Cheers,

Hiago.
diff mbox series

Patch

diff --git a/meta/conf/distro/include/maintainers.inc b/meta/conf/distro/include/maintainers.inc
index 04208ebbf22b..53bb9d024e8a 100644
--- a/meta/conf/distro/include/maintainers.inc
+++ b/meta/conf/distro/include/maintainers.inc
@@ -325,6 +325,7 @@  RECIPE_MAINTAINER:pn-libcomps = "Alexander Kanavin <alex.kanavin@gmail.com>"
 RECIPE_MAINTAINER:pn-libconvert-asn1-perl = "Tim Orling <tim.orling@konsulko.com>"
 RECIPE_MAINTAINER:pn-libdaemon = "Alexander Kanavin <alex.kanavin@gmail.com>"
 RECIPE_MAINTAINER:pn-libdazzle = "Alexander Kanavin <alex.kanavin@gmail.com>"
+RECIPE_MAINTAINER:pn-libdisplay-info = "Hiago De Franco <hiago.franco@toradex.com>"
 RECIPE_MAINTAINER:pn-libdnf = "Alexander Kanavin <alex.kanavin@gmail.com>"
 RECIPE_MAINTAINER:pn-libdrm = "Otavio Salvador <otavio.salvador@ossystems.com.br>"
 RECIPE_MAINTAINER:pn-libedit = "Khem Raj <raj.khem@gmail.com>"
diff --git a/meta/recipes-support/libdisplay-info/libdisplay-info_0.2.0.bb b/meta/recipes-support/libdisplay-info/libdisplay-info_0.2.0.bb
new file mode 100644
index 000000000000..629f0d420fee
--- /dev/null
+++ b/meta/recipes-support/libdisplay-info/libdisplay-info_0.2.0.bb
@@ -0,0 +1,16 @@ 
+SUMMARY = "EDID and DisplayID library"
+DESCRIPTION = "The goal of this library is to provide a set of high-level, \
+easy-to-use, and opinionated functions, as well as low-level functions for \
+accessing detailed information, prioritizing simplicity and correctness over \
+performance and resource usage, while being well-tested and fuzzed."
+HOMEPAGE = "https://gitlab.freedesktop.org/emersion/libdisplay-info"
+LICENSE = "MIT"
+LIC_FILES_CHKSUM = "file://LICENSE;md5=e4426409957080ee0352128354cea2de"
+DEPENDS = "hwdata-native"
+
+SRC_URI = "git://gitlab.freedesktop.org/emersion/libdisplay-info.git;branch=main;protocol=https"
+SRCREV = "66b802d05b374cd8f388dc6ad1e7ae4f08cb3300"
+
+S = "${WORKDIR}/git"
+
+inherit meson pkgconfig