[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [Subclipse-dev] Re: subclipse svn commit: r5091 - trunk/subclipse/org.tigris.subversion.subclipse.core/src/org/tigris/subversion/subclipse/core/resources/SVNMoveDeleteHook.java

From: Mark Phippard <markphip_at_gmail.com>
Date: Mon, 22 Aug 2011 09:53:39 -0400

The console showed SVN messages about delete, so that means it went to our
NotificationHandler. Do we sent notifications to refresh resources when we
receive the delete notification from SVN?

On Mon, Aug 22, 2011 at 9:50 AM, Stephen Elsemore <selsemore_at_collab.net>wrote:

> My guess would be that we didn’t receive the Delete notifications because
> our move/delete hook did not trigger them, and the Eclipse delete (which
> would have triggered them) did not happen. I’ll look into doing everything
> in our move/delete hook.****
>
> ** **
>
> *From:* Mark Phippard [mailto:markphip_at_gmail.com]
> *Sent:* Monday, August 22, 2011 6:47 AM
>
> *To:* dev_at_subclipse.tigris.org
> *Subject:* Re: [Subclipse-dev] Re: subclipse svn commit: r5091 -
> trunk/subclipse/org.tigris.subversion.subclipse.core/src/org/tigris/subversion/subclipse/core/resources/SVNMoveDeleteHook.java
> ****
>
> ** **
>
> Reading between the lines, it sounds like you are saying you are doing this
> so that Eclipse will trigger the proper refreshes after the operation
> completes?****
>
> ** **
>
> Did you look at why our NotificationHandler would not simply do it when it
> receives the Delete notifications?****
>
> ** **
>
> ** **
>
> On Mon, Aug 22, 2011 at 9:45 AM, Stephen Elsemore <selsemore_at_collab.net>
> wrote:****
>
> Eclipse doesnt have a problem if the resource has actually already been
> deleted. But I suppose this approach could cause a problem if there is some
> other move/delete hook in place that will get an error if it tries to handle
> the delete and the resource is not there. The alternative solution is to
> make sure our move/delete hook replicates everything that the Eclipse delete
> does.****
>
> *From:* Mark Phippard [mailto:markphip_at_gmail.com]
> *Sent:* Monday, August 22, 2011 6:36 AM
> *To:* dev_at_subclipse.tigris.org
> *Subject:* [Subclipse-dev] Re: subclipse svn commit: r5091 -
> trunk/subclipse/org.tigris.subversion.subclipse.core/src/org/tigris/subversion/subclipse/core/resources/SVNMoveDeleteHook.java
> ****
>
> Drive by review ...****
>
> Is this the right thing to do? Doesn't SVN delete the actual folder/file?
> Is this change only applying to some specific scenario?****
>
> On Mon, Aug 22, 2011 at 9:33 AM, <selsemore_at_tigris.org> wrote:****
>
> Author: selsemore
> Date: 2011-08-22 06:33:24-0700
> New Revision: 5091
>
> Modified:
>
> trunk/subclipse/org.tigris.subversion.subclipse.core/src/org/tigris/subversion/subclipse/core/resources/SVNMoveDeleteHook.java
>
> Log:
> Move/delete hook should return false for deleteFile and deleteFolder
> methods
> (so that Eclipse will do its thing and do the actual delete after Subclipse
> is done).
>
> Modified:
> trunk/subclipse/org.tigris.subversion.subclipse.core/src/org/tigris/subversion/subclipse/core/resources/SVNMoveDeleteHook.java
> Url:
> http://subclipse.tigris.org/source/browse/subclipse/trunk/subclipse/org.tigris.subversion.subclipse.core/src/org/tigris/subversion/subclipse/core/resources/SVNMoveDeleteHook.java?view=diff&pathrev=5091&r1=5090&r2=5091
>
> ==============================================================================
> ---
> trunk/subclipse/org.tigris.subversion.subclipse.core/src/org/tigris/subversion/subclipse/core/resources/SVNMoveDeleteHook.java
> (original)
> +++
> trunk/subclipse/org.tigris.subversion.subclipse.core/src/org/tigris/subversion/subclipse/core/resources/SVNMoveDeleteHook.java
> 2011-08-22 06:33:24-0700
> @@ -74,15 +74,12 @@
> deletedFiles.add(file);
>
> deleteResource(resource);
> -
> - tree.deletedFile(file);
> -
> } catch (SVNException e) {
> tree.failed(e.getStatus());
> } finally {
> monitor.done();
> }
> - return true;
> + return false;
>
> }
>
> @@ -113,8 +110,7 @@
> } finally {
> monitor.done();
> }
> - return true;
> -
> + return false;
> }
>
> /*
>
> ------------------------------------------------------
>
> http://subclipse.tigris.org/ds/viewMessage.do?dsForumId=3541&dsMessageId=2827597
>
> To unsubscribe from this discussion, please e-mail [unsubscribeURL]****
>
>
>
> ****
>
> --
> Thanks
>
> Mark Phippard
> http://markphip.blogspot.com/****
>
>
>
> ****
>
> ** **
>
> --
> Thanks
>
> Mark Phippard
> http://markphip.blogspot.com/****
>

-- 
Thanks
Mark Phippard
http://markphip.blogspot.com/
------------------------------------------------------
http://subclipse.tigris.org/ds/viewMessage.do?dsForumId=1043&dsMessageId=2827607
To unsubscribe from this discussion, e-mail: [dev-unsubscribe_at_subclipse.tigris.org].
Received on 2011-08-22 15:53:45 CEST

This is an archived mail posted to the Subclipse Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.