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

depend directly on Apache httpclient #591

Merged
merged 4 commits into from
Feb 15, 2019
Merged

depend directly on Apache httpclient #591

merged 4 commits into from
Feb 15, 2019

Conversation

elharo
Copy link
Contributor

@elharo elharo commented Feb 13, 2019

depend directly on Apache httpclient instead of pulling it in transitively from weird android dependency.

  • Tests pass
  • Appropriate docs were updated (if necessary)

Not sure why tests didn't run, but there's a local failure I'm investigating.

OK. tests passed locally. First failure was probably a DNS glitch caused by putting laptop to sleep in the middle.

@sduskis I'm still concerned that the kokoro jobs didn't fire here.

@elharo elharo requested a review from a team as a code owner February 13, 2019 20:13
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Feb 13, 2019
@elharo elharo added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Feb 13, 2019
@elharo elharo changed the title depend directly on Apache httpclient instead of old version bundled i… depend directly on Apache httpclient Feb 14, 2019
@elharo
Copy link
Contributor Author

elharo commented Feb 14, 2019

@sduskis Do you know how I can manually trigger the kokoro builds?

@elharo elharo added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Feb 14, 2019
@elharo
Copy link
Contributor Author

elharo commented Feb 14, 2019

OK, looks there's a kokoro:force-run label

@elharo elharo removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Feb 14, 2019
Copy link
Contributor Author

@elharo elharo left a comment

Choose a reason for hiding this comment

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

PTAL @sduskis

All Kokoro jobs passed.
I think this PR is helpful for multiple reasons:

  1. Declares dependencies directly (i.e. like strict_java_deps) instead of relying on transitive dependencies
  2. Will resolve downstream linkage errors in dependents found in the dashboard
  3. Avoids pulling in a very old and out of date android artifact that was built and released by a third party because the AOSP did not want to ship a binary of this.

@sduskis sduskis merged commit 4b1f4b8 into master Feb 15, 2019
@elharo elharo deleted the android branch February 15, 2019 18:50
clundin25 pushed a commit to clundin25/google-http-java-client that referenced this pull request Aug 11, 2022
…-sdk to v1.9.88 (googleapis#591)

[![WhiteSource Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| com.google.appengine:appengine-api-1.0-sdk | `1.9.86` -> `1.9.88` | [![age](https://badges.renovateapi.com/packages/maven/com.google.appengine:appengine-api-1.0-sdk/1.9.88/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/maven/com.google.appengine:appengine-api-1.0-sdk/1.9.88/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/maven/com.google.appengine:appengine-api-1.0-sdk/1.9.88/compatibility-slim/1.9.86)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/maven/com.google.appengine:appengine-api-1.0-sdk/1.9.88/confidence-slim/1.9.86)](https://docs.renovatebot.com/merge-confidence/) |

---

### Configuration

:date: **Schedule**: At any time (no schedule defined).

:vertical_traffic_light: **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

:recycle: **Rebasing**: Renovate will not automatically rebase this PR, because other commits have been found.

:no_bell: **Ignore**: Close this PR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box.

---

This PR has been generated by [WhiteSource Renovate](https://renovate.whitesourcesoftware.com). View repository job log [here](https://app.renovatebot.com/dashboard#github/googleapis/google-auth-library-java).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes This human has signed the Contributor License Agreement. kokoro:force-run Add this label to force Kokoro to re-run the tests.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants