You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by "Olivier Lamy (JIRA)" <ji...@codehaus.org> on 2009/03/21 12:33:12 UTC
[jira] Updated: (SCM-313) Check out command fails to report an
error
[ http://jira.codehaus.org/browse/SCM-313?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Olivier Lamy updated SCM-313:
-----------------------------
Assignee: Olivier Lamy
Fix Version/s: 1.2
> Check out command fails to report an error
> ------------------------------------------
>
> Key: SCM-313
> URL: http://jira.codehaus.org/browse/SCM-313
> Project: Maven SCM
> Issue Type: Bug
> Components: maven-scm-provider-bazaar
> Affects Versions: 1.0-rc1
> Reporter: Kohsuke Kawaguchi
> Assignee: Olivier Lamy
> Fix For: 1.2
>
>
> BazaarCheckOutCommand line 71 has the following code:
> {noformat}
> // Do the actual checkout
> String[] checkout_cmd = new String[]{BRANCH_CMD, url, checkoutDir.getAbsolutePath()};
> BazaarConsumer checkout_consumer = new BazaarConsumer( getLogger() );
> BazaarUtils.execute( checkout_consumer, getLogger(), checkoutDir.getParentFile(), checkout_cmd );
> // Do inventory to find list of checkedout files
> String[] inventory_cmd = new String[]{INVENTORY_CMD};
> BazaarCheckOutConsumer consumer = new BazaarCheckOutConsumer( getLogger(), checkoutDir );
> ScmResult result = BazaarUtils.execute( consumer, getLogger(), checkoutDir, inventory_cmd );
> return new CheckOutScmResult( consumer.getCheckedOutFiles(), result );
> {noformat}
> This code doesn't check for the error in the branch command, so when that fails the error is not reported correctly.
--
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