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.

Sign up for GitHub

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

Jump to bottom

ICU-21032 v65.x backport ICU-20958 Prevent SEGV_MAPERR in append #1154

Merged
merged 1 commit into from
May 26, 2020
Merged

ICU-21032 v65.x backport ICU-20958 Prevent SEGV_MAPERR in append #1154

merged 1 commit into from
May 26, 2020

Conversation

srl295
Copy link
Member

@srl295 srl295 commented May 22, 2020

ICU-21032 v65.x backport of ICU-20958

See #971

(cherry picked from commit b7d08bc)

@srl295 srl295 self-assigned this May 22, 2020
@srl295
Copy link
Member Author

srl295 commented May 22, 2020

/azp run CI-Exhaustive

@azure-pipelines
Copy link

Azure Pipelines failed to run 1 pipeline(s).

@srl295
Copy link
Member Author

srl295 commented May 22, 2020

Exhaustive says:

The pipeline is not valid. Could not get the latest source version for repository unicode-org/icu hosted on https://github.com/ using ref 78f4b0d. GitHub reported the error, "Internal Server Error"

@jefgen
Copy link
Member

jefgen commented May 22, 2020

Please fix the branch. :)

@jefgen
Copy link
Member

jefgen commented May 22, 2020

Exhaustive says:

The pipeline is not valid. Could not get the latest source version for repository unicode-org/icu hosted on https://github.com/ using ref 78f4b0d. GitHub reported the error, "Internal Server Error"

FWIW, I just now had issues even loading any GitHub page. I wonder if maybe there was some kind of hiccup...

@srl295 srl295 changed the base branch from master to maint/maint-65 May 22, 2020 21:08
@srl295
Copy link
Member Author

srl295 commented May 22, 2020

FWIW, I just now had issues even loading any GitHub page. I wonder if maybe there was some kind of hiccup...

or i broke it…

@srl295
Copy link
Member Author

srl295 commented May 22, 2020

/azp run CI-Exhaustive

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@srl295
Copy link
Member Author

srl295 commented May 22, 2020

azp may have failed because the commit was a no-op

@srl295 srl295 merged commit f08a84e into unicode-org:maint/maint-65 May 26, 2020
@srl295 srl295 deleted the backport-65-20958 branch May 26, 2020 16:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@FrankYFTang FrankYFTang FrankYFTang approved these changes

@markusicu markusicu Awaiting requested review from markusicu

@jefgen jefgen Awaiting requested review from jefgen

Assignees

@srl295 srl295

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@srl295 @jefgen @FrankYFTang