diff mbox series

[master/kirkstone] libdrm: remove alternative setup for kmstest

Message ID 20230105212608.2105546-1-denis@denix.org
State Accepted
Delegated to: Ryan Eatmon
Headers show
Series [master/kirkstone] libdrm: remove alternative setup for kmstest | expand

Commit Message

Denys Dmytriyenko Jan. 5, 2023, 9:26 p.m. UTC
From: Denys Dmytriyenko <denys@konsulko.com>

libdrm no longer provides kms driver and tests for it.

Fixes:

WARNING: libdrm-2.4.114-r0.arago3 do_package: libdrm: alternative target (/usr/bin/kmstest or /usr/bin/kmstest.libdrm) does not exist, skipping...
WARNING: libdrm-2.4.114-r0.arago3 do_package: libdrm: NOT adding alternative provide /usr/bin/kmstest: /usr/bin/kmstest.libdrm does not exist
WARNING: libdrm-2.4.114-r0.arago3 do_package: libdrm: alt_link == alt_target: /usr/bin/kmstest == /usr/bin/kmstest

Signed-off-by: Denys Dmytriyenko <denys@konsulko.com>
---
 meta-arago-distro/recipes-graphics/drm/libdrm_%.bbappend | 8 +-------
 1 file changed, 1 insertion(+), 7 deletions(-)

Comments

Ryan Eatmon Jan. 24, 2023, 2:16 p.m. UTC | #1
I'm not 100% certain, but all of the builds are now failing with this error:


tisdk-default-image_1.0-r0_do_rootfs.log
----------------------------------------

WARNING: kms++.postinst returned 1, marking as unpacked only, 
configuration required on target.
ERROR: Postinstall scriptlets of ['kms++'] have failed. If the intention 
is to defer them to first boot,
then please place them into pkg_postinst_ontarget:${PN} ().
Deferring to first boot via 'exit 1' is no longer supported.
Details of the failure are in 
/scratch/jenkins_builds/arago-kirkstone-wip/build/build/arago-tmp-default-glibc/work/am62xx_evm-oe-linux/tisdk-default-image/1.0-r0/temp/log.do_rootfs.
DEBUG: Python function do_rootfs finished


Seeing as this is the only recent change that mentions kms, this might 
be culprit?


On 1/5/2023 15:26, Denys Dmytriyenko wrote:
> From: Denys Dmytriyenko <denys@konsulko.com>
> 
> libdrm no longer provides kms driver and tests for it.
> 
> Fixes:
> 
> WARNING: libdrm-2.4.114-r0.arago3 do_package: libdrm: alternative target (/usr/bin/kmstest or /usr/bin/kmstest.libdrm) does not exist, skipping...
> WARNING: libdrm-2.4.114-r0.arago3 do_package: libdrm: NOT adding alternative provide /usr/bin/kmstest: /usr/bin/kmstest.libdrm does not exist
> WARNING: libdrm-2.4.114-r0.arago3 do_package: libdrm: alt_link == alt_target: /usr/bin/kmstest == /usr/bin/kmstest
> 
> Signed-off-by: Denys Dmytriyenko <denys@konsulko.com>
> ---
>   meta-arago-distro/recipes-graphics/drm/libdrm_%.bbappend | 8 +-------
>   1 file changed, 1 insertion(+), 7 deletions(-)
> 
> diff --git a/meta-arago-distro/recipes-graphics/drm/libdrm_%.bbappend b/meta-arago-distro/recipes-graphics/drm/libdrm_%.bbappend
> index 85152b5f..ea8b5ca1 100644
> --- a/meta-arago-distro/recipes-graphics/drm/libdrm_%.bbappend
> +++ b/meta-arago-distro/recipes-graphics/drm/libdrm_%.bbappend
> @@ -8,10 +8,4 @@ file://0001-libsync-add-support-for-pre-v4.7-kernels.patch \
>   file://0002-Add-sync_fence_info-and-sync_pt_info.patch \
>   "
>   
> -PR:append = ".arago3"
> -
> -inherit update-alternatives
> -
> -ALTERNATIVE_PRIORITY = "10"
> -ALTERNATIVE:${PN} = "kmstest"
> -ALTERNATIVE_LINK_NAME[kmstest] = "${bindir}/kmstest"
> +PR:append = ".arago4"
Denys Dmytriyenko Jan. 24, 2023, 7:15 p.m. UTC | #2
Sorry, my bad, this is only for master with latest libdrm, not kirkstone:
https://git.yoctoproject.org/meta-arago/commit/?id=548657e4d8118af98ad047a7c19b3cc49b7480bb


On Tue, Jan 24, 2023 at 08:16:00AM -0600, Ryan Eatmon wrote:
> I'm not 100% certain, but all of the builds are now failing with this error:
> 
> 
> tisdk-default-image_1.0-r0_do_rootfs.log
> ----------------------------------------
> 
> WARNING: kms++.postinst returned 1, marking as unpacked only,
> configuration required on target.
> ERROR: Postinstall scriptlets of ['kms++'] have failed. If the
> intention is to defer them to first boot,
> then please place them into pkg_postinst_ontarget:${PN} ().
> Deferring to first boot via 'exit 1' is no longer supported.
> Details of the failure are in /scratch/jenkins_builds/arago-kirkstone-wip/build/build/arago-tmp-default-glibc/work/am62xx_evm-oe-linux/tisdk-default-image/1.0-r0/temp/log.do_rootfs.
> DEBUG: Python function do_rootfs finished
> 
> 
> Seeing as this is the only recent change that mentions kms, this
> might be culprit?
> 
> 
> On 1/5/2023 15:26, Denys Dmytriyenko wrote:
> >From: Denys Dmytriyenko <denys@konsulko.com>
> >
> >libdrm no longer provides kms driver and tests for it.
> >
> >Fixes:
> >
> >WARNING: libdrm-2.4.114-r0.arago3 do_package: libdrm: alternative target (/usr/bin/kmstest or /usr/bin/kmstest.libdrm) does not exist, skipping...
> >WARNING: libdrm-2.4.114-r0.arago3 do_package: libdrm: NOT adding alternative provide /usr/bin/kmstest: /usr/bin/kmstest.libdrm does not exist
> >WARNING: libdrm-2.4.114-r0.arago3 do_package: libdrm: alt_link == alt_target: /usr/bin/kmstest == /usr/bin/kmstest
> >
> >Signed-off-by: Denys Dmytriyenko <denys@konsulko.com>
> >---
> >  meta-arago-distro/recipes-graphics/drm/libdrm_%.bbappend | 8 +-------
> >  1 file changed, 1 insertion(+), 7 deletions(-)
> >
> >diff --git a/meta-arago-distro/recipes-graphics/drm/libdrm_%.bbappend b/meta-arago-distro/recipes-graphics/drm/libdrm_%.bbappend
> >index 85152b5f..ea8b5ca1 100644
> >--- a/meta-arago-distro/recipes-graphics/drm/libdrm_%.bbappend
> >+++ b/meta-arago-distro/recipes-graphics/drm/libdrm_%.bbappend
> >@@ -8,10 +8,4 @@ file://0001-libsync-add-support-for-pre-v4.7-kernels.patch \
> >  file://0002-Add-sync_fence_info-and-sync_pt_info.patch \
> >  "
> >-PR:append = ".arago3"
> >-
> >-inherit update-alternatives
> >-
> >-ALTERNATIVE_PRIORITY = "10"
> >-ALTERNATIVE:${PN} = "kmstest"
> >-ALTERNATIVE_LINK_NAME[kmstest] = "${bindir}/kmstest"
> >+PR:append = ".arago4"
Ryan Eatmon Jan. 24, 2023, 7:19 p.m. UTC | #3
Well... that would certainly make a difference.  Let me test that.


On 1/24/2023 13:15, Denys Dmytriyenko wrote:
> Sorry, my bad, this is only for master with latest libdrm, not kirkstone:
> https://git.yoctoproject.org/meta-arago/commit/?id=548657e4d8118af98ad047a7c19b3cc49b7480bb
> 
> 
> On Tue, Jan 24, 2023 at 08:16:00AM -0600, Ryan Eatmon wrote:
>> I'm not 100% certain, but all of the builds are now failing with this error:
>>
>>
>> tisdk-default-image_1.0-r0_do_rootfs.log
>> ----------------------------------------
>>
>> WARNING: kms++.postinst returned 1, marking as unpacked only,
>> configuration required on target.
>> ERROR: Postinstall scriptlets of ['kms++'] have failed. If the
>> intention is to defer them to first boot,
>> then please place them into pkg_postinst_ontarget:${PN} ().
>> Deferring to first boot via 'exit 1' is no longer supported.
>> Details of the failure are in /scratch/jenkins_builds/arago-kirkstone-wip/build/build/arago-tmp-default-glibc/work/am62xx_evm-oe-linux/tisdk-default-image/1.0-r0/temp/log.do_rootfs.
>> DEBUG: Python function do_rootfs finished
>>
>>
>> Seeing as this is the only recent change that mentions kms, this
>> might be culprit?
>>
>>
>> On 1/5/2023 15:26, Denys Dmytriyenko wrote:
>>> From: Denys Dmytriyenko <denys@konsulko.com>
>>>
>>> libdrm no longer provides kms driver and tests for it.
>>>
>>> Fixes:
>>>
>>> WARNING: libdrm-2.4.114-r0.arago3 do_package: libdrm: alternative target (/usr/bin/kmstest or /usr/bin/kmstest.libdrm) does not exist, skipping...
>>> WARNING: libdrm-2.4.114-r0.arago3 do_package: libdrm: NOT adding alternative provide /usr/bin/kmstest: /usr/bin/kmstest.libdrm does not exist
>>> WARNING: libdrm-2.4.114-r0.arago3 do_package: libdrm: alt_link == alt_target: /usr/bin/kmstest == /usr/bin/kmstest
>>>
>>> Signed-off-by: Denys Dmytriyenko <denys@konsulko.com>
>>> ---
>>>   meta-arago-distro/recipes-graphics/drm/libdrm_%.bbappend | 8 +-------
>>>   1 file changed, 1 insertion(+), 7 deletions(-)
>>>
>>> diff --git a/meta-arago-distro/recipes-graphics/drm/libdrm_%.bbappend b/meta-arago-distro/recipes-graphics/drm/libdrm_%.bbappend
>>> index 85152b5f..ea8b5ca1 100644
>>> --- a/meta-arago-distro/recipes-graphics/drm/libdrm_%.bbappend
>>> +++ b/meta-arago-distro/recipes-graphics/drm/libdrm_%.bbappend
>>> @@ -8,10 +8,4 @@ file://0001-libsync-add-support-for-pre-v4.7-kernels.patch \
>>>   file://0002-Add-sync_fence_info-and-sync_pt_info.patch \
>>>   "
>>> -PR:append = ".arago3"
>>> -
>>> -inherit update-alternatives
>>> -
>>> -ALTERNATIVE_PRIORITY = "10"
>>> -ALTERNATIVE:${PN} = "kmstest"
>>> -ALTERNATIVE_LINK_NAME[kmstest] = "${bindir}/kmstest"
>>> +PR:append = ".arago4"
diff mbox series

Patch

diff --git a/meta-arago-distro/recipes-graphics/drm/libdrm_%.bbappend b/meta-arago-distro/recipes-graphics/drm/libdrm_%.bbappend
index 85152b5f..ea8b5ca1 100644
--- a/meta-arago-distro/recipes-graphics/drm/libdrm_%.bbappend
+++ b/meta-arago-distro/recipes-graphics/drm/libdrm_%.bbappend
@@ -8,10 +8,4 @@  file://0001-libsync-add-support-for-pre-v4.7-kernels.patch \
 file://0002-Add-sync_fence_info-and-sync_pt_info.patch \
 "
 
-PR:append = ".arago3"
-
-inherit update-alternatives
-
-ALTERNATIVE_PRIORITY = "10"
-ALTERNATIVE:${PN} = "kmstest"
-ALTERNATIVE_LINK_NAME[kmstest] = "${bindir}/kmstest"
+PR:append = ".arago4"