From: "Shawn O. Pearce" <spearce@spearce.org>
To: Robin Rosenberg <robin.rosenberg.lists@dewire.com>
Cc: Marek Zawirski <marek.zawirski@gmail.com>, git@vger.kernel.org
Subject: Re: [EGIT PATCH 1/2] Add utilities for figuring out repositories for selected resources
Date: Tue, 2 Sep 2008 12:00:46 -0700 [thread overview]
Message-ID: <20080902190046.GE28704@spearce.org> (raw)
In-Reply-To: <200809022053.35419.robin.rosenberg.lists@dewire.com>
Robin Rosenberg <robin.rosenberg.lists@dewire.com> wrote:
> tisdagen den 2 september 2008 17.03.23 skrev Shawn O. Pearce:
> >
> > Hmm. So if any one of the selected projects doesn't have a Git
> > repository at its top level we just plain fail and pretend none of
> > them have a Git repository?
>
> That is the way everything in Eclipse work. For an operation to be applicable
> to a selection, it must be applicable to eache element.
OK. Good reason then.
> > But I also wonder if that really makes sense when a project could
> > have a linked resource under it that points to the repository's
> > working directory. In such cases we want operations on that project
> > to potentially impact the inner repository as maybe the project
> > repository does not have a repository itself. IOW I'm questioning
> > the idea of getRepositoriesFor(getProjectsForSelectedResources()).
>
> Ouch for linked resources. I really think we should ignore the link as much
> as we can. It's a trap! I any way it needs more fixing that this and not
> only here. A patch does not have to solve all problems in the world :)
>
> I have no experience with linked resources, other that I found them enough
> awkward to use to prevent from doing that. How useful and portable is
> a C:\foo on my system? I don't mind anyone solving the problem, but it is
> very low on my personal agenda.
Linked resources are useful (sometimes), especially if the project files
are automatically generated for you from your build system. My prior
day-job had a really nice build system that did this. My current day-job
also has a build system that can generate Eclipse project files for you.
In both cases the project files aren't considered part of the source as
they can be rebuilt at any time and they both used linked resources to
point to the actual place where the source was stored, which typically
was outside of your workspace directory, while the project was within
the workspace directory.
Anyway, I wasn't trying to suggest solving the worlds problems.
But RepositoryMapping is smarter now and can (mostly) handle a
linked resource just fine. Its the higher level GUI actions that
run into trouble.
I guess we should stick this series in as-is then.
--
Shawn.
prev parent reply other threads:[~2008-09-02 19:01 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-08-31 9:38 [EGIT PATCH] Select all changes in repository for commit by default Robin Rosenberg
2008-08-31 9:38 ` [EGIT PATCH 1/2] Add utilities for figuring out repositories for selected resources Robin Rosenberg
2008-08-31 9:38 ` [EGIT PATCH 2/2] Enable commit for any resource in a Git-shared project Robin Rosenberg
2008-09-02 15:03 ` [EGIT PATCH 1/2] Add utilities for figuring out repositories for selected resources Shawn O. Pearce
2008-09-02 18:53 ` Robin Rosenberg
2008-09-02 19:00 ` 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=20080902190046.GE28704@spearce.org \
--to=spearce@spearce.org \
--cc=git@vger.kernel.org \
--cc=marek.zawirski@gmail.com \
--cc=robin.rosenberg.lists@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).