Re: sim checkout broken

[Hans-Peter Nilsson <hp@bitrange.com>]

On Wed, 26 Jun 2013, Tom Tromey wrote:
> barimba. cvs -d  :pserver:anoncvs@sourceware.org:/cvs/src co sim
> [...]
> cvs checkout: Updating src/intl
> cvs checkout: existing repository /cvs/src/CVSROOT/Emptydir does not match /cvs/src/src/gdb
> cvs checkout: ignoring module src/gdb/version.in
> barimba. echo $?
> 0
>
> I have:
>
> barimba. rpm -q cvs
> cvs-1.11.23-29.fc18.x86_64
>
> Based on "rpm -q --changelog" it is hard to believe that anything
> between -25 and -29 changed this.

What about ~/.cvsrc?
I just have a single line:
cvs -z3

(I also had a "-q" there before, but removed it to try to match
your output and the last three lines are now the same.  Still
does exit(1) on the update, though.  Also still exit(0) for the
original checkout.  I also tried adding a "checkout -P" there,
no change.)

> H-P> But what is the cause of the error?  Can't we just remove that
> H-P> Emptydir or revert what was done? The message above seems to
> H-P> indicate an operational error in whatever was done.
>
> Based on my searches it seems to be a CVS issue.
> CVS, IIUC, puts "Emptydir" into the Repository file of some directories
> in some situations.  This later confuses it.
> I don't really understand this, though, since no Repository file in my
> checkout actually has this.

Bah.

> Maybe we should just give up on part of this patch and move the
> version.in file back out of "common".

Give up?  What's that? :P

> sim really should not be using this file.

Yeah well maybe, but gdb/version.in (or somesuch) was necessary
to build at the time and adding it to the sim module was the
pragmatic solution at the time.  I didn't do it myself, but I
did suggest it or at least agreed to it, IIRC. :}

brgds, H-P