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 109277 - Moved or deleted folder not visible as a change
Summary: Moved or deleted folder not visible as a change
Status: RESOLVED FIXED
Alias: None
Product: versioncontrol
Classification: Unclassified
Component: CVS (show other bugs)
Version: 6.x
Hardware: All All
: P2 blocker (vote)
Assignee: issues@versioncontrol
URL:
Keywords:
Depends on: 62863 68320 89340
Blocks:
  Show dependency tree
 
Reported: 2007-07-10 18:05 UTC by Jesse Glick
Modified: 2007-07-13 14:05 UTC (History)
0 users

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments
Log file excerpt (1.24 KB, text/plain)
2007-07-10 18:29 UTC, Jesse Glick
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Jesse Glick 2007-07-10 18:05:00 UTC
070709. I went to nb_all in Favorites, selected openide/looks, Cut, selected contrib, Paste. Move was successful (as far
as I can tell) and contrib/looks shows up with the new badge. But openide has no modified badge, and Show Changes does
not show anything local. If I Update openide, openide/looks is resurrected! I delete it again, and try Show Changes
again; still nothing. I cannot commit the removal. So basically NB is unable to delete a folder in CVS.
Comment 1 Jesse Glick 2007-07-10 18:11:28 UTC
Workaround is to manually delete all files in dir:

cd openide/looks
cvs rm -f `perl -MFile::Find -e 'find sub {if (/^CVS$/) {$File::Find::prune = 1; return} $i = ".cvsignore"; if (-f $i)
{local (*I); open I, $i or die $i; while ($r = <I>) {chomp $r; $r =~ s/\./\\./g; $r =~ s/\*/.*/g; if (/^$r$/)
{$File::Find::prune = 1; return}}} $n = $File::Find::name; $n =~ s!^\./!!; print "$n\n" if -f and not $n =~ m!^CVS/!},
@ARGV' * .[a-zA-Z]*`
Comment 2 Jesse Glick 2007-07-10 18:28:39 UTC
To make things worse, even after the workaround to delete openide/looks, the final commit failed adding contrib/looks,
saying "CVS server: nothing known about ..." for each dir in contrib/looks. So it seems I will need to use command-line
CVS for adding files, too.
Comment 3 Jesse Glick 2007-07-10 18:29:32 UTC
Created attachment 44899 [details]
Log file excerpt
Comment 4 Jesse Glick 2007-07-10 19:01:53 UTC
Finally managed to get everything fixed up from command-line CVS, but it was a lot of work; CVS/ subdirs were present
irregularly in some dirs in contrib/looks, which caused 'cvs add' to fail in some cases. Next general (nb_all) CVS
update had many errors:

cvs update: move away `contrib/looks/api/doc/overview.html'; it is in the way
? contrib/looks/api
? contrib/looks/test/unit/src
cvs update: move away `contrib/looks/api/doc/changes/apichanges.xml'; it is in the way
C contrib/looks/api/doc/overview.html
cvs update: move away `contrib/looks/api/doc/org/netbeans/api/nodes2looks/package.html'; it is in the way
cvs update: move away `contrib/looks/api/doc/org/netbeans/spi/looks/package.html'; it is in the way
cvs update: move away `contrib/looks/api/doc/org/netbeans/spi/looks/doc-files/about.html'; it is in the way
C contrib/looks/api/doc/changes/apichanges.xml
C contrib/looks/api/doc/org/netbeans/api/nodes2looks/package.html
C contrib/looks/api/doc/org/netbeans/spi/looks/package.html
cvs update: move away `contrib/looks/api/doc/org/netbeans/spi/looks/doc-files/howto.html'; it is in the way
C contrib/looks/api/doc/org/netbeans/spi/looks/doc-files/about.html
cvs update: move away `contrib/looks/test/unit/src/org/netbeans/api/nodes2looks/HierarchyTest.java'; it is in the way
cvs update: move away `contrib/looks/test/unit/src/org/netbeans/api/nodes2looks/IsLeafTest.java'; it is in the way
C contrib/looks/api/doc/org/netbeans/spi/looks/doc-files/howto.html
C contrib/looks/test/unit/src/org/netbeans/api/nodes2looks/HierarchyTest.java
cvs update: move away `contrib/looks/test/unit/src/org/netbeans/api/nodes2looks/LookNodeEventsTest.java'; it is in the way
cvs update: move away `contrib/looks/test/unit/src/org/netbeans/api/nodes2looks/LookNodeGCTest.java'; it is in the way
C contrib/looks/test/unit/src/org/netbeans/api/nodes2looks/IsLeafTest.java
C contrib/looks/test/unit/src/org/netbeans/api/nodes2looks/LookNodeEventsTest.java
cvs update: move away `contrib/looks/test/unit/src/org/netbeans/api/nodes2looks/LookNodeLookupTest.java'; it is in the way
C contrib/looks/test/unit/src/org/netbeans/api/nodes2looks/LookNodeGCTest.java
cvs update: move away `contrib/looks/test/unit/src/org/netbeans/api/nodes2looks/LookNodeTest.java'; it is in the way
C contrib/looks/test/unit/src/org/netbeans/api/nodes2looks/LookNodeLookupTest.java
cvs update: move away `contrib/looks/test/unit/src/org/netbeans/api/nodes2looks/LookNodeValuesTest.java'; it is in the way
cvs update: move away `contrib/looks/test/unit/src/org/netbeans/api/nodes2looks/NodeLookValuesTest.java'; it is in the way
C contrib/looks/test/unit/src/org/netbeans/api/nodes2looks/LookNodeTest.java
C contrib/looks/test/unit/src/org/netbeans/api/nodes2looks/LookNodeValuesTest.java
cvs update: move away `contrib/looks/test/unit/src/org/netbeans/api/nodes2looks/TestUtil.java'; it is in the way
cvs update: move away `contrib/looks/test/unit/src/org/netbeans/spi/looks/CompositeLookValuesTest.java'; it is in the way
C contrib/looks/test/unit/src/org/netbeans/api/nodes2looks/NodeLookValuesTest.java
C contrib/looks/test/unit/src/org/netbeans/api/nodes2looks/TestUtil.java
cvs update: move away `contrib/looks/test/unit/src/org/netbeans/spi/looks/DecoratorSelectorEventsTest.java'; it is in
the way
C contrib/looks/test/unit/src/org/netbeans/spi/looks/CompositeLookValuesTest.java
cvs update: move away `contrib/looks/test/unit/src/org/netbeans/spi/looks/DecoratorSelectorValuesTest.java'; it is in
the way
C contrib/looks/test/unit/src/org/netbeans/spi/looks/DecoratorSelectorEventsTest.java
cvs update: move away `contrib/looks/test/unit/src/org/netbeans/spi/looks/DefaultLookValuesTest.java'; it is in the way
C contrib/looks/test/unit/src/org/netbeans/spi/looks/DecoratorSelectorValuesTest.java
cvs update: move away `contrib/looks/test/unit/src/org/netbeans/spi/looks/EventDeliveryTest.java'; it is in the way
C contrib/looks/test/unit/src/org/netbeans/spi/looks/DefaultLookValuesTest.java
cvs update: move away `contrib/looks/test/unit/src/org/netbeans/spi/looks/FilterLookMaskTest.java'; it is in the way
C contrib/looks/test/unit/src/org/netbeans/spi/looks/EventDeliveryTest.java
cvs update: move away `contrib/looks/test/unit/src/org/netbeans/spi/looks/GoldenEvent.java'; it is in the way
C contrib/looks/test/unit/src/org/netbeans/spi/looks/FilterLookMaskTest.java
cvs update: move away `contrib/looks/test/unit/src/org/netbeans/spi/looks/GoldenValue.java'; it is in the way
C contrib/looks/test/unit/src/org/netbeans/spi/looks/GoldenEvent.java
cvs update: move away `contrib/looks/test/unit/src/org/netbeans/spi/looks/LookListenersTest.java'; it is in the way
C contrib/looks/test/unit/src/org/netbeans/spi/looks/GoldenValue.java
cvs update: move away `contrib/looks/test/unit/src/org/netbeans/spi/looks/LookSizeTest.java'; it is in the way
cvs update: move away `contrib/looks/test/unit/src/org/netbeans/spi/looks/LookSwitcherLookLookupTest.java'; it is in the way
cvs update: move away `contrib/looks/test/unit/src/org/netbeans/spi/looks/LookValuesTest.java'; it is in the way
cvs update: move away `contrib/looks/test/unit/src/org/netbeans/spi/looks/LooksTest.java'; it is in the way
C contrib/looks/test/unit/src/org/netbeans/spi/looks/LookListenersTest.java
C contrib/looks/test/unit/src/org/netbeans/spi/looks/LookSizeTest.java
C contrib/looks/test/unit/src/org/netbeans/spi/looks/LookSwitcherLookLookupTest.java
C contrib/looks/test/unit/src/org/netbeans/spi/looks/LookValuesTest.java
cvs update: move away `contrib/looks/test/unit/src/org/netbeans/spi/looks/LooksTest.xml'; it is in the way
C contrib/looks/test/unit/src/org/netbeans/spi/looks/LooksTest.java
cvs update: move away `contrib/looks/test/unit/src/org/netbeans/spi/looks/NamespaceSelectorEventsTest.java'; it is in
the way
C contrib/looks/test/unit/src/org/netbeans/spi/looks/LooksTest.xml
cvs update: move away `contrib/looks/test/unit/src/org/netbeans/spi/looks/NamespaceSelectorTest.java'; it is in the way
cvs update: move away `contrib/looks/test/unit/src/org/netbeans/spi/looks/NodeLookEventsTest.java'; it is in the way
C contrib/looks/test/unit/src/org/netbeans/spi/looks/NamespaceSelectorEventsTest.java
C contrib/looks/test/unit/src/org/netbeans/spi/looks/NamespaceSelectorTest.java
cvs update: move away `contrib/looks/test/unit/src/org/netbeans/spi/looks/ProxyLookEventsTest.java'; it is in the way
cvs update: move away `contrib/looks/test/unit/src/org/netbeans/spi/looks/ProxyLookEventsWithDelegateObjectTest.java';
it is in the way
C contrib/looks/test/unit/src/org/netbeans/spi/looks/NodeLookEventsTest.java
C contrib/looks/test/unit/src/org/netbeans/spi/looks/ProxyLookEventsTest.java
cvs update: move away `contrib/looks/test/unit/src/org/netbeans/spi/looks/ProxyLookTest.java'; it is in the way
C contrib/looks/test/unit/src/org/netbeans/spi/looks/ProxyLookEventsWithDelegateObjectTest.java
cvs update: move away `contrib/looks/test/unit/src/org/netbeans/spi/looks/ProxyLookValuesTest.java'; it is in the way
C contrib/looks/test/unit/src/org/netbeans/spi/looks/ProxyLookTest.java
cvs update: move away `contrib/looks/test/unit/src/org/netbeans/spi/looks/SampleLook.java'; it is in the way
C contrib/looks/test/unit/src/org/netbeans/spi/looks/ProxyLookValuesTest.java
cvs update: move away `contrib/looks/test/unit/src/org/netbeans/spi/looks/SampleProvider.java'; it is in the way
cvs update: move away `contrib/looks/test/unit/src/org/netbeans/spi/looks/SampleRepObject.java'; it is in the way
C contrib/looks/test/unit/src/org/netbeans/spi/looks/SampleLook.java
C contrib/looks/test/unit/src/org/netbeans/spi/looks/SampleProvider.java
cvs update: move away `contrib/looks/test/unit/src/org/netbeans/spi/looks/SelectorEventTest1.xml'; it is in the way
cvs update: move away `contrib/looks/test/unit/src/org/netbeans/spi/looks/SelectorEventTest2.xml'; it is in the way
cvs update: move away `contrib/looks/test/unit/src/org/netbeans/spi/looks/SelectorTest.java'; it is in the way
C contrib/looks/test/unit/src/org/netbeans/spi/looks/SampleRepObject.java
C contrib/looks/test/unit/src/org/netbeans/spi/looks/SelectorEventTest1.xml
C contrib/looks/test/unit/src/org/netbeans/spi/looks/SelectorEventTest2.xml
cvs update: move away `contrib/looks/test/unit/src/org/netbeans/spi/looks/TestBaseEvents.java'; it is in the way
C contrib/looks/test/unit/src/org/netbeans/spi/looks/SelectorTest.java
cvs update: move away `contrib/looks/test/unit/src/org/netbeans/spi/looks/TestBaseValues.java'; it is in the way
C contrib/looks/test/unit/src/org/netbeans/spi/looks/TestBaseEvents.java
C contrib/looks/test/unit/src/org/netbeans/spi/looks/TestBaseValues.java

Need to delete the newly added folder and check it out fresh, I guess. Would have been better off using the command line
from the beginning.
Comment 5 Jesse Glick 2007-07-10 19:05:47 UTC
I have tried this sort of operation in the past and it has failed in the past too, but I thought all that had been
fixed; apparently not. Isn't there any sort of test for this (IMHO very basic) operation to make sure the implementation
does not regress?
Comment 6 Tomas Stupka 2007-07-13 14:05:59 UTC
regression caused with the fix for issue #105884. 
/cvs/versioncontrol/src/org/netbeans/modules/versioning/FilesystemInterceptor.java,v  <--  FilesystemInterceptor.java
new revision: 1.10; previous revision: 1.09

goiong to close this one as the mentioned fix was rolled back and the reported problem should be gone now. 

However, note that issue #105884 still remains and a cut & paste action will lead to another kind of misbehavior - see
issue #105884, and issue #109421.