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

Android: Apply fix for "Cannot use file /tmp/hsperfdata_" in javac #4881

Merged
merged 1 commit into from
Feb 12, 2025

Conversation

oxve
Copy link
Contributor

@oxve oxve commented Feb 10, 2025

The fix was previously applied via making "java" a wrapper script.
However, "javac" does not invoke "java"

(cherry picked from commit a40b631c5a9c0e195d8c919efa6f2e9d9ece45ed)

b/374150391

Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5444666

@oxve oxve requested review from dahlstrom-g and isarkis February 10, 2025 21:45
@oxve oxve force-pushed the disable-shared-mem branch from 4168f93 to 50378f6 Compare February 11, 2025 22:27
@oxve oxve marked this pull request as ready for review February 11, 2025 22:27
@oxve oxve requested a review from a team as a code owner February 11, 2025 22:27
@oxve oxve changed the title Cherry-pick a40b631 from chromium Android: Apply fix for "Cannot use file /tmp/hsperfdata_" in javac Feb 11, 2025
@oxve oxve enabled auto-merge February 11, 2025 22:29
The fix was previously applied via making "java" a wrapper script.
However, "javac" does not invoke "java"

(cherry picked from commit a40b631c5a9c0e195d8c919efa6f2e9d9ece45ed)

Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5444666
@oxve oxve force-pushed the disable-shared-mem branch from 50378f6 to acd7c39 Compare February 11, 2025 22:30
Copy link
Contributor

@dahlstrom-g dahlstrom-g left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

commit message LGTM 😆

@oxve oxve merged commit c2e4b4e into youtube:main Feb 12, 2025
126 checks passed
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.

4 participants