Re: [PATCH] headerdep: fix option processing for -I

[Sam Ravnborg <sam@ravnborg.org>]

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@pengutronix.de>
> > Cc: Vegard Nossum <vegard.nossum@gmail.com>
> > Cc: Sam Ravnborg <sam@ravnborg.org>
> > 
> > ---
> >  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