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

PatchInstaller refactor + index/integrity autogen #1418

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

Soreepeong
Copy link
Contributor

@Soreepeong Soreepeong commented Nov 1, 2023

Changes

  1. Split out handlers for each commands into different files.
  2. Added index-update: download the patch list, download the patch files if necessary, and create new patch index files from the latest resumable patch index file.
XIVLauncher.PatchInstaller.exe index-update ^
    C:\PATH_TO_PATCH_ROOT  ^
    -u <SE_username> ^
    -p <SE_password> ^
    -o <SE_OTP_if_any> ^
    --no-verify-old-patch-hash ^
    --no-verify-new-patch-hash
  1. Added index-create-integrity: create and print IntegrityCheckResult in JSON format.
XIVLauncher.PatchInstaller.exe index-create-integrity ^
    C:\PATH_TO_PATCH_ROOT ^
    C:\PATH_TO_PATCH_ROOT\game\4e9a232b\D2023.10.23.0000.0000.patch.index ^
    C:\PATH_TO_PATCH_ROOT\game\ex1\6b936f08\D2023.10.23.0000.0000.patch.index ^
    C:\PATH_TO_PATCH_ROOT\game\ex2\f29a3eb2\D2023.10.23.0000.0000.patch.index ^
    C:\PATH_TO_PATCH_ROOT\game\ex3\859d0e24\D2023.10.23.0000.0000.patch.index ^
    C:\PATH_TO_PATCH_ROOT\game\ex4\1bf99b87\D2023.10.23.0000.0000.patch.index

Confirmed stuff

  • index-create-integrity results the exact same file for 2023.10.23.0000.0000.
  • index-update worked across 6.50 and 6.51.
  • index-create, index-verify, and index-repair still work. (Mostly for debug purposes)
  • install still works.

Non-confirmed stuff

  • RPC patch/repair.

Not implemented

  • Patch download using signed URLS.

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.

None yet

1 participant