2010-03-17 08:27 AEST by Glen Starrett - Using a 2.5.03 build 3618 client to access a 2.8.01 build 3736 server results in
the following on my test rig:

C:\My_Documents\Test\Trial_Tests\sserver_test1_stuff>cvs up -P
cvsnt server: Updating .
cvsnt server: Updating More complicated directory name with spaces
cvsnt server: Updating MoreStuff
cvsnt server: cannot open directory . for empty check: No such file or directory

Normal "cvs up" works fine.

2010-03-17 08:28 AEST by Glen Starrett - Created an attachment (id=1771)
Trace of command

2010-03-17 08:28 AEST by Glen Starrett - Created an attachment (id=1772)
Output of command

2010-03-17 08:29 AEST by Glen Starrett - Created an attachment (id=1773)
cvsdiag from client

2010-03-17 08:30 AEST by Glen Starrett - Created an attachment (id=1774)
cvsdiag from server

2010-03-17 18:22 AEST by Arthur Barrett - add debugging
cvs commit: Examining src
Checking in src/update.cpp;
/cvs/cvsnt/src/update.c,v  <--  update.cpp
new revision: 1.44.2.132.6.25; previous revision: 1.44.2.132.6.24
done

2010-03-18 00:10 AEST by Glen Starrett - Created an attachment (id=1775)
RCS files of module stuff

2010-03-18 00:11 AEST by Glen Starrett - Created an attachment (id=1776)
ZIP of Sandbox

2010-03-18 00:17 AEST by Glen Starrett - One additional thing that may be relevant:  I have used the checkout -d ... when
creating the sandbox to change "stuff" to "sserver_test1_stuff".  I am within
that directory when executing the command.

2010-03-18 00:24 AEST by Glen Starrett - FYI, just tried a checkout without the -d to rename the directory, but the
result is the same (error when using -P).

2010-03-18 10:34 AEST by Arthur Barrett - opendir appears to fail on the default windows directory on Glens PC:

10:21:04: S -> do_dir_proc() -- call-back dir leave proc (if any) process_this_directory frame-
>dirleaveproc != NULL
10:21:04: S -> update_dirleave_proc(.,0,.) cwd=C:\WINDOWS\TEMP\cvs-serv1988
10:21:04: S -> isemptydir(.,0) cwd=C:\WINDOWS\TEMP
cvsnt server: cannot open directory . for empty check / exist error: No such file or directory


The importand questions are:
1) why is it trying
2) why does cvs suite 2008 not try this
3) why does it work with a suite 2009 client

eg: is the 'starting' directory different on suite 2008

eg: is something different sent to the server by suite 2009 client

2010-03-18 17:23 AEST by Arthur Barrett - add more debugging:

cvs commit: Examining src
Checking in src/update.cpp;
/cvs/cvsnt/src/update.c,v  <--  update.cpp
new revision: 1.44.2.132.6.26; previous revision: 1.44.2.132.6.25
done

2010-03-18 17:38 AEST by Arthur Barrett - Obviously bug only occurs if -P (update_prune_dirs) is set (as written in the 
subject of the bug).

The difference is that the 2.8.01 client is not sending -P to the server, here 
are CVS_CLIENT_LOG extracts from 2.5.03 .in:
read-cvsignore
read-cvswrappers
read-cvsrc2
Argument -d
Argument -P
Argument -I

And 2.8.01 .in:
read-cvsignore
read-cvswrappers
read-cvsrc2
Argument -I


In both cases I used the command line 'cvs up -dP'.  

There is no obvious difference in command option processing for 2.8.01 vs 
2.5.03 - need to debug and 'step through' to find the cause...

2010-03-18 18:17 AEST by Arthur Barrett - Changing subject from 'Update -P from 2.8.01 server by a 2.5.03 client reports 
error' to 'update -P from 2.8.01 client does not send -P'.  

Bug5860 is more like the original description 'Update -P from 2.8.01 server by 
a 2.5.03 client reports error'.

2010-03-18 19:00 AEST by Arthur Barrett - More debugging - this time the client - check that prune is being sent OK.

cvs commit: Examining src
Checking in src/update.cpp;
/cvs/cvsnt/src/update.c,v  <--  update.cpp
new revision: 1.44.2.132.6.27; previous revision: 1.44.2.132.6.26
done