Bug 1365

Summary: Replace Brought In By with Deps to show clearly the package dependencies
Product: [Build System, Metadata & Runtime] Hob Reporter: Jessica <jessica.zhang>
Component: hobAssignee: Joshua Lock - Disabled <josh>
Status: VERIFIED FIXED QA Contact:
Severity: normal    
Priority: Medium CC: jessica.zhang, jiajun.xu, lianhao.lu, liping.ke, poky.bs.watcher, poky.watcher
Version: unspecified   
Target Milestone: 1.1   
Hardware: x86   
OS: Multiple   
Whiteboard:
OS type for building Yocto: --- Type of Regression: ---
Verified: Documentation change: ---

Description Jessica 2011-08-11 11:18:44 UTC
Currently we show "Brought In By" in hob for package dependencies which is calculated based on deps list.  We should just show deps list in hob, which provides a much clearer view of package dependencies and eliminate the extra code logic for calculating BINB
Comment 1 Joshua Lock - Disabled 2011-08-17 21:42:09 UTC
I have a WIP patch which changes the Brought-in-by column to list all items which brought the package in, not just the first.

The WIP patch can be seen at: https://github.com/incandescant/bitbake/commit/b3902ac3283e6d3677560a02049255aff6b664bf
Comment 2 Joshua Lock - Disabled 2011-08-25 19:42:32 UTC
I implemented what I felt was the spirit of this bug, rather than the actual request, in the following commit:

http://git.yoctoproject.org/cgit/cgit.cgi/poky/commit/?id=b6009b295267cb991eb5efbe863ed4e4c99fb715

As described in the commit header I've changed the code to track all packages which bring the item in (i.e. depend on it). I'd be curious to know if you feel this addresses the clarity issue?
It's certainly removed some burdensome logic from the code by dropping the expensive find_alt_dependency() method.
Comment 3 Jessica 2011-08-26 10:05:22 UTC
I've looked and it definitely shows the packages dependency relationships much better
Comment 4 Joshua Lock - Disabled 2011-08-26 11:21:27 UTC
Great. So does that resolve this issue?
Comment 5 Jessica 2011-08-26 11:49:52 UTC
Yes, I think we can mark this bug resolved and just fix those several bugs relating to the details of brought in by list
Comment 7 Jessica 2011-08-30 16:50:21 UTC
Verified against latest master