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

 
 

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

Submitter:  Derek Robert Price <dprice>
Submitted:  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
* Mandatory Fields

Add a New Comment Rich Markup
   

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>
Group administrator

 

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

Attach Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

CC list is empty

 

There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

Only logged-in users can vote.

 

No changes have been made to this item

Back to the top

Powered by Savane 3.13-f8d8.
Corresponding source code