This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 28179 - Invalid annotation after Refresh Recursively
Summary: Invalid annotation after Refresh Recursively
Status: VERIFIED FIXED
Alias: None
Product: obsolete
Classification: Unclassified
Component: vcsgeneric (show other bugs)
Version: 3.x
Hardware: PC Linux
: P3 blocker (vote)
Assignee: Martin Entlicher
URL:
Keywords: T9Y
: 28177 28178 (view as bug list)
Depends on:
Blocks:
 
Reported: 2002-10-22 17:38 UTC by David Kaspar
Modified: 2003-07-15 10:01 UTC (History)
0 users

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description David Kaspar 2002-10-22 17:38:14 UTC
How to reproduce:
1) Create "..../server" directory on your local FS
2) Create "..../client" directory on your local FS
3) Mount CVS filesystem using
"Versioning|Mount|VCS Generic" action
(use local cvs, "..../server" as server-side cvs
repository, "..../client" as client-side cvs
repository)
4) Do "CVS|Init" on FS root node
5) Do "CVS|Checkout" on FS root node
6) Use external FS manager for creating
".../client/some_directory" directory
7) Do "CVS|Refresh Recursively" action on root node
8) NOTE - OK - "some_directory" node is shown with
"Local" status
7) Do "CVS|Refresh Recursively" action on root
node once again
8) ERROR - "some_directory" node is shown but
WITHOUT any annotation
Comment 1 David Kaspar 2002-10-22 17:42:16 UTC
Adding T9Y keyword
Comment 2 Martin Entlicher 2002-10-22 19:06:41 UTC
This is probably some remaining problem in cache. Refresh does not
return local files, the cache should assure, that local files will be
set as [Local].
Comment 3 Martin Entlicher 2002-10-23 15:28:49 UTC
*** Issue 28178 has been marked as a duplicate of this issue. ***
Comment 4 Martin Entlicher 2002-10-23 15:29:22 UTC
*** Issue 28177 has been marked as a duplicate of this issue. ***
Comment 5 Martin Entlicher 2003-01-23 20:57:19 UTC
Well the Recursive Refresh worked worse that I thought. It used wrong
paths for directory containers after introduction of COMMON_PARENT
variable.
Fixed in the main trunk:

/cvs/vcscore/src/org/netbeans/modules/vcscore/cmdline/CommandLineVcsDirReaderRecursive.java,v 
<--  CommandLineVcsDirReaderRecursive.java
new revision: 1.25; previous revision: 1.24

/cvs/vcsgeneric/profiles/cvsprofiles/src/org/netbeans/modules/vcs/profiles/cvsprofiles/list/CvsListRecursiveCommand.java,v 
<--  CvsListRecursiveCommand.java
new revision: 1.6; previous revision: 1.5
Comment 6 Jiri Kovalsky 2003-01-24 13:59:21 UTC
I am sorry but the fix is not complete. If one runs recursive refresh
on root of filesystem, everything gets [Local] status. Therefore I am
reopening this issue to fix also this inconsistency. Trying to verify
in development build #200301240100 of NetBeans 4.0.
Comment 7 dmladek 2003-01-24 18:36:45 UTC
Well Martin, I hope I was testing it on right build where your fix
from Jan 23 should be propagated.
I'm using  NetBeans IDE Dev (Build 200301240100).

And in this build if I performing Refresh Recursively repeatedly on
externaly created dir....
Once this dir is [local] once it seems that it's been added into repo:-(

strange isn't it? Reopening...
Comment 8 dmladek 2003-01-24 18:38:51 UTC
:-) We were doing on the same bug at the same  time Jiri... :-)
Comment 9 Martin Entlicher 2003-01-28 17:40:05 UTC
O.K., so next attempt to fix this:

/cvs/vcscore/src/org/netbeans/modules/vcscore/caching/VcsFSCache.java,v 
<--  VcsFSCache.java
new revision: 1.35; previous revision: 1.34
/cvs/vcscore/src/org/netbeans/modules/vcscore/caching/VcsCacheDir.java,v 
<--  VcsCacheDir.java
new revision: 1.42; previous revision: 1.41
/cvs/vcscore/src/org/netbeans/modules/vcscore/caching/VcsCache.java,v 
<--  VcsCache.java
new revision: 1.50; previous revision: 1.49
/cvs/vcscore/src/org/netbeans/modules/vcscore/cache/CacheHandler.java,v 
<--  CacheHandler.java
new revision: 1.14; previous revision: 1.13
Comment 10 David Kaspar 2003-07-15 10:01:11 UTC
Verified in 3.5