Regarding your points:

1. I thought that colon was mandatory, not optional. That's why I took it out of the optional part
2. My bad, I put the slash in the wrong place (most embarrassingly, since I need that slash in the middle of the remote module name).

However, I honestly believe regexp is way too complicated, error-prone and hard to maintain for this job. Just a couple of StringTokenizers, first using commas, then colons for the cvsroot, should be enough. Just my two cents.

Anyway, the committed change works perfectly for my psfs. Thank you!

This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators.
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to