Re: [PATCH] svncopy.pl externals tag fix

On 4/17/06, Michael W Thelen <mike@pietdepsi.com> wrote:
> Michael W Thelen wrote:
> > Thank you for the patch! It looks like the original code was probably
> > intending to make the space optional, but wrongly used \s+? instead of
> > \s*. Since the $morespacing variable is never used in your patch,
> > there's really no need to capture it. Something like the patch below
> > might be cleaner. Unfortunately I can't test it at the moment. Would
> > someone be willing to try it out and either approve or commit it?
>
> I did get around to testing this patch after tweaking it again slightly,
> and it works as expected.

+1 to commit.

Cheers,

David

--
David James -- http://www.cs.toronto.edu/~james
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org