diff mbox series

[meta-oe,2/2] drm-lease-manager: add new recipe

Message ID 20251217161114.403899-2-renaud.barrau@madic.com
State New
Headers show
Series [meta-oe,1/2] libtoml: add new recipe | expand

Commit Message

Renaud Barrau Dec. 17, 2025, 4:11 p.m. UTC
From: Renaud Barrau <rbarrau33@gmail.com>

Create a recipe for the drm-lease-manager project  https://github.com/AGLExport/drm-lease-manager

Signed-off-by: Renaud Barrau <rbarrau33@gmail.com>
---
 .../drm-lease-manager_git.bb                  | 20 +++++++++++++++++++
 1 file changed, 20 insertions(+)
 create mode 100644 meta-oe/recipes-graphics/drm-lease-manager/drm-lease-manager_git.bb

Comments

Gyorgy Sarvari Dec. 17, 2025, 6:14 p.m. UTC | #1
On 12/17/25 17:11, Renaud Barrau via lists.openembedded.org wrote:
> From: Renaud Barrau <rbarrau33@gmail.com>
>
> Create a recipe for the drm-lease-manager project  https://github.com/AGLExport/drm-lease-manager
>
> Signed-off-by: Renaud Barrau <rbarrau33@gmail.com>
> ---
>  .../drm-lease-manager_git.bb                  | 20 +++++++++++++++++++
>  1 file changed, 20 insertions(+)
>  create mode 100644 meta-oe/recipes-graphics/drm-lease-manager/drm-lease-manager_git.bb
>
> diff --git a/meta-oe/recipes-graphics/drm-lease-manager/drm-lease-manager_git.bb b/meta-oe/recipes-graphics/drm-lease-manager/drm-lease-manager_git.bb
> new file mode 100644
> index 0000000000..621a08f324
> --- /dev/null
> +++ b/meta-oe/recipes-graphics/drm-lease-manager/drm-lease-manager_git.bb
> @@ -0,0 +1,20 @@
> +SUMMARY = "Recipe for drm-lease-manager"
> +HOMEPAGE = "https://github.com/AGLExport/drm-lease-manager"
> +LICENSE = "Apache-2.0"
> +LIC_FILES_CHKSUM = "file://LICENSE;md5=3b83ef96387f14655fc854ddc3c6bd57"
> +
> +SRC_URI = "git://github.com/AGLExport/drm-lease-manager;protocol=https;branch=master"

This seems to be a development fork, not the upstream repo:
https://git.automotivelinux.org/src/drm-lease-manager/ - I think this
(or a mirror of it) would be a better source.

> +
> +SRCREV = "e6c3bf4ff76ae5eeefdb149e4b4167195dfcfb61"
> +
> +S = "${WORKDIR}/git"

[Same comment about S as in the libtoml recipe]

> +
> +inherit meson pkgconfig
> +
> +DEPENDS += "libdrm pkgconfig-native libtoml"
> +
> +# Change default path /var/run/drm-lease-manager.
> +# For exemple if you wish to move the runtime dir of drm-lease-manager to /etc :
> +# EXTRA_OEMESON += "-Druntime_subdir=/etc/drm-lease-manager"
> +
> +FILES:${PN} += "${bindir}/drm-lease-manager"

Is this needed to specify? I would expect that ${PN} package grabs the
content of ${bindir} by default.
Gyorgy Sarvari Dec. 17, 2025, 6:21 p.m. UTC | #2
On 12/17/25 19:14, Gyorgy Sarvari wrote:
> On 12/17/25 17:11, Renaud Barrau via lists.openembedded.org wrote:
>> From: Renaud Barrau <rbarrau33@gmail.com>
>>
>> Create a recipe for the drm-lease-manager project  https://github.com/AGLExport/drm-lease-manager
>>
>> Signed-off-by: Renaud Barrau <rbarrau33@gmail.com>
>> ---
>>  .../drm-lease-manager_git.bb                  | 20 +++++++++++++++++++
>>  1 file changed, 20 insertions(+)
>>  create mode 100644 meta-oe/recipes-graphics/drm-lease-manager/drm-lease-manager_git.bb
>>
>> diff --git a/meta-oe/recipes-graphics/drm-lease-manager/drm-lease-manager_git.bb b/meta-oe/recipes-graphics/drm-lease-manager/drm-lease-manager_git.bb
>> new file mode 100644
>> index 0000000000..621a08f324
>> --- /dev/null
>> +++ b/meta-oe/recipes-graphics/drm-lease-manager/drm-lease-manager_git.bb
>> @@ -0,0 +1,20 @@
>> +SUMMARY = "Recipe for drm-lease-manager"
>> +HOMEPAGE = "https://github.com/AGLExport/drm-lease-manager"
>> +LICENSE = "Apache-2.0"
>> +LIC_FILES_CHKSUM = "file://LICENSE;md5=3b83ef96387f14655fc854ddc3c6bd57"
>> +
>> +SRC_URI = "git://github.com/AGLExport/drm-lease-manager;protocol=https;branch=master"
> This seems to be a development fork, not the upstream repo:
> https://git.automotivelinux.org/src/drm-lease-manager/ - I think this
> (or a mirror of it) would be a better source.
>
>> +
>> +SRCREV = "e6c3bf4ff76ae5eeefdb149e4b4167195dfcfb61"
>> +
>> +S = "${WORKDIR}/git"
> [Same comment about S as in the libtoml recipe]
>
>> +
>> +inherit meson pkgconfig
>> +
>> +DEPENDS += "libdrm pkgconfig-native libtoml"
>> +

Sorry for the spam, one more minor note: when you inherit pkgconfig, you
don't need to add pkgconfig-native to DEPENDS - the pkgconfig class
takes care of this (actually that's the only thing this class does)

>> +# Change default path /var/run/drm-lease-manager.
>> +# For exemple if you wish to move the runtime dir of drm-lease-manager to /etc :
>> +# EXTRA_OEMESON += "-Druntime_subdir=/etc/drm-lease-manager"
>> +
>> +FILES:${PN} += "${bindir}/drm-lease-manager"
> Is this needed to specify? I would expect that ${PN} package grabs the
> content of ${bindir} by default.
diff mbox series

Patch

diff --git a/meta-oe/recipes-graphics/drm-lease-manager/drm-lease-manager_git.bb b/meta-oe/recipes-graphics/drm-lease-manager/drm-lease-manager_git.bb
new file mode 100644
index 0000000000..621a08f324
--- /dev/null
+++ b/meta-oe/recipes-graphics/drm-lease-manager/drm-lease-manager_git.bb
@@ -0,0 +1,20 @@ 
+SUMMARY = "Recipe for drm-lease-manager"
+HOMEPAGE = "https://github.com/AGLExport/drm-lease-manager"
+LICENSE = "Apache-2.0"
+LIC_FILES_CHKSUM = "file://LICENSE;md5=3b83ef96387f14655fc854ddc3c6bd57"
+
+SRC_URI = "git://github.com/AGLExport/drm-lease-manager;protocol=https;branch=master"
+
+SRCREV = "e6c3bf4ff76ae5eeefdb149e4b4167195dfcfb61"
+
+S = "${WORKDIR}/git"
+
+inherit meson pkgconfig
+
+DEPENDS += "libdrm pkgconfig-native libtoml"
+
+# Change default path /var/run/drm-lease-manager.
+# For exemple if you wish to move the runtime dir of drm-lease-manager to /etc :
+# EXTRA_OEMESON += "-Druntime_subdir=/etc/drm-lease-manager"
+
+FILES:${PN} += "${bindir}/drm-lease-manager"