Skip to content

Conversation

@srl295
Copy link
Member

@srl295srl295 commented Apr 26, 2019

  • Floating patch for ICU 64.x

(does not apply to 63.x)

ICU Bug: https://unicode-org.atlassian.net/browse/ICU-20575
Backport of: unicode-org/icu#634
Fixes: #27418

  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included: TODO!!
  • commit message follows commit guidelines

@srl295srl295 added the i18n-api Issues and PRs related to the i18n implementation. label Apr 26, 2019
@srl295srl295 self-assigned this Apr 26, 2019
@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-botnodejs-github-bot added the tools Issues and PRs related to the tools directory. label Apr 26, 2019
@srl295srl295 marked this pull request as ready for review April 26, 2019 22:30
@nodejs-github-bot
Copy link
Collaborator

@srl295
Copy link
MemberAuthor

to quote @refack in #27415 . do we want to fasttrack this one also? it's a similar crash and may be causing the other issue to be exposed.

Please 👍 if you concur.

@refackrefack added the fast-track PRs that do not need to wait for 48 hours to land. label Apr 27, 2019
@nodejs-github-bot
Copy link
Collaborator

@targos
Copy link
Member

Landed in 31ac8b9

@targostargos closed this Apr 27, 2019
targos pushed a commit that referenced this pull request Apr 27, 2019
- Floating patch for ICU 64.x - includes test case ICU Bug: https://unicode-org.atlassian.net/browse/ICU-20575 Backport of: unicode-org/icu#634Fixes: #27418 PR-URL: #27435 Reviewed-By: Refael Ackermann <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Michaël Zasso <[email protected]>
targos pushed a commit that referenced this pull request Apr 27, 2019
- Floating patch for ICU 64.x - includes test case ICU Bug: https://unicode-org.atlassian.net/browse/ICU-20575 Backport of: unicode-org/icu#634Fixes: #27418 PR-URL: #27435 Reviewed-By: Refael Ackermann <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Michaël Zasso <[email protected]>
This was referenced Apr 27, 2019
BethGriggs pushed a commit that referenced this pull request May 10, 2019
- Floating patch for ICU 64.x - includes test case ICU Bug: https://unicode-org.atlassian.net/browse/ICU-20575 Backport of: unicode-org/icu#634Fixes: #27418 PR-URL: #27435 Reviewed-By: Refael Ackermann <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Michaël Zasso <[email protected]>
MylesBorins pushed a commit that referenced this pull request May 16, 2019
- Floating patch for ICU 64.x - includes test case ICU Bug: https://unicode-org.atlassian.net/browse/ICU-20575 Backport of: unicode-org/icu#634Fixes: #27418 PR-URL: #27435 Reviewed-By: Refael Ackermann <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Michaël Zasso <[email protected]>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fast-trackPRs that do not need to wait for 48 hours to land.i18n-apiIssues and PRs related to the i18n implementation.toolsIssues and PRs related to the tools directory.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

backport ICU-20575 to fix err/crasher in ICU 64.x

5 participants

@srl295@nodejs-github-bot@targos@refack@richardlau