dep-usage: create edge only for those direct or transitive dependencies.
Review Request #3978 — Created June 6, 2016 and submitted
|nhoward_tw, stuhood, zundel|
Discovered this while working on using dep-usage to clean up twitter internal
build files, when a source file belongs to more than one target, say
A.java, not uncommon when globs especially rglobs is used.
:c, which further depends on
:a, in this case, only
:ashould be marked as undeclared dependency.
:bas undeclared dependency can potentially introduce circular
dependency, in this example, say
:balso depends on
This review applies the smae check used by
Address Stu's comments
Revision 2 (+36 -3)