# [Patch] git-cvsimport: tiny fix

4 messages from 2006-05-10 to 2006-05-18. Participants: Elrond, Junio C Hamano, Martin Langhoff.
Thread: https://gitlist.dev/t/4118

## Elrond, 2006-05-10 17:37

Subject: [Patch] git-cvsimport: tiny fix
Message-ID: <20060510173703.GA10335@memak.tu-darmstadt.de>
URL: https://gitlist.dev/e/20060510173703.GA10335%40memak.tu-darmstadt.de

```
git-cvsimport: Handle "Removed" from pserver

Sometimes the pserver says "Removed" instead of
"Remove-entry".

Signed-off-by: Elrond <elrond+kernel.org@samba-tng.org>
---
Hi,

At least the above happened to me on a repository I tried
to convert.
Without the patch, it just die("Unknown: Removed ...")s.


    Elrond

--- git-cvsimport.perl
+++ git-cvsimport.perl
@@ -350,7 +350,7 @@ sub _line {
 				return $res;
 			} elsif($line =~ s/^E //) {
 				# print STDERR "S: $line\n";
-			} elsif($line =~ /^Remove-entry /i) {
+			} elsif($line =~ /^(Remove-entry|Removed) /i) {
 				$line = $self->readline(); # filename
 				$line = $self->readline(); # OK
 				chomp $line;

```

## Junio C Hamano, 2006-05-17 19:12

Subject: Re: [Patch] git-cvsimport: tiny fix
Message-ID: <7vd5eccvns.fsf@assigned-by-dhcp.cox.net>
URL: https://gitlist.dev/e/7vd5eccvns.fsf%40assigned-by-dhcp.cox.net
In-Reply-To: <20060510173703.GA10335@memak.tu-darmstadt.de>

```
Elrond <elrond+kernel.org@samba-tng.org> writes:

> git-cvsimport: Handle "Removed" from pserver
>
> Sometimes the pserver says "Removed" instead of
> "Remove-entry".
>
> Signed-off-by: Elrond <elrond+kernel.org@samba-tng.org>
> ---
> Hi,
>
> At least the above happened to me on a repository I tried
> to convert.
> Without the patch, it just die("Unknown: Removed ...")s.

Could somebody who actually works with CVS import Ack this?
Pretty please?

```

## Martin Langhoff, 2006-05-18 04:53

Subject: Re: [Patch] git-cvsimport: tiny fix
Message-ID: <46a038f90605172153mac96f40id9a50d2f29c75915@mail.gmail.com>
URL: https://gitlist.dev/e/46a038f90605172153mac96f40id9a50d2f29c75915%40mail.gmail.com
In-Reply-To: <7vd5eccvns.fsf@assigned-by-dhcp.cox.net>

```
On 5/18/06, Junio C Hamano <junkio@cox.net> wrote:
> Could somebody who actually works with CVS import Ack this?
> Pretty please?

Sounds sane. It would be interesting to hear about what repo (and
server) this was seen against. Elrond, can you tell us more about
this?

cheers,


martin

```

## Elrond, 2006-05-18 13:29

Subject: Re: [Patch] git-cvsimport: tiny fix
Message-ID: <20060518132956.GA19476@memak.tu-darmstadt.de>
URL: https://gitlist.dev/e/20060518132956.GA19476%40memak.tu-darmstadt.de
In-Reply-To: <46a038f90605172153mac96f40id9a50d2f29c75915@mail.gmail.com>

```
On Thu, May 18, 2006 at 04:53:56PM +1200, Martin Langhoff wrote:
> On 5/18/06, Junio C Hamano <junkio@cox.net> wrote:
> >Could somebody who actually works with CVS import Ack this?
> >Pretty please?
> 
> Sounds sane. It would be interesting to hear about what repo (and
> server) this was seen against. Elrond, can you tell us more about
> this?

This is a private local repository.
git-cvsimport starts a local "cvs server" for this.

I tried to create a minimal repo, that would trigger the
same behaviour, but didn't succeed. My current guessing is,
that this happened in cooperation with "cvsps" caching.
(cvsps is a tool used by git-cvsimport).

I will move the cvsps cache for my problematic repo out of
the way and try a git-cvsimport from scratch to verify that
the above mentioned issue is related to cvsps caching.
But I still think, that handling all cvs pserver replies
should be done anyway in git-cvsimport (when it relies on
this mode of operation).


    Elrond

```