Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Breaking changes for Groovy 2.5, JGit 5, and JSch agent removal #246

Merged
merged 11 commits into from
Jul 4, 2018

Conversation

ajoberstar
Copy link
Owner

This contains a few somewhat drastic changes. They should make the library more maintainable, but it may cause some upgrade hiccups.

One issue to still research is whether we can prevent SSH from blocking when you don't have either a password-free SSH key or one loaded into your system agent.

Separate the core grgit lib and the Gradle plugin so that its easier to
test a Groovy 2.5 dependency in core and not in the plugin.
Update the core lib's dependencies to Groovy 2.5. Gradle still seems to
work fine with its 2.4 version.

This fixes #237.
Since JGit 5 changed behavior to match C Git, this allows people to
revert to the previous behavior.
To be more compatible with Java 9+.

This fixes #241.
This adds an extra dependency for a potentially dubious use case, so
seems worth dropping.

This fixes #244
The jsch-agent project hasn't been maintained in 4 years and its been
the source of many issues on the Grgit repo.

We'll now leverage system SSH instead.

This fixes #245.
This allows better preview in GitHub and other editors.
Old Grgit 2 options are still documented in a separate doc.
Tied to the changes of breaking this into two JARs.
@ajoberstar ajoberstar merged commit 5c4e1e6 into master Jul 4, 2018
@ajoberstar ajoberstar deleted the groovy branch July 4, 2018 02:13
@childnode childnode mentioned this pull request Jul 16, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant