[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[bug #14904] Partial commit breaks the CVS contract with the client

From: Rahul Bhargava
Subject: [bug #14904] Partial commit breaks the CVS contract with the client
Date: Wed, 2 Nov 2005 12:25:05 -0800
User-agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7.8) Gecko/20050511 Firefox/1.0.4


                 Summary: Partial commit breaks the CVS contract with the
                 Project: Concurrent Versions System
            Submitted by: b_rahul
            Submitted on: Wed 11/02/05 at 12:25
                Category: None
                Severity: 3 - Normal
              Item Group: None
                  Status: None
                 Privacy: Public
             Assigned to: None
             Open/Closed: Open
           Fixed Release: None
   Fixed Feature Release: None



This is a bug we uncovered while running sanity.sh. If you look at the test
rmadd-14 in sanity.sh, it is trying to do a dir level commit:

  dotest_fail rmadd-14 "${testcvs} -q ci -r mybranch -m add" \
"${PROG} commit: Up-to-date check failed for .file2'
${PROG} \[commit aborted\]: correct above errors first!"

The commit fails with an up-to-date check as file2 is not on the baranch, yet
the file3 is surreptiously added to the Attic! The CVS client does not get any
message to that effect. This is in violation of the normal commit behaviour of
aborting the entire commit if any one of the file is not up-to-date. 

The next test in the suite, rmadd-15 should not be finding the file in

          dotest rmadd-15 "${testcvs} -q ci -r mybranch -m add file3" \
"RCS file: ${CVSROOT_DIRNAME}/first-dir/Attic/file3,v
Checking in file3;
${CVSROOT_DIRNAME}/first-dir/Attic/file3,v  <--  file3
new revision: 1\.1\.2\.1; previous revision: 1\.1

We don't have a bug fix/patch for this yet but just want to track the issue
for now.

Rahul Bhargava,
WANdisco, Inc.


Carbon-Copy List:

CC Address                          | Comment
aahlad@wandisco.com                 | 


Reply to this item at:


  Message sent via/by Savannah

reply via email to

[Prev in Thread] Current Thread [Next in Thread]