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.6 #2348

Merged
merged 1 commit into from
Mar 20, 2018
Merged

Conversation

chrisdennis
Copy link
Member

No description provided.

@@ -1,10 +1,16 @@
plugins {
id 'org.akhikhl.gretty' version '2.0.0' apply false
buildscript {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For me, this is supposed to be legacy style. No?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes... but something odd is happening if I try and do this any other way. Not sure if I'm screwing up, or if something is wrong with the way the forked version of the plugin is being published.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When going back to the 2.0.0 version it "works" (jacoco plugin is failing). But I can't start the app with the new version when doing ../../gradlew appStart

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's not Jacoco thats broken, it's Jacoco trying to interact with the gretty task. Gretty uses internal gradle APIs that changed in 4.6 and had to updated things. The broken appStart is an oversight, need to make sure the submodules can resolve the runner. Fixing.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems I tripping on akhikhl/gretty#425

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Replacing the whole buildscript with

plugins {
  id 'org.gretty' version '2.1.0' apply false
}

seems to work fine.

Including launching the demos with appStart.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Right it does today: https://plugins.gradle.org/plugin/org.gretty

Version 2.1.0 (latest)
Created 20 March 2018.

I'll update accordingly.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ah did not pay attention to the dates (PR or plugin publication) :-)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

héhé... yes... because I tried that earlier. It wasn't working because of a missing jetty 9...

@chrisdennis chrisdennis force-pushed the gradle-4.6 branch 2 times, most recently from adcbde8 to 2667faa Compare March 20, 2018 17:14
}

subprojects {
apply plugin: 'war'
apply plugin: 'org.akhikhl.gretty'
apply plugin: 'org.gretty'
Copy link
Contributor

@henri-tremblay henri-tremblay Mar 20, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So far, without adding jcenter, I can't find the gretty jetty9 jar. So it's not working for me.

This is working:

subprojects {
  apply plugin: 'war'
  apply plugin: 'org.gretty'

  repositories {
    jcenter()
  }

  gretty {
    httpPort = 8080
    contextPath = '/'
    servletContainer = 'jetty9'
  }

  dependencies {
    compile 'javax.servlet:javax.servlet-api:3.1.0', project(':impl')
    runtime 'ch.qos.logback:logback-classic:1.2.3', 'com.h2database:h2:1.4.196'
  }
}

@henri-tremblay henri-tremblay merged commit 8b5b0b5 into ehcache:master Mar 20, 2018
@chrisdennis chrisdennis deleted the gradle-4.6 branch March 29, 2018 14:17
@henri-tremblay henri-tremblay added this to the 3.6.0 milestone Jul 13, 2018
@henri-tremblay henri-tremblay modified the milestones: 3.6.0, 3.5.3 Sep 25, 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.

3 participants