You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by "Rich Seller (JIRA)" <ji...@codehaus.org> on 2009/03/06 18:21:13 UTC

[jira] Commented: (MRELEASE-358) CheckDependencySnapshotsPhase fails when resolving all dependencies

    [ http://jira.codehaus.org/browse/MRELEASE-358?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=168210#action_168210 ] 

Rich Seller commented on MRELEASE-358:
--------------------------------------

Has anybody had a chance to look at this yet?

> CheckDependencySnapshotsPhase fails when resolving all dependencies
> -------------------------------------------------------------------
>
>                 Key: MRELEASE-358
>                 URL: http://jira.codehaus.org/browse/MRELEASE-358
>             Project: Maven 2.x Release Plugin
>          Issue Type: Bug
>          Components: prepare
>    Affects Versions: 2.0-beta-7
>            Reporter: Rich Seller
>         Attachments: patch.txt
>
>
> In CheckDependencySnapshotsPhase, if '0' (All) is selected in response to the prompt "specify the selection number ..." the dependencies are not removed from the original set, so a ReleaseFailureException is thrown at the end of checkProject().
> In the attached patch the dependencies are removed from the original set, and the snapshotSet is returned from resolveSnapshots() to be checked in checkProject().
> In the tests, I implemented a custom Prompter because I don't know how to set multiple conditional responses on JMock1 mocks.
> One other change made while testing: in processSnapshot, "What is the next development version?" is set with a singleton list so a different value has been entered. I removed the list to make the supplied value the default but allow different responses. I can raise this as a separate issue if required

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