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

Linux: Fix AssetBundleManager.GetAssetVersion only working sometimes #1130

Merged
merged 1 commit into from
Feb 9, 2025

Conversation

regalialong
Copy link
Contributor

On Linux, the cache version directories are lowercase but versionHex here is completely uppercase. This would be fine on case insensitive systems but ext4 isn't, hence it will fail the existence check later on in CheckVRChatCache.

The annoying part is that if the version hexadecimal does not create letters, the check can work as intended :))))

You don't know how much """"""fun""""""" this was to debug this without a project.

Untested on Windows so please check

@Natsumi-sama
Copy link
Member

Oh yea these sorts of bugs are the worst to hunt down.

@regalialong
Copy link
Contributor Author

Oh yea these sorts of bugs are the worst to hunt down.

Oh no, finding it was fine, I figured it was something along the lines.
The actual issue is that the only valid project that Rider takes it the CEF project, the Electron project refuses to load

so I can't do breakpoints in the actual files, I need to attach, halt, and then ask it to disassemble the app dll, then make breakpoints there

@Natsumi-sama Natsumi-sama merged commit 9d7ba34 into vrcx-team:master Feb 9, 2025
Natsumi-sama pushed a commit to Natsumi-sama/VRCX that referenced this pull request Feb 9, 2025
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.

2 participants