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

Update java_tools v11.7.1 #15258

Closed
wants to merge 2 commits into from
Closed

Update java_tools v11.7.1 #15258

wants to merge 2 commits into from

Conversation

hvadehra
Copy link
Member

@hvadehra hvadehra commented Apr 14, 2022

A new release was required because 0c65082 caused java_tools to be built targetting java11, inadvertently dropping support for jre8/9/10

@hvadehra hvadehra requested a review from cushon April 14, 2022 17:38
@hvadehra hvadehra marked this pull request as ready for review April 14, 2022 17:43
@bazel-io bazel-io closed this in 22c91da Apr 14, 2022
@fmeum
Copy link
Collaborator

fmeum commented Apr 14, 2022

@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 Apr 14, 2022
hvadehra added a commit to hvadehra/bazel that referenced this pull request Apr 15, 2022
A new release was required because bazelbuild@0c65082 caused java_tools to be built targetting java11, inadvertently dropping support for jre8/9/10

Closes bazelbuild#15258.

PiperOrigin-RevId: 441834316

(cherry picked from commit 22c91da)
meteorcloudy pushed a commit that referenced this pull request Apr 19, 2022
* Update java_tools v11.7.1

A new release was required because 0c65082 caused java_tools to be built targetting java11, inadvertently dropping support for jre8/9/10

Closes #15258.

PiperOrigin-RevId: 441834316

(cherry picked from commit 22c91da)

* Automatic code cleanup.

PiperOrigin-RevId: 414481809
(cherry picked from commit 1987af2)

* Automatic code cleanup.

PiperOrigin-RevId: 415349792

(cherry picked from commit ccc9b9d)

A clean cherry pick was not possible since the contents of various files in the change have diverged since this release branch was cut.
@ckolli5
Copy link

ckolli5 commented Apr 19, 2022

@bazel-io fork 5.2.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 Apr 19, 2022
@meteorcloudy
Copy link
Member

@ckolli5 I have merged #15231 which cherry picked this for 5.2.0

@hvadehra hvadehra deleted the hvadehra-javatools11.7.1 branch November 2, 2022 05:45
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.

6 participants