You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@maven.apache.org by "Dennis Lundberg (JIRA)" <ji...@codehaus.org> on 2006/01/20 21:02:06 UTC

[jira] Created: (MPCHANGELOG-81) Some valid scm urls are not allowed by this plugin

Some valid scm urls are not allowed by this plugin
--------------------------------------------------

         Key: MPCHANGELOG-81
         URL: http://jira.codehaus.org/browse/MPCHANGELOG-81
     Project: maven-changelog-plugin
        Type: Bug

    Reporter: Dennis Lundberg


The solution to MPCHANGELOG-72 introduced a new bug. It is no longer possible to use an scm url that uses "|" as the separator.

There is also a possible future problem in this plugin with regards to scm urls. Upcoming versions of maven-scm will allow an scm url to start with "scm|" as well as "scm:". 

-- 
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


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org


[jira] Closed: (MPCHANGELOG-81) Some valid scm urls are not allowed by this plugin

Posted by "Lukas Theussl (JIRA)" <ji...@codehaus.org>.
     [ http://jira.codehaus.org/browse/MPCHANGELOG-81?page=all ]
     
Lukas Theussl closed MPCHANGELOG-81:
------------------------------------

     Resolution: Fixed
    Fix Version: 1.9

Patch applied. Thanks!
For the record: we agreed on the scm-dev list that the format for a valid scm url should be scm:<provider><delimiter><provider-parameters>, ie it has to start with 'scm:', but <delimiter> can be either ':' or '|'.

> Some valid scm urls are not allowed by this plugin
> --------------------------------------------------
>
>          Key: MPCHANGELOG-81
>          URL: http://jira.codehaus.org/browse/MPCHANGELOG-81
>      Project: maven-changelog-plugin
>         Type: Bug

>     Reporter: Dennis Lundberg
>      Fix For: 1.9
>  Attachments: ChangeLog2Test.java, ChangeLogTest.java, MPCHANGELOG-81-2.patch, MPCHANGELOG-81.patch
>
>
> The solution to MPCHANGELOG-72 introduced a new bug. It is no longer possible to use an scm url that uses "|" as the separator.
> There is also a possible future problem in this plugin with regards to scm urls. Upcoming versions of maven-scm will allow an scm url to start with "scm|" as well as "scm:". 

-- 
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


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org


[jira] Commented: (MPCHANGELOG-81) Some valid scm urls are not allowed by this plugin

Posted by "Lukas Theussl (JIRA)" <ji...@codehaus.org>.
    [ http://jira.codehaus.org/browse/MPCHANGELOG-81?page=comments#action_56515 ] 

Lukas Theussl commented on MPCHANGELOG-81:
------------------------------------------

Are you sure this is a valid connection string (from RepositoryTest.java):

scm:cvs|pserver|anoncvs@cvs.apache.org|/home/cvspublic|module

Shouldn't the separator be set to the character after the 'scm'?

> Some valid scm urls are not allowed by this plugin
> --------------------------------------------------
>
>          Key: MPCHANGELOG-81
>          URL: http://jira.codehaus.org/browse/MPCHANGELOG-81
>      Project: maven-changelog-plugin
>         Type: Bug

>     Reporter: Dennis Lundberg
>  Attachments: ChangeLogTest.java, MPCHANGELOG-81.patch
>
>
> The solution to MPCHANGELOG-72 introduced a new bug. It is no longer possible to use an scm url that uses "|" as the separator.
> There is also a possible future problem in this plugin with regards to scm urls. Upcoming versions of maven-scm will allow an scm url to start with "scm|" as well as "scm:". 

-- 
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


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org


[jira] Commented: (MPCHANGELOG-81) Some valid scm urls are not allowed by this plugin

Posted by "Dennis Lundberg (JIRA)" <ji...@codehaus.org>.
    [ http://jira.codehaus.org/browse/MPCHANGELOG-81?page=comments#action_56541 ] 

Dennis Lundberg commented on MPCHANGELOG-81:
--------------------------------------------

That is a valid question. At the moment it does not have to be the same.
I ran this set of commands:
mvn scm:validate -DconnectionUrl="scm:cvs:pserver:anoncvs@cvs.apache.org:/home/cvspublic:module"
mvn scm:validate -DconnectionUrl="scm:cvs|pserver|anoncvs@cvs.apache.org|/home/cvspublic|module"
mvn scm:validate -DconnectionUrl="scm|cvs|pserver|anoncvs@cvs.apache.org|/home/cvspublic|module"

Using maven-scm-alpha-2 (couldn't find alpha-1 anywhere) produced these results:
VALID
VALID
INVALID

Using maven-scm-beta-3-SNAPSHOT produced these results:
VALID
VALID
VALID

As you can see this is consistent with what is in RepositoryTest. In both versions a url starting with "scm:" can use "|" as separator.

The only difference is in the third case, which I indicated is something that is coming in the next version of maven-scm.

> Some valid scm urls are not allowed by this plugin
> --------------------------------------------------
>
>          Key: MPCHANGELOG-81
>          URL: http://jira.codehaus.org/browse/MPCHANGELOG-81
>      Project: maven-changelog-plugin
>         Type: Bug

>     Reporter: Dennis Lundberg
>  Attachments: ChangeLogTest.java, MPCHANGELOG-81.patch
>
>
> The solution to MPCHANGELOG-72 introduced a new bug. It is no longer possible to use an scm url that uses "|" as the separator.
> There is also a possible future problem in this plugin with regards to scm urls. Upcoming versions of maven-scm will allow an scm url to start with "scm|" as well as "scm:". 

-- 
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


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org


[jira] Updated: (MPCHANGELOG-81) Some valid scm urls are not allowed by this plugin

Posted by "Dennis Lundberg (JIRA)" <ji...@codehaus.org>.
     [ http://jira.codehaus.org/browse/MPCHANGELOG-81?page=all ]

Dennis Lundberg updated MPCHANGELOG-81:
---------------------------------------

    Attachment: ChangeLog2Test.java

> Some valid scm urls are not allowed by this plugin
> --------------------------------------------------
>
>          Key: MPCHANGELOG-81
>          URL: http://jira.codehaus.org/browse/MPCHANGELOG-81
>      Project: maven-changelog-plugin
>         Type: Bug

>     Reporter: Dennis Lundberg
>  Attachments: ChangeLog2Test.java, ChangeLogTest.java, MPCHANGELOG-81.patch
>
>
> The solution to MPCHANGELOG-72 introduced a new bug. It is no longer possible to use an scm url that uses "|" as the separator.
> There is also a possible future problem in this plugin with regards to scm urls. Upcoming versions of maven-scm will allow an scm url to start with "scm|" as well as "scm:". 

-- 
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


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org


[jira] Updated: (MPCHANGELOG-81) Some valid scm urls are not allowed by this plugin

Posted by "Dennis Lundberg (JIRA)" <ji...@codehaus.org>.
     [ http://jira.codehaus.org/browse/MPCHANGELOG-81?page=all ]

Dennis Lundberg updated MPCHANGELOG-81:
---------------------------------------

    Attachment: ChangeLogTest.java

> Some valid scm urls are not allowed by this plugin
> --------------------------------------------------
>
>          Key: MPCHANGELOG-81
>          URL: http://jira.codehaus.org/browse/MPCHANGELOG-81
>      Project: maven-changelog-plugin
>         Type: Bug

>     Reporter: Dennis Lundberg
>  Attachments: ChangeLogTest.java
>
>
> The solution to MPCHANGELOG-72 introduced a new bug. It is no longer possible to use an scm url that uses "|" as the separator.
> There is also a possible future problem in this plugin with regards to scm urls. Upcoming versions of maven-scm will allow an scm url to start with "scm|" as well as "scm:". 

-- 
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


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org


[jira] Updated: (MPCHANGELOG-81) Some valid scm urls are not allowed by this plugin

Posted by "Dennis Lundberg (JIRA)" <ji...@codehaus.org>.
     [ http://jira.codehaus.org/browse/MPCHANGELOG-81?page=all ]

Dennis Lundberg updated MPCHANGELOG-81:
---------------------------------------

    Attachment: MPCHANGELOG-81-2.patch

> Some valid scm urls are not allowed by this plugin
> --------------------------------------------------
>
>          Key: MPCHANGELOG-81
>          URL: http://jira.codehaus.org/browse/MPCHANGELOG-81
>      Project: maven-changelog-plugin
>         Type: Bug

>     Reporter: Dennis Lundberg
>  Attachments: ChangeLog2Test.java, ChangeLogTest.java, MPCHANGELOG-81-2.patch, MPCHANGELOG-81.patch
>
>
> The solution to MPCHANGELOG-72 introduced a new bug. It is no longer possible to use an scm url that uses "|" as the separator.
> There is also a possible future problem in this plugin with regards to scm urls. Upcoming versions of maven-scm will allow an scm url to start with "scm|" as well as "scm:". 

-- 
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


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org


[jira] Updated: (MPCHANGELOG-81) Some valid scm urls are not allowed by this plugin

Posted by "Dennis Lundberg (JIRA)" <ji...@codehaus.org>.
     [ http://jira.codehaus.org/browse/MPCHANGELOG-81?page=all ]

Dennis Lundberg updated MPCHANGELOG-81:
---------------------------------------

    Attachment: MPCHANGELOG-81.patch

> Some valid scm urls are not allowed by this plugin
> --------------------------------------------------
>
>          Key: MPCHANGELOG-81
>          URL: http://jira.codehaus.org/browse/MPCHANGELOG-81
>      Project: maven-changelog-plugin
>         Type: Bug

>     Reporter: Dennis Lundberg
>  Attachments: ChangeLogTest.java, MPCHANGELOG-81.patch
>
>
> The solution to MPCHANGELOG-72 introduced a new bug. It is no longer possible to use an scm url that uses "|" as the separator.
> There is also a possible future problem in this plugin with regards to scm urls. Upcoming versions of maven-scm will allow an scm url to start with "scm|" as well as "scm:". 

-- 
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


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org