Opened 5 years ago

Last modified 4 years ago

#2412 new defect

CVS incremental update works improperly when using modules instead of directories

Reported by: daviddev Owned by:
Priority: patches-accepted Milestone: 0.9.+
Version: 0.8.7p1 Keywords: cvs
Cc:

Description

Using the new CVS source steps, when passing a CVS module (i.e. one defined in CVSROOT/modules) to the "cvsmodule" parameter, instead of a directory name, the incremental update won't work.

The reason is that the content of "CVS/Repository" will not match the module name given. Instead, the name of the real folder in the repository will appear leading to a non-match and, in the end, to a clobber update.

I am not myself an expert on CVS, but my guess is that the value in "CVS/Repository" cannot be used to check if we can do an update in the build directory.

One solution is to remove this check, but in some cases (e.g. when passing directory names in the "cvsmodule" parameter) there will be a loss of functionality.

Change History (2)

comment:1 Changed 5 years ago by tom.prince

  • Milestone changed from undecided to 0.8.+
  • Priority changed from major to patches-accepted

comment:2 Changed 4 years ago by dustin

  • Milestone changed from 0.8.+ to 0.9.+

Ticket retargeted after milestone closed

Note: See TracTickets for help on using tickets.