Maven SCM

test with CVS NT

Details

  • Type: Test Test
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: 1.0-beta-3
  • Component/s: maven-scm-provider-cvs
  • Labels:
    None
  • Number of attachments :
    0

Description

Vincent reported test failures on CVS NT... test it and fix if required.

Issue Links

relates to

Bug - A problem which impairs or prevents the functions of the product. SCM-121 Intermittent CVS test failures

  • Major - Major loss of function.
  • Open - The issue is open and ready for the assignee to start work on it.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Emmanuel Venisse added a comment -

All tests are ok with cvsnt except CvsCheckoutCommandTest

it run the following command :
cvs -z3 -f -d D:\apache\maven\trunks\scm\maven-scm-providers\maven-scm-provider-cvs\src\test\repository -q checkout -r 1.107.4 -d testCheckOutWithTag test-repo/checkout

CVSNT doesn't allow numeric tag name (1.107.4) : cvs [checkout aborted]: Numeric directory tags are not allowed.

Need to check if user use CVSNT and verify tag format.

Show
Emmanuel Venisse added a comment - All tests are ok with cvsnt except CvsCheckoutCommandTest it run the following command : cvs -z3 -f -d D:\apache\maven\trunks\scm\maven-scm-providers\maven-scm-provider-cvs\src\test\repository -q checkout -r 1.107.4 -d testCheckOutWithTag test-repo/checkout CVSNT doesn't allow numeric tag name (1.107.4) : cvs [checkout aborted] : Numeric directory tags are not allowed. Need to check if user use CVSNT and verify tag format.
Hide
Permalink
Emmanuel Venisse added a comment -

All tests are ok with CVSNT

Show
Emmanuel Venisse added a comment - All tests are ok with CVSNT
Hide
Permalink
Dennis Lundberg added a comment -

Running the command mentioned by Emmanuel in the previous comment works fine for me using CVSNT 2.0.51 on Windows XP. It produces the following results:

U testCheckOutWithTag/src/java/org/apache/maven/MavenUtils.java

Show
Dennis Lundberg added a comment - Running the command mentioned by Emmanuel in the previous comment works fine for me using CVSNT 2.0.51 on Windows XP. It produces the following results: U testCheckOutWithTag/src/java/org/apache/maven/MavenUtils.java
Hide
Permalink
Emmanuel Venisse added a comment -

Hmm, doesn't work with CVSNT 2.5.03 (downloaded today).

What is the correct version? what is the non bugged version?
Your version get the same result than linux cvs.
I'd be +1 to remove my test on cvsnt in checkout command and i'm not sure a numeric tag is used by users because it can checkout only one file.
WDYT?

Show
Emmanuel Venisse added a comment - Hmm, doesn't work with CVSNT 2.5.03 (downloaded today). What is the correct version? what is the non bugged version? Your version get the same result than linux cvs. I'd be +1 to remove my test on cvsnt in checkout command and i'm not sure a numeric tag is used by users because it can checkout only one file. WDYT?
Hide
Permalink
Dennis Lundberg added a comment -

Some reading in the Cederqvist
http://ximbiot.com/cvs/manual/cvs-1.11.21/cvs_17.html#SEC157
http://ximbiot.com/cvs/manual/cvs-1.11.21/cvs_16.html#SEC117
http://ximbiot.com/cvs/manual/cvs-1.11.21/cvs_4.html#SEC48

My conclusion from the references above is that you are not allowed to use "." character in the value for the "-r" option in a checkout command.

Show
Dennis Lundberg added a comment - Some reading in the Cederqvist http://ximbiot.com/cvs/manual/cvs-1.11.21/cvs_17.html#SEC157 http://ximbiot.com/cvs/manual/cvs-1.11.21/cvs_16.html#SEC117 http://ximbiot.com/cvs/manual/cvs-1.11.21/cvs_4.html#SEC48 My conclusion from the references above is that you are not allowed to use "." character in the value for the "-r" option in a checkout command.
Hide
Permalink
Emmanuel Venisse added a comment -

I don't agree. I prefer remove my check for CVSNT and let cvs send the error message if it doesn't support a tag format. We'll have the same result (a ScmException) if it doesn't supported.

Show
Emmanuel Venisse added a comment - I don't agree. I prefer remove my check for CVSNT and let cvs send the error message if it doesn't support a tag format. We'll have the same result (a ScmException) if it doesn't supported.
Hide
Permalink
Dennis Lundberg added a comment -

I agree with your decision to remove the check for CVSNT.

Show
Dennis Lundberg added a comment - I agree with your decision to remove the check for CVSNT.

People

  • Assignee:
    Emmanuel Venisse
    Reporter:
    Brett Porter
Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: