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

Bumping google-http-client version #171

Merged
merged 2 commits into from
Aug 2, 2018

Conversation

garrettjonesgoogle
Copy link
Member

@garrettjonesgoogle garrettjonesgoogle commented Aug 2, 2018

No description provided.

@garrettjonesgoogle
Copy link
Member Author

cc @elharo and @saturnism :-) (this is only the first step in the dependency pachinko machine)

@elharo
Copy link
Contributor

elharo commented Aug 2, 2018

Tests failed. Probably not related but this should probably be fixed first:

[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-checkstyle-plugin:2.17:check (checkstyle) on project google-auth-library-parent: Execution checkstyle of goal org.apache.maven.plugins:maven-checkstyle-plugin:2.17:check failed: Plugin org.apache.maven.plugins:maven-checkstyle-plugin:2.17 or one of its dependencies could not be resolved: Could not find artifact com.sun:tools:jar:1.7.0 at specified path /Library/Java/JavaVirtualMachines/jdk-10.0.1.jdk/Contents/Home/../lib/tools.jar -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/PluginResolutionException

@garrettjonesgoogle
Copy link
Member Author

I reran the mac build (which is the one that failed) and it failed again...

@garrettjonesgoogle
Copy link
Member Author

Well, bumping checkstyle fixed the mac build. Maybe there was some weird dependency interaction caused by the bump of http-client - but it's fixed now!

@elharo
Copy link
Contributor

elharo commented Aug 2, 2018

FYI, I don't see the guava-jdk5 to guava switch? Is that inside the new version of http-client?

@garrettjonesgoogle
Copy link
Member Author

Yes the switch is in http-client: googleapis/google-http-java-client@7b0c01c - which was released in 1.24.1.

@garrettjonesgoogle
Copy link
Member Author

Hmm after more research, I found that google-http-client's dependency on guava-jdk5 was provided, so it wasn't actually polluting the dependency tree; it turns out that it is google-api-client that has the compile dependency on guava-jdk5. Thus, the present PR doesn't change the guava-jdk5 problem. It's good to get the version bumped though!

@chingor13 chingor13 merged commit df46fd3 into googleapis:master Aug 2, 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.

None yet

3 participants