[Buildroot] [PATCH 11/12 v3] support/scripts: use show-info to extract dependency graph
ricardo.martincoski at gmail.com
Mon Apr 22 00:14:42 UTC 2019
One question related to an output that changes with this patch and also some
On Mon, Apr 15, 2019 at 04:47 PM, Yann E. MORIN wrote:
> Currently, we extract the dependency graph from the aptly named but
> ad-hoc show-dependency-graph rule.
> We now have a better solution to report package information, with
> Since show-dependency-graph never went into a release so far, and
> show-info does provide the same (and more), swith to using show-info.
> Thanks to Adam for suggesting the coding style to have a readable code
> that is not ugly but still pleases flake8. Thanks to Arnout for
> suggesting the use of dict.get() to further simplify the code.
> Note: we do not use th reverse_dependencies field because it only
> contains those packages that have a kconfig option, so we'd miss most
> host packages.
When I do this before and after this patch:
$ make qemu_arm_versatile_defconfig
$ make graph-depends
The color for "rootfs-common" and "rootfs-ext2" changes.
Can you reproduce it?
Is this intended?
If not intended, do we care enough?
> + versions[pkg] = \
> + None if pkg_list[pkg]["type"] == "rootfs" \
> + else "virtual" if pkg_list[pkg]["virtual"] \
> + else pkg_list[pkg]["version"]
Why not the straightforward version?
if pkg_list[pkg]["type"] == "rootfs":
versions[pkg] = None
versions[pkg] = "virtual"
versions[pkg] = pkg_list[pkg]["version"]
More information about the buildroot