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

Partially revert "Android: Clean up some hardcoded strings" #9841

Merged
merged 1 commit into from Jul 6, 2021

Conversation

JosJuice
Copy link
Member

@JosJuice JosJuice commented Jun 24, 2021

This partially reverts commit cbc4989 of PR #9835 due to a crash: https://bugs.dolphin-emu.org/issues/12561

I can't debug what the cause of the crash is due to not having an Android TV device. Let's just revert this for now to fix the crash.

@JosJuice JosJuice added the WIP / do not merge Work in progress (do not merge) label Jun 24, 2021
@JosJuice JosJuice force-pushed the revert-9835 branch 3 times, most recently from fef72d8 to 748a94e Compare June 24, 2021 16:22
@JosJuice JosJuice removed the WIP / do not merge Work in progress (do not merge) label Jun 24, 2021
@mbc07
Copy link
Contributor

mbc07 commented Jun 25, 2021

It wasn't the most straightforward process but I remember being able to compile a JIT-less version of the Android port before. You should be able to use a JIT-less build with the Android TV images from the official Android emulator (Google only publishes x86 images for Android TV so regular builds can't be used)...

@JMC47
Copy link
Contributor

JMC47 commented Jun 30, 2021

Can we have this merged before an article in order to make sure a beta isn't broken on T.V. devices?

This partially reverts commit cbc4989
due to a crash: https://bugs.dolphin-emu.org/issues/12561

I can't debug what the cause of the crash is due to not having an
Android TV device. Let's just revert this for now to fix the crash.
@leoetlino
Copy link
Member

This is a pretty severe regression, so let's merge this to fix the crash for now.

@leoetlino leoetlino merged commit be0163b into dolphin-emu:master Jul 6, 2021
11 checks passed
@JosJuice JosJuice deleted the revert-9835 branch July 6, 2021 11:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
4 participants