Fix RevisionSyntaxException in the Push wizard

The refspec panel passed strings like "master [branch]" to the revision
parser. The revision parser used to allow garbage after the revision,
but was tightened, which then triggered this bug.

Instead of working with the content proposal label, add a method to get
the corresponding content. Then use this when working with ref specs.

Another unhandled RevisionSyntaxException was in tryResolveLocalRef,
e.g. when calling completion with "refs/".

Bug: 393227
Change-Id: Id9b21ab851bb715264c304f08821df5323698d80
Add Show In support for files in history and commit view

Bug: 363567
Change-Id: Ib89e5a5380d30a5120d874f02635aaf8f74b68a5
Signed-off-by: Dani Megert <Daniel_Megert@ch.ibm.com>
Increase default SWTBot timeout to 20 seconds

Too many (otherwise good) UI tests are failing because they run into a
timeout, lessening the usefulness of them.

Note that polling is still done each 500 ms as per SWTBot default, so
the tests should not take longer in general.

Bug: 345812
Change-Id: I17ac04228f4cf5a7df0c9066d5d4e853c2ab9fc5
[stagingView] Add Show In support for unstaged/staged files

Bug: 363567
Change-Id: I00bf92b16dcf3887d0596c87a386830770b68e28
Merge "[stagingView] Add Show In support for unstaged/staged files"
Remove redundant implements declarations

Unlike the change in I3520bc4a9 they are not necessarily illegal.

Change-Id: I859f5e83a4aabcd4fe4ce28950af417336613ecf
Signed-off-by: Robin Stocker <robin@nibor.org>
GitSubscriberMergeContextTest: Fix internal class warning

Change-Id: Iccf07057a9c06f4c63c0493e926a7350e2d45e79
Remove old commented code from RepositoriesView

Delete of repositories has been implemented with
Ice562d007d71dd1f3493f3ae748ec5ba21e6330a.

Change-Id: Id5d533f0843e304ccfdd116b7552bb4fe6dd77d8
Merge "Remove old commented code from RepositoriesView"
Make GitRepositoriesViewRepoDeletionTest more reliable

Add a job family and use joinJobs instead of waitInUI.

Change-Id: I6e2710c8b598fce260f046a9ca0815c2854a1d4a
Merge "Fix RevisionSyntaxException in the Push wizard"
[stagingView] Add Show In to context menu of files

UIUtils now contains a method for creating the show in menu including
the active key binding. This can later be reused in other places, so
that the key binding is shown everywhere.

Bug: 363567
Change-Id: I4e56160a2e690bd94307527204c51bad311add4c
Signed-off-by: Matthias Sohn <matthias.sohn@sap.com>
Limit version range for dependencies in 'org.eclipse.egit.import'

Change-Id: I968c8ebad7d1ead2597ba10b1af99237053aafcb
Signed-off-by: Jens Baumgart <jens.baumgart@sap.com>
Signed-off-by: Matthias Sohn <matthias.sohn@sap.com>
Signed-off-by: Dani Megert <Daniel_Megert@ch.ibm.com>
Use IPath instead of IResource for non-workspace paths

The implementations of getMappingForNonWorkspaceResource() and
getRepoRelativePath() in RepositoryMapping were confusing workspace
paths with filesystem locations.

The problem is that IResource#getFullPath() returns e.g. /foo/bar (where
/ means workspace root), whereas IResource#getLocation() returns the
absolute file system path. This caused later code which treated the
paths as absolute file system paths to return incorrect results.

Instead of incorrectly using an IResource for non-workspace files, we
now work with IPath if necessary, e.g. for comparing with index from
Staging View.

Bug: 378948
Change-Id: Ifdda37fc51a0cb29145e1c29b09d45317fc068ea
Signed-off-by: Robin Stocker <robin@nibor.org>
Fix LocalNonWorkspaceTypedElement using getFullPath for absolute path

Similar to bug 378948, it was confusing workspace paths with filesystem
locations.

Change it to accept an IPath directly and work with that to get to the
File object. Also, correct the error handling which passed null as an
IStatus causing an NPE.

Bug: 394166
Change-Id: I63364bd6d5f51fda5a69ab88793b680adb7fe574
Use UIUtils.createShowInMenu to unify label and key binding

This also results in Git Tree Compare and History/Commit Editor views to
now also show the key binding for the Show In menu.

Bug: 363567
Change-Id: Ia8a9caf8f3268f22e2f1480caa29c41b1a110734
Signed-off-by: Dani Megert <Daniel_Megert@ch.ibm.com>
Fix index diff refresh in LocalNonWorkspaceTypedElement after save

With Ifdda37fc51a0cb29145e1c29b09d45317fc068ea, the mapping is no longer
returned for non-existing resources. So use the path to get the mapping.

Also remove the unnecessary check for IFile, which is always true.

Bug: 394166
Change-Id: Ifb13e8f99edd8695f2672f93d2700db42cdadfa6
Signed-off-by: Matthias Sohn <matthias.sohn@sap.com>
Depend on org.eclipse.core.runtime >= 3.7

Since If7a0d7bfb4dd98d2373531eee63f042376cce228, we depend on 3.7 for
the INSTANCE fields of InstanceScope/DefaultScope.

Without this change, it's still possible to install EGit into Eclipse
3.5/3.6, but running it will cause errors because of the missing fields.

Bug: 365425
Change-Id: Ide4bfe98c85ecc2719968540c20f0f9c13729039
Signed-off-by: Matthias Sohn <matthias.sohn@sap.com>