Maven 2.x Release Plugin

Creating label fails for ClearCase with message 'Unable to determine VOB for pathname ".".'

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 2.1
  • Fix Version/s: 2.2
  • Component/s: prepare
  • Labels:
    None
  • Environment:
    Windows XP
    Maven 3.0
    Base ClearCase 7
  • Number of attachments :
    4

Description

When executing release:prepare it fails when trying to create the label type. However, this works with v2.0 of the m-release-p.

  1. Text File
    mrelease-622.log
    07/Dec/10 6:08 AM
    5 kB
    Anders Hammar
  2. Text File
    MRELEASE-622.patch
    10/Dec/10 4:58 AM
    14 kB
    Anders Hammar
  3. Text File
    MRELEASE-622-updated.patch
    18/Mar/11 4:59 PM
    14 kB
    Anders Hammar
  4. Text File
    ReleaseUtil.getCommonBasedir.txt
    09/Dec/10 8:48 AM
    1 kB
    Anders Hammar

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Anders Hammar added a comment -

I'm experiencing this with both dynamic and snapshot views. The Clearcase client is v7.0.1.4.

Show
Anders Hammar added a comment - I'm experiencing this with both dynamic and snapshot views. The Clearcase client is v7.0.1.4.
Hide
Permalink
Anders Hammar added a comment -

I think this is related to the fix in MRELEASE-536.
After debugging, the main difference between v2.0 and v2.1 is that the working dir path is all in lower case in v2.1. That's the result of MRELEASE-536 and possibly that makes the clearcase call fail.
I'm attaching the debug output when executing. Interestingly, the dir path is just made lower case when creating the label (type), not when checking in the updated pom. Checking in the updated pom works, but not creating the label type.

Show
Anders Hammar added a comment - I think this is related to the fix in MRELEASE-536 . After debugging, the main difference between v2.0 and v2.1 is that the working dir path is all in lower case in v2.1. That's the result of MRELEASE-536 and possibly that makes the clearcase call fail. I'm attaching the debug output when executing. Interestingly, the dir path is just made lower case when creating the label (type), not when checking in the updated pom. Checking in the updated pom works, but not creating the label type.
Hide
Permalink
Anders Hammar added a comment -

Verified that the problem is that the working dir is made into all lower case, which is a side-effect of the fix for MRELEASE-536.
Would like input on if changing to use canonical path for comparison would cause any problems?

Show
Anders Hammar added a comment - Verified that the problem is that the working dir is made into all lower case, which is a side-effect of the fix for MRELEASE-536 . Would like input on if changing to use canonical path for comparison would cause any problems?
Hide
Permalink
Anders Hammar added a comment -

Attaching example of how canonical path could be used in ReleaseUtil.getCommonBasedir(). I can prepare a proper patch for it, but would like someone with more indepth knowledge of the plugin to comment on the approach first.

Show
Anders Hammar added a comment - Attaching example of how canonical path could be used in ReleaseUtil.getCommonBasedir(). I can prepare a proper patch for it, but would like someone with more indepth knowledge of the plugin to comment on the approach first.
Hide
Permalink
Anders Hammar added a comment -

Created proper patch with the suggested change to use canonical path. Tested with ClearCase on Windows.

Show
Anders Hammar added a comment - Created proper patch with the suggested change to use canonical path. Tested with ClearCase on Windows.
Hide
Permalink
Olivier Lamy added a comment -

Hi Andres,
Any way you provide a new patch.
I can apply it due to some other changes.
Thanks

Show
Olivier Lamy added a comment - Hi Andres, Any way you provide a new patch. I can apply it due to some other changes. Thanks
Hide
Permalink
Anders Hammar added a comment -

Ok, updated patch attached.

Show
Anders Hammar added a comment - Ok, updated patch attached.
Hide
Permalink
Olivier Lamy added a comment -

fixed rev 1083092
Thanks !

Show
Olivier Lamy added a comment - fixed rev 1083092 Thanks !
Hide
Permalink
Anders Hammar added a comment -

Thanks Olivier!

Show
Anders Hammar added a comment - Thanks Olivier!

People

  • Assignee:
    Olivier Lamy
    Reporter:
    Anders Hammar
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: