[ Do you mind if these messages go to both your email addresses, or
should I remove one or the other? ]
Josef Sipek <jsipek@xxxxxxxxxxxxxxxxx> writes:
> On Sun, Jul 29, 2007 at 12:50:15AM -0700, Eric Lesh wrote:
> ...
>> diff --git a/guilt b/guilt
>> index f67bfb5..774909e 100755
>> --- a/guilt
>> +++ b/guilt
>> @@ -178,7 +178,8 @@ get_series()
>> # - whitespace only
>> # - optional whitespace followed by '#' followed by more
>> # optional whitespace
>> - grep -ve '^[[:space:]]*\(#.*\)*$' "$series"
>> + # also remove comments from end of lines
>> + grep -ve '^[[:space:]]*\(#.*\)*$' < "$series" | sed -e 's/[[:space:]]*#.*$//'
>
> I'd be tempted to replace the whole thing with one sed script...something
> like (not tested):
>
> "
> /^[[:space:]]*#/ ! {
> s/[[:space:]]*#.*$//
>
> p
> }
> "
>
sed -e "/^[[:space:]]*\(#.*\)*$/d
/^[[:space:]]*\(#.*\)*$/!{
s/[[:space:]]*#.*$//
}
" $series
is the best I can do.
sed -e "/^[[:space:]]*\(#.*\)*$/d" -e "s/[[:space:]]*#.*$//" $series
works too, and is maybe more readable.
My sed-foo is weak, though.
Eric
-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
