[bug #17530] Updates which only affect sticky tags should not output 'U' status.

[Derek Robert Price]

URL:
  <http://savannah.nongnu.org/bugs/?17530>

                 Summary: Updates which only affect sticky tags should not
output 'U' status.
                 Project: Concurrent Versions System
            Submitted by: dprice
            Submitted on: Friday 08/25/2006 at 20:42
                Category: Bug Report
                Severity: 2 - Minor
              Item Group: None
                  Status: Confirmed
                 Privacy: Public
             Assigned to: None
             Open/Closed: Open
                 Release: 1.11.22
           Fixed Release: None
   Fixed Feature Release: None

    _______________________________________________________

Details:

Per the notes in bug #17168, updates which only affect sticky tags but which
do not change file contents should not output a `U' status.

To demonstrate, `cvs up -rbranch && cvs up -A', with a branch that mirrors
the content of the trunk.

This should be possible simple to do in update_fileproc.  Have patch_file set
docheckout to -1 when there are no differences between the workfile and the
new version.  Then, in update_fileproc, when docheckout == -1, skip calling
server_updated and just abuse Checked-in response (perhaps via
server_checked_in()) to send the entry to the client, as is already done in
several other locations.

Unfortunately, this would really be an abstraction violation.  Classify_File
should return T_ENTRY or the like if only the entry should be updated, but to
do this would require some test checkouts, and to do that efficiently would
require caching the checkouts for later use by patch_file or whatever.  All
this would be a lot of work to get right.






    _______________________________________________________

Reply to this item at:

  <http://savannah.nongnu.org/bugs/?17530>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.nongnu.org/