Project

General

Profile

Actions

Bug #767

closed

add csup to world

Added by skif over 16 years ago. Updated about 15 years ago.

Status:
Closed
Priority:
Low
Assignee:
-
Category:
-
Target version:
-
Start date:
Due date:
% Done:

0%

Estimated time:

Actions #1

Updated by corecode over 16 years ago

We think we should try to move away from cvsup. Csup is just the client part and thus doesn't help with running the server.

cheers
simon

Actions #2

Updated by pavalos over 16 years ago

Although I agree that this doesn't help with the server part (mainly used by
the source mirrors), won't this help end-user clients?

Actions #3

Updated by corecode over 16 years ago

But why should we add this to world? Wouldn't it be better to add this as a pkgsrc package?

cheers
simon

Actions #4

Updated by erik-wikstrom over 16 years ago

It's already in pkgsrc, under /net.

Actions #5

Updated by pavalos over 16 years ago

I'm not sure. If it's in pkgsrc, can anyone think of why we need it
in world?

--Peter

Actions #6

Updated by wa1ter over 16 years ago

The csup in pkgsrc still doesn't work properly with the version of CVS
used in DF. IIRC Yonetani-san posted a patch many months ago.

Actions #7

Updated by qhwt+dfly over 16 years ago

This comes from how cvsup servers out there respond to a client when it
found an RCS keyword unknown to it (refered to as "newphrase" in rcsfile(5)).
in an RCS file. In such a case, cvsup server tries to send a delta command
with that unknown keyword (marked as 'n' or 'N' as desribed in doc/Protocol
found in cvsup source distribution). The server sends this special command
even if it's running in check out mode, in which clients usually doesn't care
about the changes in RCS keywords (well, at least as of cvs-1.12.13, future
versions of cvs or rcs may add a new keyword which may affect checkouts).

The current cvsup client code doesn't do anything for such commands.
The current csup implementation aborts as soon as it found a delta command
with sub commands other than 'L', 'S', or 'T'.
My patch changes csup to also accept the delta commands with 'n'/'N'
and do nothing.

Maxime seemed to consider it was a bug in cvsup server or the protocol
to send delta commands which have nothing to do with checkout mode, so
he didn't want to add a workaround for no reason. I agree with this,
but I also care about how long it takes for cvsup mirrors around the world
to update to a newer cvsup version, especially when the problem is only
critical to csup users trying to checkout a CVS repository containing
commitid keyword.

Cheers.

Actions #8

Updated by corecode over 16 years ago

So this patch probably should be included as pkgsrc patch then?

It disturbs me that authors keep sticking to their code, even if it is obvious that it is broken or at least inoperable. I had the same experience with the cvsync authors, who just didn't want to fix their protocol, so that the resulting RCS files wouldn't become invalid. Maybe I should simply write a cvs syncer myself, but then I wonder if I should invest energy in a tool that should be replaced anyways. Regarding CVS syncers, do people think that cvsync's or cvsup's protocol design is better?

cheers
simon

Actions #9

Updated by qhwt+dfly over 16 years ago

I'll post a patch to pkgsrc-bug if he doesn't reply to me in a few days.

Cheers.

Actions #10

Updated by pavalos about 15 years ago

Now that we've switched to git, we don't need this in base.

Actions

Also available in: Atom PDF