Message ID | 1401564012-23822-1-git-send-email-francois.perrad@gadz.org |
---|---|
State | Superseded |
Headers | show |
François, All, On 2014-05-31 21:20 +0200, Francois Perrad spake thusly: > target and host packages don't use the same background color Interesting. Since, I also have a series touching the graphs, I'll take your patch in my series to do the appropriate transformations/adaptations. Thanks! Regards, Yann E. MORIN. > Signed-off-by: Francois Perrad <francois.perrad@gadz.org> > --- > support/scripts/graph-depends | 21 ++++++++++++++------- > 1 file changed, 14 insertions(+), 7 deletions(-) > > diff --git a/support/scripts/graph-depends b/support/scripts/graph-depends > index e2a5e1e..443ac7d 100755 > --- a/support/scripts/graph-depends > +++ b/support/scripts/graph-depends > @@ -222,15 +222,22 @@ for dep in dependencies: > > # Print the attributes of a node: label and fill-color > def print_attrs(pkg): > + name = pkg_node_name(pkg) > if pkg == 'all': > - print "all [label = \"ALL\"]" > - print "all [color=lightblue,style=filled]" > - return > - print "%s [label = \"%s\"]" % (pkg_node_name(pkg), pkg) > - if mode == PKG_MODE and pkg == rootpkg: > - print "%s [color=lightblue,style=filled]" % pkg_node_name(rootpkg) > + label = 'ALL' > + else: > + label = pkg > + if pkg == 'all' or (mode == PKG_MODE and pkg == rootpkg): > + color = 'lightblue' > else: > - print "%s [color=grey,style=filled]" % pkg_node_name(pkg) > + if pkg.startswith('host') \ > + or pkg.startswith('toolchain') \ > + or pkg.startswith('rootfs'): > + color = 'gainsboro' > + else: > + color = 'grey' > + print "%s [label = \"%s\"]" % (name, label) > + print "%s [color=%s,style=filled]" % (name, color) > > # Print the dependency graph of a package > def print_pkg_deps(depth, pkg): > -- > 1.9.1 > > _______________________________________________ > buildroot mailing list > buildroot@busybox.net > http://lists.busybox.net/mailman/listinfo/buildroot
Farnçois, All, On 2014-05-31 21:20 +0200, Francois Perrad spake thusly: > target and host packages don't use the same background color > + color = 'gainsboro' > + else: > + color = 'grey' The difference between 'gainsboro' and 'grey' is very light. At first, I thought I had forgotten to apply your patch, so the difference is light. Did you choose it on-purpose? Did you choose it because it really looks different when printed? I'd like we get colours that are visually different, both on-display and printed. I'll try to find a better default that is more visually different, and add a new option to specify alternate colours. Regards, Yann E. MORIN.
2014-05-31 22:33 GMT+02:00 Yann E. MORIN <yann.morin.1998@free.fr>: > Farnçois, All, > > On 2014-05-31 21:20 +0200, Francois Perrad spake thusly: >> target and host packages don't use the same background color >> + color = 'gainsboro' >> + else: >> + color = 'grey' > > The difference between 'gainsboro' and 'grey' is very light. At first, I > thought I had forgotten to apply your patch, so the difference is light. > > Did you choose it on-purpose? Did you choose it because it really looks > different when printed? I don't like flashy color. Feel free to experiment another set of color (don't forget black&white printing). See http://graphviz.org/content/color-names François > > I'd like we get colours that are visually different, both on-display and > printed. I'll try to find a better default that is more visually > different, and add a new option to specify alternate colours. > > Regards, > Yann E. MORIN. > > -- > .-----------------.--------------------.------------------.--------------------. > | Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: | > | +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ | > | +33 223 225 172 `------------.-------: X AGAINST | \e/ There is no | > | http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. | > '------------------------------^-------^------------------^--------------------' > _______________________________________________ > buildroot mailing list > buildroot@busybox.net > http://lists.busybox.net/mailman/listinfo/buildroot
François, All, On 2014-06-01 15:23 +0200, François Perrad spake thusly: > 2014-05-31 22:33 GMT+02:00 Yann E. MORIN <yann.morin.1998@free.fr>: > > Farnçois, All, > > > > On 2014-05-31 21:20 +0200, Francois Perrad spake thusly: > >> target and host packages don't use the same background color > >> + color = 'gainsboro' > >> + else: > >> + color = 'grey' > > > > The difference between 'gainsboro' and 'grey' is very light. At first, I > > thought I had forgotten to apply your patch, so the difference is light. > > > > Did you choose it on-purpose? Did you choose it because it really looks > > different when printed? > > I don't like flashy color. > Feel free to experiment another set of color (don't forget black&white > printing). > See http://graphviz.org/content/color-names OK, thanks! I kept your patch as-is, and added an option to change the colours: https://www.gitorious.org/buildroot/buildroot/commit/2092984509902a1f12e30c69f413acf0f55ec0ac Regards, Yann E. MORIN.
diff --git a/support/scripts/graph-depends b/support/scripts/graph-depends index e2a5e1e..443ac7d 100755 --- a/support/scripts/graph-depends +++ b/support/scripts/graph-depends @@ -222,15 +222,22 @@ for dep in dependencies: # Print the attributes of a node: label and fill-color def print_attrs(pkg): + name = pkg_node_name(pkg) if pkg == 'all': - print "all [label = \"ALL\"]" - print "all [color=lightblue,style=filled]" - return - print "%s [label = \"%s\"]" % (pkg_node_name(pkg), pkg) - if mode == PKG_MODE and pkg == rootpkg: - print "%s [color=lightblue,style=filled]" % pkg_node_name(rootpkg) + label = 'ALL' + else: + label = pkg + if pkg == 'all' or (mode == PKG_MODE and pkg == rootpkg): + color = 'lightblue' else: - print "%s [color=grey,style=filled]" % pkg_node_name(pkg) + if pkg.startswith('host') \ + or pkg.startswith('toolchain') \ + or pkg.startswith('rootfs'): + color = 'gainsboro' + else: + color = 'grey' + print "%s [label = \"%s\"]" % (name, label) + print "%s [color=%s,style=filled]" % (name, color) # Print the dependency graph of a package def print_pkg_deps(depth, pkg):
target and host packages don't use the same background color Signed-off-by: Francois Perrad <francois.perrad@gadz.org> --- support/scripts/graph-depends | 21 ++++++++++++++------- 1 file changed, 14 insertions(+), 7 deletions(-)