EGit v5.8.0.202006091008-r

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

Try different way to skip pack200 packing for org.eclipse.egit.doc

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

Update documentation for 5.8

Change-Id: I7780a4e15c52b4bb1f252f670b044013b9906576
Signed-off-by: Thomas Wolf <thomas.wolf@paranor.ch>
Signed-off-by: Matthias Sohn <matthias.sohn@sap.com>

Add reference to egit-github repository

This could help users having an installation of both egit and the github
integration and trying to upgrade to the latest release or nightly build
using one of the egit update sites and have not checked "Contact all
update sites". Add references in org.eclipse.jgit.repository/pom.xml
using JBoss repository-utils plugin.

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

Rename category experimental to additional and move mylyn feature to it

The gitflow feature is in the wild for a long time now and isn't
experimental anymore. Hence rename EGit category "experimental" to
"additional".

In 2020-06 mylyn is no longer part of most EPP packages hence move it
also to the new category "additional".

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

Merge branch 'master' into stable-5.8

* master:
  Remove packed version of egit.doc from update site
  Layout fix in RepositorySelectionPage
  Fix QuickOutline command in DiffEditor
  Show repository name in commit dialog
  Show visible and overall staging count when filtering
  Update Orbit to R20200529191137 for final 2020-06
  Add the new org.eclipse.jgit.gpg.bc feature needed to sign commits
  Add the new org.eclipse.jgit.ssh.jsch feature to use jsch
  Don't install the WindowCache JMX bean
  Update jetty to 9.4.28.v20200408
  Add 4.16 staging target platform
  Rename interactive rebase job
  Block commit on error
  Disable "sign" button if no GpgSigner is present
  [releng] Update Orbit link in p2 repository to S20200519202422
  Update Orbit to S20200519202422 and ant to 1.10.8
  StagingView: speed up updating staged/unstaged viewers
  Staging View: Allow "Push HEAD" when there is nothing to commit
  CommitFileDiffViewer: fire SelectionChangedEvent after selectAll
  Staging View: Always show dialog for Commit And Push...
  Dynamic label for "Open in Commit Viewer"
  Interactive rebase: show view if successfully prepared
  Staging View: "Commit and Push" uses Gerrit dialog only w/ Gerrit remote
  Minor clean-up in NodesByCommitTimeFilter

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

Remove packed version of egit.doc from update site

egit.doc is the biggest contributor to the update site. The pack200
compressed version is almost as big as the not packed version. Therefore
do not even try to create a packed version of that bundle. This removes
5 MB from each update site that we publish.

This cannot be tested locally, since the signing profile cannot be
executed locally. Running with the default profile works fine locally.

Change-Id: I49908c706693d2bd01effb0716f596818effdc37
Signed-off-by: Michael Keppler <Michael.Keppler@gmx.de>
Signed-off-by: Matthias Sohn <matthias.sohn@sap.com>

Merge changes I7826cc17,Idd6ae15f

* changes:
  Show repository name in commit dialog
  Show visible and overall staging count when filtering

Layout fix in RepositorySelectionPage

When configuring a push URI, the button to select a local bundle
file is not shown. However, the button to select a local directory
as upstream repository was set to span 3 columns in that case. That's
one too many, and broke the layout of the page completely in that
case.

Reduce the span to 2 columns: the one for the "Local Folder..."
button plus the one for the (not shown) "Local Bundle File..." button.

Change-Id: I2df6dc9bf466cd638b8aa0bdbcb9f3aff01c5c16
Signed-off-by: Thomas Wolf <thomas.wolf@paranor.ch>

Fix QuickOutline command in DiffEditor

1. To make the command and its key binding show up in the key bindings
   preference page, the command needs to have a category.
2. To make the texts displayed in that preference page make sense, the
   command must have a sensible description.
3. The command name should just be "Quick Outline" similar to the other
   commands from Java, Mylyn, or PDE.
4. The context "org.eclipse.egit.ui.DiffViewer" for the keybinding must
   be a child of textEditorScope, otherwise there can be key conflicts.
   (For instance, lsp4e uses M1+O in the textEditorScope for "Go to
   Symbol in File". With our binding in the window-wide context, this
   conflicts. With our binding in a child of textEditorScope, it won't
   conflict but override.)
5. The DiffEditor must set the "org.eclipse.egit.ui.DiffViewer" context.

Bug: 563986
Change-Id: I08f549ddf038a2d1b9520cf75d99bc596241df16
Signed-off-by: Thomas Wolf <thomas.wolf@paranor.ch>

Show repository name in commit dialog

We recently added the repository to some other dialogs that can be
invoked via Ctrl-3. Similarly the commit dialog should show the
repository we are committing to.

Change-Id: I7826cc17b5e977be89649afaaee6d26c708c13ea
Signed-off-by: Michael Keppler <Michael.Keppler@gmx.de>

Show visible and overall staging count when filtering

When the file name filter in the staging view is active, show both the
visible and overall number of staging entries, even if they are the
same.

This makes it more explicit that a filter has been applied (since there
can be a short delay between the input of the filter text and the
refresh of the view, especially with other jobs working concurrently).

Bug:553053
Change-Id: Idd6ae15f4b6dd5c5fe3967eddc813fac00875e5a
Signed-off-by: Michael Keppler <Michael.Keppler@gmx.de>

Update Orbit to R20200529191137 for final 2020-06

Change-Id: Ic251900cee49645c4a79012f2c03cb33cdd901c1
Signed-off-by: Thomas Wolf <thomas.wolf@paranor.ch>

Add the new org.eclipse.jgit.gpg.bc feature needed to sign commits

Bug: 559106
JGit-Dependency: I42fd6c00002e17aa9a7be96ae434b538ea86ccf8
Change-Id: I6b1444d070596bfcc089ec247b52593482b2b8db
Signed-off-by: Matthias Sohn <matthias.sohn@sap.com>

Add the new org.eclipse.jgit.ssh.jsch feature to use jsch

Add the fragment to the runtime classpath by adding an extra classpath
entry in the build.properties.

Bug: 553625
JGit-Dependency: I5979c8a9dbbe878a2e8ac0fbfde7230059d74dc2
Change-Id: Ifda66091d5dc21cdf1dec0d8ff698bf430bdc65d
Signed-off-by: Matthias Sohn <matthias.sohn@sap.com>

Don't install the WindowCache JMX bean

The JMX registration accesses the git user config during plug-in
activation and can be rather costly. Since this JMX bean is not
really needed in Eclipse configure the WindowCache to not use it.

Also catch ExceptionInInitializerError, which got thrown in the
reported case where this JMX registration went wrong.[1]

[1] https://www.eclipse.org/forums/index.php/t/1103963/

Bug: 563740
JGit-Dependency: I07ef7ae2f0208d177d2a03862846a8efe0191956
Change-Id: Ia5efb33ca40d1c32c6a587cad4f36ec70f4928c3
Signed-off-by: Thomas Wolf <thomas.wolf@paranor.ch>

Update jetty to 9.4.28.v20200408

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

Add 4.16 staging target platform

* replace 4.15 staging by 4.15 release (and fix the wrong name inside
the TPD file)
* add 4.16 staging
* change the naming schema of the staging.tpd file to make it more
obvious when it becomes outdated

Code still compiles fine with both egit-4.15 and egit-4.16-staging
platforms.

Change-Id: I07e05110f80ddd505063bbf2d022182de581c550
Signed-off-by: Michael Keppler <Michael.Keppler@gmx.de>

Rename interactive rebase job

Interactive rebase uses job names corresponding to the jgit commands
implementing interactive rebase. The command processing all the steps
(and thereby doing the main part of the user visible operation) should
be named "Interactive rebase" instead to make it easy to recognize the
origin of that job in the progress view.

Bug: 562950
Change-Id: I881788a161766249082c0510d76dd30d80f3578a
Signed-off-by: Michael Keppler <Michael.Keppler@gmx.de>

Block commit on error

Commit cc5d94f2 removed the initial condition "commitEnabled" when
inlining the method for calculating the button enablements. That broke
the "block commit" functionality.

Re-add the flag, but ignore it in case of "push head only" mode. That
way staging a file with error leads to both buttons blocked, unstaging
that file leads to an enabled "Push HEAD" button.

Bug: 563512
Change-Id: I4899e27ad158ac746c41b778861c71c3e42473a4
Signed-off-by: Michael Keppler <Michael.Keppler@gmx.de>

Disable "sign" button if no GpgSigner is present

JGit is moving the GPG signing support into a separate, optional
fragment. Although EGit will be set up such that this fragment is
included when EGit is installed, it's still possible that there is
no GpgSigner present at run-time.

Disable the button in that case, and give it a tooltip telling the
user that there is no signing support. Since this indicates a mis-
configuration, I've opted for disabling the button instead of hiding
it altogether.

Bug: 559106
Change-Id: Ifc477ef242b68d9f2b5ce367b493ed047b2eec00
Signed-off-by: Thomas Wolf <thomas.wolf@paranor.ch>

[releng] Update Orbit link in p2 repository to S20200519202422

Change-Id: I65be21b70b667fe2cbb9ca30b1be092094c5cea4
Signed-off-by: Thomas Wolf <thomas.wolf@paranor.ch>

Update Orbit to S20200519202422 and ant to 1.10.8

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

StagingView: speed up updating staged/unstaged viewers

Updating the selection used contains() on the values of a map. That
may be inefficient if many elements are removed. The values are either
StagingEntry or StagingFolderEntry objects, and we know that no two
of them can compare equal(), so use a LinkedHashSet instead.

Moreover, if all paths were removed, avoid any viewer state restoration
at all.

Bug: 563167
Change-Id: I5c07481972625f625dd4ae34ee85330e0118648e
Signed-off-by: Thomas Wolf <thomas.wolf@paranor.ch>

Staging View: Allow "Push HEAD" when there is nothing to commit

The "Commit and Push" button becomes "Push HEAD" in case there is
no change to be committed. This makes the most common push operation
directly accessible in the staging view and saves a few clicks in
some cases as the user doesn't have to switch to some other view.

If the repository has no HEAD (unborn branch) do not offer "Push
HEAD" but only "Commit and Push".

Other minor UI changes: don't require the buttons to be the same size
to avoid "jumping" buttons when the text on the "Commit and Push" or
"Push HEAD" button changes, and use the PushWizardDialog also for the
"Push To Gerrit..." wizard (so that the the "Finish" button is labeled
"Push"). Remove setHelpAvailable(true) calls on the dialog since there
is no help available. No point showing the help icon if it doesn't do
anything.

Bug: 512457
Change-Id: Ic67f17480458054bd8cdcf6b93061374da3b6c2f
Signed-off-by: Mickael Istria <mistria@redhat.com>
Signed-off-by: Thomas Wolf <thomas.wolf@paranor.ch>

CommitFileDiffViewer: fire SelectionChangedEvent after selectAll

Table.selectAll() does not send a SelectionChangedEvent. Update
the viewer selection explicitly to provoke one.

Bug: 563124
Change-Id: I869b99de5a10e89b346706192f54679830513be6
Signed-off-by: Thomas Wolf <thomas.wolf@paranor.ch>

Staging View: Always show dialog for Commit And Push...

So it offers opportunity for users to verify and cancel the push
operation.

Bug: 501419
Change-Id: Ia76f1e7bce7193a5cce161e52f5bb28b28e72ca7
Signed-off-by: Mickael Istria <mistria@redhat.com>

Dynamic label for "Open in Commit Viewer"

When multiple commits are selected, the command in the git history
view opens them all in individual commit editors. Make that clearer
by changing the menu entry to "Open All in Commit Viewers" in that
case.

Change-Id: I2bd3565507afcd6ae545e2dc118f1df3d731f478
Signed-off-by: Thomas Wolf <thomas.wolf@paranor.ch>

Interactive rebase: show view if successfully prepared

Open the view when an interactive rebase is started and was prepared
successfully.

Bug: 562618
Change-Id: I48cb91fcbce1b54e0ad51e9f2aababed88d637a1
Signed-off-by: Thomas Wolf <thomas.wolf@paranor.ch>

Staging View: "Commit and Push" uses Gerrit dialog only w/ Gerrit remote

The Gerrit push dialog requires some compatible Gerrit remote
declarations to be usable, and is unusable without it. So we
check the remote configs for compatible Gerrit ones before opening the
Gerrit push dialog, and stick to usual push branch dialog otherwise.

Bug: 562982
Change-Id: I73a17d4e3b66579d1fea7b319a93fcf8b24c4df3
Signed-off-by: Mickael Istria <mistria@redhat.com>

Minor clean-up in NodesByCommitTimeFilter

Use a PriorityQueue with a Comparator instead of implementing
Comparable. Resolves a Spotbugs warning about equals() not being
implemented. Also make more fields final.

Change-Id: Id28d3cf670bc91e8dee3c903a4e4689acf0d07ac
Signed-off-by: Thomas Wolf <thomas.wolf@paranor.ch>

Prepare 5.8.0-SNAPSHOT builds

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

EGit v5.8.0.202005061305-m2

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

Update target platforms to Orbit I20200506000552

Picks up Bouncy Castle 1.65, which will be required by JGit soon.

Bug: 553206
Change-Id: Idfe9297ec47c4ec2b93d092d1ea1ca3e14829171
Signed-off-by: Thomas Wolf <thomas.wolf@paranor.ch>
Signed-off-by: Matthias Sohn <matthias.sohn@sap.com>

Upgrade Tycho to 1.7.0

With this version the switch to JUnit 5 will be much easier, since Tycho
now supports the recent JUnit 5.6.0, and therefore the JUnit version is
finally the same in Maven/Tycho and in Eclipse.

At my workplace, this has been the first version of Tycho, where JUnit 5
tests work flawlessly both in Eclipse and in Maven.

Change-Id: Ie96786fd8da018578d6fd695d2cd11e55fde5a3e
Signed-off-by: Michael Keppler <Michael.Keppler@gmx.de>

Clear cached refs on repositories view refresh

When a full refresh of the repositories view occurs, the cached refs
must be cleared. Otherwise EGit may not pick up external changes.

Bug: 562661
Change-Id: I323018f084f56dc92d3a00afe86a1252297fbb47
Signed-off-by: Thomas Wolf <thomas.wolf@paranor.ch>

Update target platform generator to 3.0.0-SNAPSHOT from cbi project

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

Always use https to access download.eclipse.org

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

Use sentence style for "No staged files" error

Change-Id: I897de32eb21f8a1a1d7abea165da1d73104ed5fe
Signed-off-by: Michael Keppler <Michael.Keppler@gmx.de>

GitCloneDropAdapter: correct CloneCommand invocation

Invoke the command through the IHandlerService instead of executing
it directly with a fake and incomplete ExecutionEvent.

This ensures that the command gets a proper event with all details
filled in and prevents an NPE that otherwise would occur during
command execution when the command tries to access the variable
for the current selection.

Bug: 562386
Change-Id: If080b1be111c13fc79e53aea31c07fe5c727b506
Signed-off-by: Thomas Wolf <thomas.wolf@paranor.ch>

Clone command should allow a clone URI as parameter

Add a "repositoryUri" command parameter to the RepositoriesViewClone
command.

Bug: 562323
Change-Id: If306e7dcd5dd0bb76096dfab88fff8adfe2754c7
Signed-off-by: Andrew Obuchowicz <aobuchow@redhat.com>

Merge "Allow to select bundle files as source URIs"

Allow to select bundle files as source URIs

Bug: 560903
Change-Id: I3a56d26535eb6385eb21836835c1073ca9b33e22
Signed-off-by: Konrad Windszus <konrad_w@gmx.de>

[repo view] tag filtering by commit time

Using the tag filter text pattern #<n>, the last n tags (based on the
commit they tag) are shown.

Bug: 545240
Change-Id: Ie3bc430dfdb8b100e369e8f55fe26406277950db
Signed-off-by: Alexander Nittka <alex@nittka.de>

[staging view] "Compare with HEAD" menu item for staged files

Add an explicit "Compare with HEAD" context menu item. The operation
was already available on double-click, but a menu item is easier to
discover.

Bug: 562367
Change-Id: I1f03b8a72f37f60beb0d293cd5cc5c270539d8b5
Signed-off-by: Thomas Wolf <thomas.wolf@paranor.ch>

Move website project to releng working set

If the egit website project is contained in the workspace, it should be
shown in the releng working set.

Change-Id: I7ddfb2fb28e323089e3d01b8f47c0c4bb68d29ae
Signed-off-by: Michael Keppler <Michael.Keppler@gmx.de>

Use lambda instead of anonymous Runnables in RemoveCommand

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

Unify icons and labels of "compare with" menus

* same capitalization for all labels containing "each other"
* same icons for "compare with" and "compare with in tree"

Bug:562094
Change-Id: I69d59fccd2f8e4bc858b878c6c85fef1757d6c13
Signed-off-by: Michael Keppler <Michael.Keppler@gmx.de>

Close repository in CloneOperationTest

That avoids the locked pack file exceptions when running the test on
Windows.

Bug:562158
Change-Id: Ie36dc2b28c422175b36767234e3f112ecef4dabb
Signed-off-by: Michael Keppler <Michael.Keppler@gmx.de>

Fix typo in README

Change-Id: I583c203eee1d2c98be4d0df6b8e0e1ae248d3204
Signed-off-by: Michael Keppler <Michael.Keppler@gmx.de>

Log command exceptions

Bug: 561971
Change-Id: I5dce37c448805f34ced4a374488be81a5ef60f11
Signed-off-by: Thomas Wolf <thomas.wolf@paranor.ch>

Prevent ClassCastException during cloning

Cloning commands must not assume the RepositoriesView to be active.
Otherwise a ClassCastException may occur when trying to expand a group
node.

Bug: 562109
Change-Id: I18fb36c3b8fb28823b4465e487589e7f58406e7a
Signed-off-by: Alexander Nittka <alex@nittka.de>

RepositoriesView: show conflict decoration for conflicting files

Decorates elements of type FILE, FOLDER and WORKINGDIR with the
conflicts icon if the underlying resources are in a conflicting state.

Bug: 422365
Change-Id: I1edd8909be1e0ebd6d5b2839ddfb0617c56dca05
Signed-off-by: Simon Muschel <smuschel@gmx.de>

Remove logging that server is a Gerrit server

If I clone a repo from Gerrit, my Error log view gets focus and shows me
the message that the server was detected to be a Gerrit server. I think
this is more disturbing (as it lands in the Error Log view) than helpful
and suggest to remove this logging.

Change-Id: I44ab51c1cee50ee8385dfd8c16c8dfc6ed9f9b6e
Signed-off-by: Lars Vogel <Lars.Vogel@vogella.com>

Use REVERT_HEAD for comparing a revert conflict

Bug: 562073
Change-Id: I16b2bbee220b2d7208c2c23ba135775a9aec463c
Signed-off-by: Thomas Wolf <thomas.wolf@paranor.ch>

allow specifying the tag option when cloning

This change allows the user to specify the tag option for the remote
configuration when initially cloning the repository.

Bug: 561021
Change-Id: Idd9939d1e90d5eb1b14d3af370d495d4005feb73
Signed-off-by: Alexander Nittka <alex@nittka.de>

[repo view] Paste Repository URI menu items

It is confusing that the "paste" menu contribution is shown everywhere
but the corresponding clone repository action (for which it is a short
cut) is not.
After this change the paste command is still always active, but only
shown in the context menu for an empty or single repository group
selection. It is also added to the view menu.

Bug: 561016
Change-Id: I45448536cbd094c82a17d3dc4c7f8f285cc32eca
Signed-off-by: Alexander Nittka <alex@nittka.de>

[repo view] tag filtering: ensure status bar update

Always re-set the selection after filtering to make sure the label
and the status bar update correctly. Also guard against the tree
having been disposed.

Change-Id: I4301844979722bffd874e5ba1a079aa76747bad3
Signed-off-by: Thomas Wolf <thomas.wolf@paranor.ch>

[repo view] Prevent NPE in tag filtering

A ConcurrentHashMap cannot store null values.

Change-Id: I6b32c76b6495616724e93ce6001b2da23e3486a6
Signed-off-by: Thomas Wolf <thomas.wolf@paranor.ch>

Welcome page should use SVGs instead of pixel graphics

The welcome page content should no longer use pixel graphics as they
look blurry on high resolution displays. As the page displays HTML
anyway: Just let's use SVGs for that.

Bug 561573
Change-Id: I2d1c66af540e7df819ce60a2ad10fd509033c09c
Signed-off-by: Matthias Becker <ma.becker@sap.com>

Resolve a Spotbugs warning

Use x % 2 != 0 instead of x % 2 == 1.

Change-Id: Idcb0c9b132dbc6ac7066b7ea89fe08f79121ee90
Signed-off-by: Thomas Wolf <thomas.wolf@paranor.ch>

[staging] Also enable commit buttons after merge

The buttons must also be enabled if a conflicting merge has been
resolved such that there are no staged changes.

Bug: 525685
Change-Id: I67de884aa1f007401c0734dd90c5c818ce6c9054
Signed-off-by: Thomas Wolf <thomas.wolf@paranor.ch>

[staging view ] commit button enablement

The commit button should be enabled only if there is something to
commit.

Bug: 525685
Change-Id: Ib5915668a0e96786c01e431f685fb4bce13aa613
Signed-off-by: Alexander Nittka <alex@nittka.de>

[unified diff] quick outline

Add a quick outline to the diff editor. This allows for quickly
filtering the outline tree to find a file matching the search text.

Bug: 561015
Change-Id: I9768d67e8a812423d0992cbf352877bb5622c9a1
Signed-off-by: Alexander Nittka <alex@nittka.de>

[repo view] Tag filtering

Provide a new command that allows a user to filter tags by name
in the repositories view. The command is bound to the standard
"Find" command (Ctrl/Cmd-F) and is for now enabled only on the
TagsNode. The command expands the node and pops up a search text
field providing near-realtime filtering of the children of the
"Tags" node.

Filter patterns accept the '*' and '?' wildcards, and can be
anchored at the beginning (first character '^') and at the end
(last character '$').

Filter settings are persisted in the EGit UI preferences and are
restored from there on startup.

Bug: 545240
Change-Id: I9393248990cdca951f2ff6ba71969f20ecd23f74
Signed-off-by: Thomas Wolf <thomas.wolf@paranor.ch>

Merge branch 'stable-5.7' into master

* stable-5.7:
  [repo view] Avoid potential NPE in label provider
  Prepare 5.7.1-SNAPSHOT builds
  EGit v5.7.0.202003110725-r
  Update documentation for 5.7.0

Change-Id: Ib5806a2ed2ed2c81a81395317cdab6abe1baa2cf
Signed-off-by: Thomas Wolf <thomas.wolf@paranor.ch>

[cleanup] Remove no longer needed CommonUtils.getService()

Use IServiceLocator.getService() directly; it's generified since
Eclipse 4.5.

Change-Id: I6c86d1b47813c07927ae3bb85e6ca17d28c30125
Signed-off-by: Thomas Wolf <thomas.wolf@paranor.ch>

[history] Better handling of Find Toolbar action's accelerator

Resolve a TODO in GitHistoryPage: don't hard-code Ctrl/Cmd-F as
accelerator for the "show/hide find toolbar" action. Instead set
the action's command ID to that of the global find action. The
framework then does the rest, and the action also respects user-
defined key bindings.

Change-Id: Ifc5e40cf9618827e5c9d4cb0c8a7f3d76f4f71ea
Signed-off-by: Thomas Wolf <thomas.wolf@paranor.ch>

Refactor simple toggle commands

Provide a single non-abstract command implementation. Move the ids for
the "branch hierarchy" and "show commits on refs" up to that class.
These commands had originated in the repositories view, but as that
view's content and label provider became more general, they were re-
used elsewhere, too, and so today these toggles have an effect beyond
the repositories view. Moving these to a general common ToggleCommand
class thus makes sense.

Move the ID for "link repositories view with selection" to the
repositories view.

Keep the command IDs unchanged. Adapt tests.

Change-Id: Ica8b0c388722904cdf435691e2dd7f7508f9a315
Signed-off-by: Thomas Wolf <thomas.wolf@paranor.ch>

close submodules before deleting worktree

On Windows, deleting a repository containing submodules fails due to
file handles staying open. Submodules must be closed before the workTree
files are deleted.
My assumption is that otherwise the meta information about the submodule
is deleted and subRepo in RemoveCommand#closeSubmoduleRepositories is
null, so the submodule is not closed.

Bug: 408846
Change-Id: I76e030990021ef9c381760701f2ea9cd33de1768
Signed-off-by: Alexander Nittka <alex@nittka.de>

[history view test] Fix bug in GenerateHistoryJob

The test failed because the job sometimes would not update the UI.

Change-Id: I38d357c6029b551d1e0516bbe9ca2c37e3839fea

[repo view] Avoid potential NPE in label provider

Don't set a style for the paths of repositories or working trees if no
styles were set originally.

Bug: 561377
Change-Id: I0f02d8cbcdec17be844896c3ae2234b455a71f20
Signed-off-by: Thomas Wolf <thomas.wolf@paranor.ch>

[unified diff] compact outline

Allow the user to toggle between flat and compact tree presentation of
the outline tree.

Bug: 561015
Change-Id: Ia006608d95a4fd0219af96a2ea8c3a1e7f8f7a41
Signed-off-by: Alexander Nittka <alex@nittka.de>

history view test stability 2

Waiting for jobs in assertNoCommit did not reliably solve the sporadic
HistoryViewTest failures. Other tests suggested that an unselect before
changing the tree content might be better.

Change-Id: I7fa74a57e1553e33160bd0e1e697310e20aee1f2
Signed-off-by: Alexander Nittka <alex@nittka.de>

Update to org.apache.sshd 2.4.0

Change target platforms to use Orbit I20200319180910 and regenerate
them.

Bug: 561078
JGit-Dependency: I582f6b98055b013c006f2c749890fe6db801cbaa
Change-Id: Ieadfb7647027500f9aff9bad010ee4498a873f46
Signed-off-by: Thomas Wolf <thomas.wolf@paranor.ch>

[unified diff] Correct hyperlinks and outline context menu

Use "previous version" only if the diff is indeed against the first
parent. Otherwise use "base version".

Also show the commit id of the version, and add the short commit
message as a tooltip on the context menu entries. (Shows only on
newer Eclipse platforms; c.f. bug 473959.)

Try to show the link for opening the workspace version also in the
hyperlinks for the old version, if possible. Because for a "copy"
diff it would be very strange to have a link to show the new file
on an "old version" link, try to open the old file name if it still
exists. To get still a somewhat consistent behavior, don't show the
link on the old version if the file does not exist. Showing a link
to open the new version there would give very confusing behavior.
Also add the file name to the link text.

As a result, the "Open Working Tree Version" link for the "old
version" is

* ADD, DELETE, and RENAME diff: absent
* MODIFY diff: present, same file as "new version"
* COPY diff: present, old file name

Bug: 561012
Change-Id: I67177e4f71a4f295bd26a297ebfbecc7abdaac43
Signed-off-by: Thomas Wolf <thomas.wolf@paranor.ch>

[unified diff] add collapse outline action

For better usage, a collapse all action is added to the unified diff
outline.

Bug: 561015
Change-Id: I53fa81ffe102c7f046907e9da33d24e6bbc6c6bc
Signed-off-by: Alexander Nittka <alex@nittka.de>

history view test stability

Wait for jobs in assertNoCommit in analogy to assertCommitsAfterBase.
The test failed in multiple builds with changes unrelated to this test -
always in assertNoCommit.

Change-Id: Ia35779075d80988dc1c978d0b263de9da5ad7e13
Signed-off-by: Alexander Nittka <alex@nittka.de>

GitAdapterFactory: remove adaptation to IHistoryPageSource

This was moved to API GitHistoryAdapterFactory in commit f826c57f.
See also the EGit 5.7 N&N, where this removal was announced.[1]

[1] https://wiki.eclipse.org/EGit/New_and_Noteworthy/5.7#API

Change-Id: I5faf1b0281914697cf3645fe5ec8611613f40be2
Signed-off-by: Thomas Wolf <thomas.wolf@paranor.ch>

[oomph] Use EGit 5.7.0.202003110725-r as API baseline

Change-Id: I61fe1ad5c7b810bf9c6e80c45eadc9d580b41630
Signed-off-by: Thomas Wolf <thomas.wolf@paranor.ch>

Prepare 5.7.1-SNAPSHOT builds

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

EGit v5.7.0.202003110725-r

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

Update documentation for 5.7.0

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

[oomph] Update and cycle API baselines

Add 5.7.0, remove 5.4.0.

Change-Id: Ifea8cef525d7f7dfbe289e1e57b7e8a6a8990a1c
Signed-off-by: Thomas Wolf <thomas.wolf@paranor.ch>

Merge branch 'stable-5.7'

* stable-5.7:
  Prepare 5.7.1-SNAPSHOT builds
  EGit v5.7.0.202003090808-r
  Prepare 5.7.0-SNAPSHOT builds
  EGit v5.7.0.202002241735-m3

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

Prepare 5.7.1-SNAPSHOT builds

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

Prepare 5.8.0-SNAPSHOT builds

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

EGit v5.7.0.202003090808-r

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

Merge branch 'master' into stable-5.7

* master:
  Use 2 GB heap in Egit Oomph setup
  Avoid manifest marked modified during broken build
  Using for-each loop in egit.ui.internal
  Call FS.FileStoreAttributes.setBackground
  Add 4.14 and 4.15-staging target platforms
  Update Orbit to R20200224183213 for final 2020-03
  Provide a public adapter factory for IHistoryPageSource
  Update Orbit to S20200224183213 for 2020-03 RC1
  [releng] Update year in copyright notices for features & branding
  [repo view] Expand repository group when a repository is assigned
  Dynamic label for Push to/Fetch from Upstream
  [repo view] Add compare commands
  Create branch from repository group

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

Merge branch 'stable-5.6'

* stable-5.6:
  Call FS.FileStoreAttributes.setBackground

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

Merge "Avoid manifest marked modified during broken build"

Use 2 GB heap in Egit Oomph setup

1 GB is not sufficient anymore and can easily lead to OOM, especially
when reloading target platforms a lot.

Bug:559533
Change-Id: I9f824e30d55f44bf0ae2ddcfd2106b2c7560c543
Signed-off-by: Michael Keppler <Michael.Keppler@gmx.de>
Signed-off-by: Matthias Sohn <matthias.sohn@sap.com>

Avoid manifest marked modified during broken build

When the target platform is broken, then the build will remove the OSGI
service declarations from the manifest. Later with a correct target
platform they are re-added, but at the end of the manifest.

Check in this modified version to avoid the manifest being marked as
changed during future occurrences of such delete-add-cycles.

Change-Id: Ibfcd229c8695c6489dd2a42fd126fc3955f15e23
Signed-off-by: Michael Keppler <Michael.Keppler@gmx.de>

Using for-each loop in egit.ui.internal

Running the JDT cleanup action for using a for-each loop on egit.ui

Change-Id: I0d9ad37bec859ac392be22a824000b462a7051a0
Signed-off-by: Lars Vogel <Lars.Vogel@vogella.com>

Call FS.FileStoreAttributes.setBackground

When setting the asycnhronous background flag, use the
FS.FileStoreAttributes static method directly to avoid loading
unnecessary FS related classes.

Bug: 560528
Change-Id: I4200b7b06de3acaadf58cba542ec13174f6aa190
Signed-off-by: Alex Blewitt <alex.blewitt@gmail.com>

Add 4.14 and 4.15-staging target platforms

- former 4.14-staging and new 4.14 target have identical content
- create 4.15-staging from 4.14 staging without changing contained
plugins/features

I've tested the 4.15 target. There are more deprecations, but otherwise
it seems to work okay.

Change-Id: I86a30cc3b41c49a088f9ac28559e67924e6ce866
Signed-off-by: Michael Keppler <Michael.Keppler@gmx.de>
Signed-off-by: Matthias Sohn <matthias.sohn@sap.com>

Update Orbit to R20200224183213 for final 2020-03

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

Provide a public adapter factory for IHistoryPageSource

This facilitates third-party bundles showing a git history determined
by objects unknown to EGit. The third-party object must be adaptable
in some way to Repository and optionally to RevCommit so that EGit can
figure out which history to show.

Use the new factory internally, but leave GitAdapterFactory also able
to adapt objects to IHistoryPageSource; there might be existing third-
party bundles that use it even though it's internal, and there's no
need to break such third parties by being overly strict.

Bug: 558980
Change-Id: I2743d7c3f199cd0dd61c6797eb6125f8656ba9ad
Signed-off-by: Thomas Wolf <thomas.wolf@paranor.ch>

Update Orbit to S20200224183213 for 2020-03 RC1

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

[releng] Update year in copyright notices for features & branding

All were very outdated. Bump upper end of range to 2020.

Change-Id: I89c619c13d5f80ba5dadcccc6cf27cd84ff5dabc
Signed-off-by: Thomas Wolf <thomas.wolf@paranor.ch>