[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Savannah-hackers-public] Re: log_accum
From: |
Derek R. Price |
Subject: |
[Savannah-hackers-public] Re: log_accum |
Date: |
Fri, 17 Nov 2006 14:25:24 -0500 |
User-agent: |
Thunderbird 1.5.0.8 (Windows/20061025) |
Sylvain Beucler wrote:
>>> my @option_spec = ("config|c=s@",
>>> "mail-to|m=s@",
>>> - "tag|only-tag|r=s@",
>>> + "only-tag|tag|r=s@",
>>> "file-prefix|file-text|T=s", "user|u=s",
>> I don't see any good reason for this reordering, but I included it
>> anyhow.
>
> That one is important (maybe I should have commented my changes :/),
> because GetOpt uses the first alternative in $config afaics. So in
> this case: $config{only-tag} (previously: $config{tag}).
>
> &{$config{'only-tag'} won't be called if 'tag' is the first
> alternative. Since you already used $config{tag} to store the option
> values and $config{only-tags} as the callback, I just set 'only-tag'
> as the first alternative.
Ah, yes. That sounds right. Sorry for the confusion and thanks for the
refresher! :)
Derek
--
Derek R. Price
CVS Solutions Architect
Get CVS support at Ximbiot <http://ximbiot.com>!
v: +1 248.835.1260
f: +1 248.835.1263
<mailto:address@hidden>