Skip to content

Conversation

@snazarkin
Copy link
Contributor

@snazarkin snazarkin commented Aug 4, 2021

This is follow up for 8191278 that fixes crash of zero builds. Applies cleanly.


Progress

  • Change must not contain extraneous whitespace
  • Commit message must refer to an issue

Integration blocker

 ⚠️ Dependency #173 must be integrated first

Issue

Reviewing

Using git

Checkout this PR locally:
$ git fetch https://git.openjdk.java.net/jdk11u-dev pull/192/head:pull/192
$ git checkout pull/192

Update a local copy of the PR:
$ git checkout pull/192
$ git pull https://git.openjdk.java.net/jdk11u-dev pull/192/head

Using Skara CLI tools

Checkout this PR locally:
$ git pr checkout 192

View PR using the GUI difftool:
$ git pr show -t 192

Using diff file

Download this PR as a diff file:
https://git.openjdk.java.net/jdk11u-dev/pull/192.diff

@bridgekeeper
Copy link

bridgekeeper bot commented Aug 4, 2021

👋 Welcome back snazarki! A progress list of the required criteria for merging this PR into pr/173 will be added to the body of your pull request. There are additional pull request commands available for use with this pull request.

@openjdk openjdk bot changed the title Backport 302b8d06ce595b6071d728f6ab10dde561dcd54b 8226878: zero crashes after JDK-8191278 Aug 4, 2021
@openjdk
Copy link

openjdk bot commented Aug 4, 2021

This backport pull request has now been updated with issue from the original commit.

@openjdk openjdk bot added backport Port of a pull request already in a different code base clean Identical backport; no merge resolution required rfr Pull request is ready for review labels Aug 4, 2021
@mlbridge
Copy link

mlbridge bot commented Aug 4, 2021

Webrevs

@RealCLanger
Copy link
Contributor

@snazarkin, since this item was rejected by the maintainers, would you mind closing this PR?

@snazarkin snazarkin closed this Sep 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport Port of a pull request already in a different code base clean Identical backport; no merge resolution required rfr Pull request is ready for review

Development

Successfully merging this pull request may close these issues.

2 participants