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

use version 27.0.1-jre of guava instead of 27.0.1-android #3174

Closed
wants to merge 1 commit into from
Closed

use version 27.0.1-jre of guava instead of 27.0.1-android #3174

wants to merge 1 commit into from

Conversation

dbyron0
Copy link

@dbyron0 dbyron0 commented Apr 9, 2019

The dependency on guava 27.0.1-android instead of 27.0.1-jre is causing problems in a gradle project that uses swagger-core 1.5.22. With the android guava, I get errors like:

* What went wrong:
A problem occurred configuring project ':app'.
> Failed to notify project evaluation listener.
   > Could not create task ':app:extractDebugAnnotations'.
      > com.google.common.collect.ImmutableList.toImmutableList()Ljava/util/stream/Collector;
   > com.google.common.collect.ImmutableList.toImmutableList()Ljava/util/stream/Collector;

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

This error goes away with 27.0.1-jre. google/guava#2914 explains a bit about why.

I'm a little confused how we got to the android version in the first place. #3122 is the PR that did it, motivated by #3079, but there's another PR linked to that issue that made a change against master (#3121) that used 27.0.1-jre.

Thanks for your help.

@dbyron0
Copy link
Author

dbyron0 commented Apr 10, 2019

OK, I see that sticking with java 7 is the likely motivation here....Leaving open for awhile in case there are more comments.

@frantuma
Copy link
Member

It's as you mention in your comment, please check #3122 (comment) and #3122 (comment) for motivation and usage of different version if needed. Closing PR, please reopen if you're still experiencing issues

@frantuma frantuma closed this Apr 10, 2019
@dbyron0 dbyron0 deleted the guava_27_0_1_jre branch April 10, 2019 13:28
@dbyron0
Copy link
Author

dbyron0 commented Apr 10, 2019

Thanks for the links. They help close the loop.

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