git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Shawn O. Pearce" <spearce@spearce.org>
To: Robin Rosenberg <robin.rosenberg@dewire.com>
Cc: git@vger.kernel.org
Subject: Re: [EGIT PATCH] Circument situations where Eclipse tries to decorate resources that do not exist
Date: Mon, 10 Mar 2008 19:40:28 -0400	[thread overview]
Message-ID: <20080310234028.GI8410@spearce.org> (raw)
In-Reply-To: <200803100033.53905.robin.rosenberg@dewire.com>

Robin Rosenberg <robin.rosenberg@dewire.com> wrote:
> This happens when the workspace has not been refreshed properly, e.g.
> when switching branches behind Eclipse's back.
> 
> Signed-off-by: Robin Rosenberg <robin.rosenberg@dewire.com>
> ---
>  .../internal/decorators/GitResourceDecorator.java  |    7 +++++++
>  1 files changed, 7 insertions(+), 0 deletions(-)
> 
> This one seems hard to replicate though I think it shouldn't. Maybe Eclipse refereshes on demand
> sometime and sometimes not.

Yup.  Earlier versions used to do a periodic refresh more often,
which really screwed up my Ant based build when Eclipse started
to _also_ build the same project to the same class folder at the
same time.

3.3.1.1 seems to not do this as often to me, and tends to wait for
a user-demanded refresh.  But then every once in a while it just
starts building all of a sudden, which makes me think they still
are doing a periodic refresh.  I don't know what is up with Eclipse.

And SWT is all about "we need access to the host native drawing APIs
to get good performance" but then Eclipse's resource system does
not appear to have filesystem monitors like inotify implemented for
the major OSes that Eclipse runs on.  Yet another aspect I don't get.

Yea, yea, if it bothers me, write the patch and submit it to Eclipse.
I'm too busy working on Git support.  I'll wait for someone else
to write it and submit it for me.  They may be waiting for me to
finish Git support first.  ;-)

> This stacktrace suggests this is the right thing to do.

Yea, I think so too.  Sucks.
 
-- 
Shawn.

      reply	other threads:[~2008-03-10 23:41 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-03-09 23:33 [EGIT PATCH] Circument situations where Eclipse tries to decorate resources that do not exist Robin Rosenberg
2008-03-10 23:40 ` Shawn O. Pearce [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20080310234028.GI8410@spearce.org \
    --to=spearce@spearce.org \
    --cc=git@vger.kernel.org \
    --cc=robin.rosenberg@dewire.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).