[ http://jira.codehaus.org/browse/SCM-139?page=comments#action_59384 ] 

Emmanuel Venisse commented on SCM-139:
--------------------------------------

i thought to a patch that remove "lot of code in different place" and use your 
class instead.

> Create a utility class for scm url checking/parsing
> ---------------------------------------------------
>
>          Key: SCM-139
>          URL: http://jira.codehaus.org/browse/SCM-139
>      Project: Maven SCM
>         Type: New Feature

>   Components: maven-scm-api
>     Reporter: Dennis Lundberg
>  Attachments: SCM-139-2.zip, SCM-139.zip, ScmUrlUtils.java
>
>
> There is a lot of code in different places, both in maven-scm and elsewhere, 
> that checks and parses scm url:s. I propose that a utility class ScmUrlUtils 
> be crated in maven-scm-api where such code (static methods) can be placed. 
> The code there should not be scm provider specific. This should be 
> accompanied by a test suite.
> This concept might also be applied to individual scm providers, e.g. there 
> could be a CvsScmUrlUtils in maven-scm-provider-cvs.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira

Reply via email to