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

Revert "Install Java 8, Android command line tools and SDK for Ubuntu" #189

Conversation

jpsim
Copy link
Contributor

@jpsim jpsim commented Dec 9, 2022

Reverts #188

This grew the Docker image by an unreasonable amount, going from 1.79 GB to 3.77 GB.

@jpsim jpsim requested a review from a team as a code owner December 9, 2022 14:48
@phlax
Copy link
Member

phlax commented Dec 9, 2022

lgtm, but needs DCO

@jpsim jpsim force-pushed the revert-188-install-android-command-line-tools-and-sdk-for-ubuntu branch from 1ae9b66 to 0ca6559 Compare December 9, 2022 14:59
@jpsim
Copy link
Contributor Author

jpsim commented Dec 9, 2022

lgtm, but needs DCO

I created the revert using GitHub's UI. Is there a way to configure it to apply signoff when that's used? Or is it necessary to sign off locally?

@phlax
Copy link
Member

phlax commented Dec 9, 2022

not sure if there is a way to get gh to do it - i dont use it for that reason - i think you have to sign it off locally

Copy link
Member

@phlax phlax left a comment

Choose a reason for hiding this comment

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

lgtm, thanks @jpsim

@jpsim
Copy link
Contributor Author

jpsim commented Dec 9, 2022

FYI I don't have merge access on this repo.

@keith keith merged commit 5c8e6ab into envoyproxy:main Dec 9, 2022
@jpsim jpsim deleted the revert-188-install-android-command-line-tools-and-sdk-for-ubuntu branch December 9, 2022 17:50
htuch pushed a commit that referenced this pull request Dec 9, 2022
  [skip ci]
  Revert "Install Java 8, Android command line tools and SDK for Ubuntu" (#189)

This reverts commit dce0a8c.

Signed-off-by: JP Simard <[email protected]>
htuch pushed a commit that referenced this pull request Dec 9, 2022
  [skip ci]
  Regenerate linux toolchains from 5c8e6ab

  [skip ci]
  Revert "Install Java 8, Android command line tools and SDK for Ubuntu" (#189)

This reverts commit dce0a8c.

Signed-off-by: JP Simard <[email protected]>
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