27297 - cvs release requires write acccess to repository

Status: RESOLVED WORKSFORME

(mozilla.org Graveyard :: Server Operations, task, P3)

Description

[John G. Myers]

Using cvs-mirror.mozilla.org, I'm getting:

> cvs release -d ucvibm
? .deps
You have [0] altered files in this repository.
Are you sure you want to release (and delete) directory `ucvibm': yes
cvs-1.10.8 [server aborted]: "release" requires write access to the repository

This makes no sense; I should not need write access to the repository in order
to do a cvs release.

Comment 1

[Risto Kotalampi]

This is cvs client issue... and for some reason it tries to delete that dir from 
cvs-mirror, and of course that's not allowed.

Comment 2

[John G. Myers]

That's ludicrous.  The client is notifying the server of the release so the 
server can log the event.  If the server isn't interested, it should treat it as 
a noop.

Comment 3

[Daniel (Leaf) Nunes]

You don't have to get hostile with risto, john. This looks like a problem with
how we have the cvs-mirror set up, but it could be a limitation in cvs. I'm not
sure we can turn off trying to write the history file on releases. I suggest
working around with `rm -rf'.

Comment 4

[Risto Kotalampi]

I'm happy with leaf's suggestion... and don't want to spend more time with this 
right now. Maybe sometimes when I have more time. I'll leave this open but 
probably won't touch for a while.

Comment 5

[John G. Myers]

I apologize if my comment was hostile, it was not intended to be.

rm -rf has the problem that it doesn't take things out of CVS/Entries, so cvs 
starts puking on subsequent operations.

Comment 6

[Risto Kotalampi]

Moving to Drew. This is an issue with cvs-mirror.

Comment 7

[Drew Streib]

That error does indicate that "release" will require write access. This message
only shows up when it is more than the history file as well, as anonymous
checkouts technically require write access to the history file, but not write
access to the repository.

I'll check with Jim Kingdon, our resident CVS expert, as to why this is. But for
now it does appear to be a CVS constraint, and not necessarily a configurable
one (unless we want to hack CVS somewhat, which I think is not preferred).

Comment 8

[Dave Miller [:justdave]]

dave@ibook [0:58 docs 56] tcsh> cvs version
Client: Concurrent Versions System (CVS) 1.11.5 (client/server)
Server: Concurrent Versions System (CVS) 1.11.4 (client/server)
dave@ibook [0:59 docs 57] tcsh> cat CVS/Root
:pserver:[e-mail]:/cvsroot
dave@ibook [0:57 docs 58] tcsh> cvs release -d txt
You have [0] altered files in this repository.
Are you sure you want to release (and delete) directory `txt': yes
dave@ibook [0:57 docs 59] tcsh>

works for me.