Re: sim checkout broken

[Tom Tromey <tromey@redhat.com>]

Tom> Could you say exactly what you are doing?

H-P> Oh my, seems I'm a victim of the reproducibility-ignorance-demon
H-P> despite extensive training.  Sorry about that.  Hubris, I guess. :/

Don't worry about it.

Tom> Indeed, if I run the same "cvs co sim" command twice, the second time I
Tom> get:
Tom> 
Tom> cvs checkout: existing repository /cvs/src/CVSROOT/Emptydir does not
Tom> match /cvs/src/src/gdb
Tom> 
Tom> ... but cvs still exits with status 0 for me.

H-P> Odd; it being 1 for me doesn't seem version-dependent.

Ugh.

H-P> cvs checkout: existing repository /cvs/src/CVSROOT/Emptydir does not
H-P> match /cvs/src/src/gdb

H-P> I don't think it would.  On another machine, at the second
H-P> checkout (with the first identical command populating the tree)
H-P> with a CVS version apparently matching yours, the same thing
H-P> happens:

H-P> cvs checkout: existing repository /cvs/src/CVSROOT/Emptydir does not
H-P> match /cvs/src/src/gdb
H-P> cvs checkout: ignoring module src/gdb/version.in
H-P> [hp@derp sim]$ echo $?
H-P> 1
H-P> [hp@derp sim]$ rpm -q cvs
H-P> cvs-1.11.23-25.fc17.x86_64

There must be some difference since:

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.

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.


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

sim really should not be using this file.

Tom