On 14.04.2015 23:20, gregor herrmann wrote:
> Source: box2d
> Severity: minor
> 
> box2d's watch file kills PET, the last line of the python backtrace
> is
> 
> sre_constants.error: invalid group reference
> 
> Of course PET should handle the problem more gracefully but the
> regexp still seems to have a problem:
> 
> version=3
> opts=uversionmangle=s/(\d)[\/]$/$1$2/,\
> dversionmangle=s/\+(debian|dfsg|ds|deb)\d*$// \
> http://box2d.googlecode.com/svn/tags/v(\d.*)
> 
> 
> Unless I'm mistaken, the $2 in the opts line seems to be the problem
> as it doesn't refer to any capture group.
> 
> Not sure what the intention actually is but this should probably be either
>     s/(\d)(\/)$/$1$2/
> or
>     s/(\d)\/$/$1/
> 
>>From PET's point of view, a commit in git would be enough to deal
> with the problem :)

Hi,

Hmm, version 1 doesn't work but version 2 does. I've just committed the fix.

Cheers,

Markus

Attachment: signature.asc
Description: OpenPGP digital signature

Reply via email to