[Buildroot] [PATCH 3/7] graph-depends: handle the depth argument in a more pythonic way

Yann E. MORIN yann.morin.1998 at free.fr
Fri May 9 15:28:56 UTC 2014


Samuel, All,

On 2014-05-09 17:14 +0200, Samuel Martin spake thusly:
> Yann, all,
> 
> On Fri, May 9, 2014 at 4:58 PM, Yann E. MORIN <yann.morin.1998 at free.fr> wrote:
> > From: "Yann E. MORIN" <yann.morin.1998 at free.fr>
> >
> > Add some comment as well, enhance help text.
> >
> > Signed-off-by: "Yann E. MORIN" <yann.morin.1998 at free.fr>
> > Cc: Thomas Petazzoni <thomas.petazzoni at free-electrons.com>
> > ---
> >  support/scripts/graph-depends | 10 ++++++----
> >  1 file changed, 6 insertions(+), 4 deletions(-)
> >
> > diff --git a/support/scripts/graph-depends b/support/scripts/graph-depends
> > index 5bfbc64..dd8b549 100755
> > --- a/support/scripts/graph-depends
> > +++ b/support/scripts/graph-depends
> > @@ -33,13 +33,16 @@ FULL_MODE = 1
> >  PKG_MODE  = 2
> >
> >  mode = 0
> > +
> > +# Limit drawing the dependency graph to this depth. 0 means 'no limit'.
> >  max_depth = 0
> >
> >  parser = argparse.ArgumentParser(description="Graph pacakges dependencies")
> >  parser.add_argument("--package", '-p', metavar="PACKAGE",
> >                      help="Graph the dependencies of PACKAGE")
> > -parser.add_argument("--depth", '-d', metavar="DEPTH",
> > -                    help="Limit the dependency graph to DEPTH levels")
> > +parser.add_argument("--depth", '-d', metavar="DEPTH", dest="depth",
> > +                    help="Limit the dependency graph to DEPTH levels; 0 means no limit.")
> > +parser.set_defaults(depth=0)
> 
> You can also do:
> 
> parser.add_argument("--depth", '-d', metavar="DEPTH", dest="depth",
>     help="Limit the dependency graph to DEPTH levels; 0 means no limit.",
>     type=int, default=0)

Aha! Nice! Thank you. :-)

Then I suppose that:    max_depth = int(args.depth)
has to be just:         max_depth = args.depth
Right?

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.  |
'------------------------------^-------^------------------^--------------------'



More information about the buildroot mailing list