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

Set -source 8 -target 8 in the open-source build, and remove Java8Usage. #5444

Closed
wants to merge 1 commit into from

Conversation

copybara-service[bot]
Copy link
Contributor

Set -source 8 -target 8 in the open-source build, and remove Java8Usage.

This is largely a rollback of the original Java8Usage change, but I've modified it in a few ways:

  • I set -source 8 -target 8 in the backport.
  • I kept the code to make includes/excludes fully work with maven-compiler-plugin, since that could save us some confusion down the line.
  • I incorporated a rollback of the changes to Preconditions from CL 361700569, in which I moved the warning from MoreObjects to Preconditions.
  • I added errors to our internal release scripts so that we don't accidentally make a release that drops Java 7 support before we're ready.

Fixes #5269

…sage.

This is largely a rollback of the original Java8Usage change, but I've modified it in a few ways:

- I set `-source 8 -target 8` in the backport.
- I kept the code to make includes/excludes fully work with maven-compiler-plugin, since that could save us some confusion down the line.
- I incorporated a rollback of the changes to Preconditions from CL 361700569, in which I moved the warning from MoreObjects to Preconditions.
- I added errors to our internal release scripts so that we don't accidentally make a release that drops Java 7 support before we're ready.

Fixes #5269

PiperOrigin-RevId: 364405695
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Build Android flavor with -source 8 -target 8 (but still avoid Java 7+ APIs)
1 participant