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

Upgrade to Gradle 4.0 #96

Merged
merged 4 commits into from
Jun 24, 2017
Merged

Upgrade to Gradle 4.0 #96

merged 4 commits into from
Jun 24, 2017

Conversation

melix
Copy link
Owner

@melix melix commented Jun 23, 2017

This pull request updates the plugin for compatibility with Gradle 4.0.

  • minimal requirement for Gradle is 4.0
  • uses the new worker API, which greatly simplifies the setup of the project
  • requires the Shadow plugin 2.0.1

melix added 4 commits June 1, 2017 22:15
Note that this version would only be compatible with Gradle 4.0+ and Shadow plugin 2.0+. It makes use of the new
worker API, which avoids us doing all the nasty jar repackaging of the runtime of JMH: a worker process is automatically
spawned by Gradle with the appropriate classpath. This means that we can compile this plugin with a specific version
of JMH, but the user can override the version, and we would still use the `Main` entry point instead of an exec task
to execute the benchmarks.

This commit also fixes the [warning issued by Gradle 4.0](#94), which is the main reason for broken compatibility with
released under 4.0: the newer API doesn't exist on 3.5.

This commit however breaks a test with the shadow plugin, and is waiting for the resolution of [this bug](GradleUp/shadow#297).
Signed-off-by: Cedric Champeau <[email protected]>
@melix melix requested a review from aalmiray June 23, 2017 18:10
@coveralls
Copy link

Coverage Status

Coverage decreased (-7.5%) to 33.394% when pulling 81b84d9 on cc/upgrade-to-gradle-4 into e417850 on master.

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.

2 participants