Ticket #145 (closed enhancement: fixed)

Opened 5 years ago

Last modified 5 years ago

cvs update does not prune empty directories

Reported by: blueyed Owned by: lele
Priority: minor Milestone: VersionOne
Component: cvs Version: 0.9
Keywords: Cc:

Description

I am converting a CVS repository to bzr. But the "deleted"/empty directories in the cvs repo stay in the bzr repo.

Without looking much at the code, this looks like the "cvs update" command is missing the "-P" (prune empty directories) option.

I'm using tailor 0.9.30 and the "cvs" (not cvsps) source.

Change History

comment:1 Changed 5 years ago by lele

  • Status changed from new to closed
  • Resolution set to fixed

Fixed by [1503].

comment:2 Changed 5 years ago by blueyed

  • Status changed from closed to reopened
  • Resolution fixed deleted

Thank you for looking into this, but the fix appears to be only for "cvsps", while I'm experiencing this with "cvs".

There's no "cvs update" involved here with "cvs", but only "rlog".

It needs probably the same logic as when using "-P": if the last file from a directory gets removed, remove also the directory itself in the same changeset.

comment:3 Changed 5 years ago by lele

Uh? Have you actually tried the new version? Yes, [1503] touches the cvsps backend, but the cvs inherits from that most of its behaviour...

comment:4 Changed 5 years ago by blueyed

Sorry for the confusion. I've just tested it (with changes involved this time) and have seen that "cvs up -P" gets used indeed.

So please close the ticket again.

Thanks.

comment:5 Changed 5 years ago by lele

  • Status changed from reopened to closed
  • Resolution set to fixed
Note: See TracTickets for help on using tickets.