On Sun, Mar 29, 2009 at 08:26:17PM +0200, Uwe Kleine-König wrote: > Hello, > > sorry for replying to my own mail, but I found a further enhancement. > > On Sun, Mar 29, 2009 at 03:01:47PM +0200, Uwe Kleine-König wrote: > > -I takes an argument. Without this change only a 1 is added to > > @opt_include which is not helpful. > > > > Signed-off-by: Uwe Kleine-König <u.kleine-koenig@xxxxxxxxxxxxxx> > > Cc: Vegard Nossum <vegard.nossum@xxxxxxxxx> > > Cc: Sam Ravnborg <sam@xxxxxxxxxxxx> > > > > --- > > scripts/headerdep.pl | 2 +- > > 1 files changed, 1 insertions(+), 1 deletions(-) > > > > diff --git a/scripts/headerdep.pl b/scripts/headerdep.pl > > index 97399da..536408e 100755 > > --- a/scripts/headerdep.pl > > +++ b/scripts/headerdep.pl > > @@ -19,7 +19,7 @@ my $opt_graph; > > version => \&version, > > > > all => \$opt_all, > > - I => \@opt_include, > > + "I:s" => \@opt_include, > This should better be "I=s". The difference is that with : the argument > is optional, with = it is not. Applied with this fix and Vegards ack. Sam -- To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
