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

[Gyorgy Sarvari <skandigraun@gmail.com>]

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.