Bug 9849 - libdrm autodetects dependencies
Summary: libdrm autodetects dependencies
Status: RESOLVED FIXED
Alias: None
Product: OE-Core
Classification: Build System, Metadata & Runtime
Component: graphics (show other bugs)
Version: 2.2
Hardware: x86 Multiple
: Medium normal
Target Milestone: 2.2 M4
Assignee: Jussi Kukkonen
QA Contact:
URL:
Whiteboard:
Depends on:
Blocks:
 
Reported: 2016-06-29 14:53 UTC by Martin Jansa
Modified: 2016-09-15 06:34 UTC (History)
3 users (show)

See Also:
OS type for building Yocto: ---
Type of Regression: ---
Verified:
Documentation change: No (bug/feature does not impact docs)

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Martin Jansa 2016-06-29 14:53:23 UTC 
WARN: libdrm: libdrm-tests rdepends on cunit, but it isn't a build dependency?
WARN: libdrm: libdrm-tests rdepends on libdrm-amdgpu, but it isn't a build dependency?
Comment 1 Jussi Kukkonen 2016-09-05 12:18:43 UTC 
> WARN: libdrm: libdrm-tests rdepends on cunit, but it isn't a build dependency?

This slipped M3 but I'll send a fix for it once the milestone release is done.

> WARN: libdrm: libdrm-tests rdepends on libdrm-amdgpu, but it isn't a build dependency?

This one is just weird... what is it trying to say? libdrm-tests & libdrm-amdgpu are both built from same source with the same recipe.
Comment 2 Martin Jansa 2016-09-13 14:19:15 UTC 
> This one is just weird... what is it trying to say? libdrm-tests & libdrm-amdgpu are both built from same source with the same recipe.

Does it need RRECOMMENDS_${PN}-tests += "${PN}-amdgpu"?
Comment 3 Jussi Kukkonen 2016-09-15 06:34:51 UTC 
(In reply to comment #2)
> > This one is just weird... what is it trying to say? libdrm-tests & libdrm-amdgpu are both built from same source with the same recipe.
> 
> Does it need RRECOMMENDS_${PN}-tests += "${PN}-amdgpu"?

I don't think that makes sense: it is correctly autodetecting the runtime dependency. Anyway, neither warning should now show up as I've disabled cunit-tests:

commit 0825fcf7001aabc2084cc10e8d9aebbf327c884b
Author: Jussi Kukkonen <jussi.kukkonen@intel.com>
Date:   Tue Sep 13 11:46:41 2016 +0300

    libdrm: Explicitly disable cunit tests
    
    Add patch to control building cunit-tests explicitly. Disable
    them in the recipe.
    
    Fixes [YOCTO #9849].
    
    (From OE-Core rev: cd287235fc5b9d0c174a10a89e31ffd391806113)