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

Make AutoBazelRepositoryProcessor compatible with Java 8 #17460

Closed
wants to merge 1 commit into from

Conversation

fmeum
Copy link
Collaborator

@fmeum fmeum commented Feb 10, 2023

The three-argument overload of Stream#iterate is only available as of Java 9.

Fixes #17458

The three-argument overload of `Stream#iterate` is only available as of
Java 9.
@fmeum
Copy link
Collaborator Author

fmeum commented Feb 10, 2023

@bazel-io flag

@bazel-io bazel-io added the potential release blocker Flagged by community members using "@bazel-io flag". Should be added to a release blocker milestone label Feb 10, 2023
@fmeum
Copy link
Collaborator Author

fmeum commented Feb 10, 2023

@cushon Could you review this? Do you see a way to verify Java 8 compatibility in CI with the current setup? At least based on its name darwin-arm64-openjdk-8-xcode should be have a JDK 8 installed, is that correct?

@sgowroji sgowroji added the team-Rules-Java Issues for Java rules label Feb 10, 2023
@ShreeM01
Copy link
Contributor

@bazel-io fork 6.1.0

@bazel-io bazel-io removed the potential release blocker Flagged by community members using "@bazel-io flag". Should be added to a release blocker milestone label Feb 10, 2023
@cushon
Copy link
Contributor

cushon commented Feb 10, 2023

@cushon Could you review this?

LGTM, thanks

Do you see a way to verify Java 8 compatibility in CI with the current setup? At least based on its name darwin-arm64-openjdk-8-xcode should be have a JDK 8 installed, is that correct?

I agree that it would be nice to have CI coverage for this as long as targeting Java 8 is supposed. @comius, WDYT about this?

@ShreeM01 ShreeM01 added the awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally label Feb 14, 2023
@sgowroji sgowroji removed the awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally label Feb 16, 2023
@fmeum fmeum deleted the 17458-runfiles-library-java-8 branch February 16, 2023 12:17
ShreeM01 added a commit that referenced this pull request Feb 16, 2023
The three-argument overload of `Stream#iterate` is only available as of Java 9.

Fixes #17458

Closes #17460.

PiperOrigin-RevId: 509885763
Change-Id: If83d832b3590b8598c0f4163afa3c7054e7c9786

Co-authored-by: Fabian Meumertzheim <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team-Rules-Java Issues for Java rules
Projects
None yet
Development

Successfully merging this pull request may close these issues.

@bazel_tools//tools/java/runfiles:auto_bazel_repository_processor does not compile with JDK 8
5 participants