bugConcurrent Versions System - Bugs: bug #17530, Updates which only affect sticky...


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

Submitted by:  Derek Robert Price <dprice>
Submitted on:  Fri 25 Aug 2006 08:42:01 PM UTC  
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

Add a New Comment(Rich Markup)

You are not logged in

Please log in, so followups can be emailed to you.


Fri 25 Aug 2006 08:42:01 PM UTC, original submission:

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.

Derek Robert Price <dprice>
Project Administrator


(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:

No files currently attached


Depends on the following items: None found

Items that depend on this one: None found


CC list is empty


Do you think this task is very important?
If so, you can add your encouragement to it.
This task has 0 encouragements so far.

Only logged-in users can vote.




No changes have been made to this item

Back to the top

Powered by Savane 3.3