Skip to content

Conversation

@sjenkins7
Copy link
Contributor

This corrects the casting for the Remote Patch Installer on Finish, which triggers failures in Core.

See: goatcorp/XIVLauncher.Core#287 (comment) for context

Tested on Linux via Flatpak

@wolfcomp
Copy link
Contributor

This will cause the patcher to fail on windows from what has been talked about in the discord.

@sjenkins7
Copy link
Contributor Author

sjenkins7 commented Dec 19, 2025

I don't have a Windows install to test with right now - will see if I can get one together to make sure it works on both

@sjenkins7 sjenkins7 force-pushed the fix-patcher-failure-on-finish branch from 5034ec9 to 65f1d1b Compare December 19, 2025 23:23
@sjenkins7
Copy link
Contributor Author

sjenkins7 commented Dec 19, 2025

Was able to reproduce it on both at the end of patching boot

Did a clean setup on both Windows & Linux. I don't have the space right now to do a full re-download on both a Windows VM & Linux, but seeing it pass the boot stage gives me confidence that both appear to work now.

Tested on Windows by building an MSI with the same workflow as in https://github.com/goatcorp/FFXIVQuickLauncher/blob/master/.github/workflows/ci-workflow.yml
Tested on Linux via Flatpak

Doing one last test, by restoring my Linux install to a snapshot before today's HF, but I believe this is good to go if someone wants to sanity check on their side too

EDIT: clarification

Windows installer implicitly triggers ToString
Linux does not - causing failure
@sjenkins7 sjenkins7 force-pushed the fix-patcher-failure-on-finish branch from 65f1d1b to b38a40c Compare December 19, 2025 23:32
@sjenkins7
Copy link
Contributor Author

Also patched up from before the HF to current on Linux without issue

@goaaats goaaats merged commit 8547d31 into goatcorp:master Dec 20, 2025
2 checks passed
@sjenkins7 sjenkins7 deleted the fix-patcher-failure-on-finish branch December 20, 2025 21:03
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