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
GH-25633: [CI][Java][macOS] Ensure using bundled RE2 #33711
Conversation
If we have Homebrew's RE2, we may mix re2.h from Homebrew's RE2 and bundled RE2. If we mix re2.h and libre2.a, we may generate wrong re2::RE2::Options. It may crashes our program.
@github-actions crossbow submit java-jars |
|
Revision: 547a127 Submitted crossbow builds: ursacomputing/crossbow @ actions-15715d7ab8
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1 awesome, thanks @kou I will merge once the CB job is green!
@github-actions crossbow submit java-jars |
Revision: 28e221e Submitted crossbow builds: ursacomputing/crossbow @ actions-c094071131
|
Benchmark runs are scheduled for baseline = fa59051 and contender = eae79e7. eae79e7 is a master commit associated with this PR. Results will be available as each benchmark for each run completes. |
### Rationale for this change If we have Homebrew's RE2, we may mix re2.h from Homebrew's RE2 and bundled RE2. If we mix re2.h and libre2.a, we may generate wrong re2::RE2::Options. It may crashes our program. ### What changes are included in this PR? Ensure removing Homebrew's RE2. ### Are these changes tested? Yes. ### Are there any user-facing changes? No. * Closes: #25633 Authored-by: Sutou Kouhei <kou@clear-code.com> Signed-off-by: Jacob Wujciak-Jens <jacob@wujciak.de>
Rationale for this change
If we have Homebrew's RE2, we may mix re2.h from Homebrew's RE2 and bundled RE2.
If we mix re2.h and libre2.a, we may generate wrong re2::RE2::Options. It may crashes our program.
What changes are included in this PR?
Ensure removing Homebrew's RE2.
Are these changes tested?
Yes.
Are there any user-facing changes?
No.