On Thu, 2009-07-09 at 09:58 -0700, Russ Allbery wrote:
> Stefan Fritsch <[email protected]> writes:
> > chmod `dpkg-statoverride --list /usr/sbin/apache2 | cut -f 3` 
> > /usr/sbin/apache2
> 
> This is a bug in $LEADIN that causes it to catch things immediately
> after the *closing* backtick in a backtick expression.  This will also
> affect checkbashisms in devscripts.
> 
> I think the fix is to change:
> 
>     our $LEADIN = qr'(?:(?:^|[`&;(|{])\s*|(?:if|then|do|while)\s+)';
> 
> to:
> 
>     our $LEADIN = qr'(?:(?:^|[&;(|{]|^[^`]*`)\s*|(?:if|then|do|while)\s+)';
> 
> but I'm not sure and could use more review.

I can't find any obvious issues with it from a few tests, fwiw.

Adam



-- 
To UNSUBSCRIBE, email to [email protected]
with a subject of "unsubscribe". Trouble? Contact [email protected]

Reply via email to