[SCM-313] baazar : Check out command fails to report an error

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.0-rc1
• Fix Version/s: 1.2
• Component/s: maven-scm-provider-bazaar
• Assignee: Olivier Lamy
• Reporter: Kohsuke Kawaguchi
• Created: 13/May/07 09:35
• Updated: 21/Mar/09 06:36
• Resolved: 21/Mar/09 06:36

Description

BazaarCheckOutCommand line 71 has the following code:

        // 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 );

This code doesn't check for the error in the branch command, so when that fails the error is not reported correctly.