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

Graceful fallback to readonly mode if RDB can't be opened readwrite #15569

Merged
merged 3 commits into from
Aug 10, 2023

Conversation

JoeOsborn
Copy link
Contributor

My guess is that this will fix #15566 , but I can't test on UWP.

This PR also replaces some stray tabs with spaces.

@GABO1423
Copy link
Contributor

GABO1423 commented Aug 9, 2023

I should be able to test this later today, I'll let you know what results I can find.

@GABO1423
Copy link
Contributor

GABO1423 commented Aug 9, 2023

This sadly did not fix the issue @JoeOsborn

@JoeOsborn
Copy link
Contributor Author

Interesting. The only code path I changed connected with just listing database entries was in opening the DB with different flags—the rest of the changes concerned find_entry, find_index, and create_index. Are those used in RA’s database listing code?

@JoeOsborn
Copy link
Contributor Author

Sorry for the CI noise, I’m trying to fix this from my phone!

@GABO1423
Copy link
Contributor

I got the strength to get to the computer, I'll get to testing this now

@GABO1423
Copy link
Contributor

Good news, these new commits did the trick!

Unknown-2023_08_09-23_13_31
Unknown-2023_08_09-23_13_53

@JoeOsborn
Copy link
Contributor Author

I’m glad it works. I still don’t know why the fallback to opening readonly didn’t do the trick, but it does feel more correct to only open the file readonly when we don’t need to write to it.

@LibretroAdmin
Copy link
Contributor

Would you guys like me to squash and merge this PR?

Would that fix the issue for Xbox?

@GABO1423
Copy link
Contributor

Would you guys like me to squash and merge this PR?

Would that fix the issue for Xbox?

Yes, the screenshots above show that this fixes the issue.

@LibretroAdmin LibretroAdmin merged commit 679083a into libretro:master Aug 10, 2023
22 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.

(UWP) Databases don't work anymore
3 participants