LADX: move client out of root

[threeandthreee]

What is this fixing or adding?

Moves the client out of root.

Much easier to distribute beta clients like this, and as I understand it we want this stuff out of root anyway.

How was this tested?

It's been in the beta build for a good while now with no reported issues.

[threeandthreee]

@palex00 would you mind testing?

[Berserker66]

old client needs to be explicitly deleted in inno_setup.iss as well.

[Zunawe]

If you're removing the top-level script entirely, you'll want to modify inno_setup.iss to change the windows file extension registration from the LADX client exe to the launcher exe.

[threeandthreee]

Root: HKCR; Subkey: ".apladx";                                   ValueData: "{#MyAppName}ladxpatch";        Flags: uninsdeletevalue; ValueType: string;  ValueName: "";
Root: HKCR; Subkey: "{#MyAppName}ladxpatch";                     ValueData: "Archipelago Links Awakening DX Patch"; Flags: uninsdeletekey;   ValueType: string;  ValueName: "";
Root: HKCR; Subkey: "{#MyAppName}ladxpatch\DefaultIcon";         ValueData: "{app}\ArchipelagoLauncher.exe,0";                           ValueType: string;  ValueName: "";
Root: HKCR; Subkey: "{#MyAppName}ladxpatch\shell\open\command";  ValueData: """{app}\ArchipelagoLauncher.exe"" ""%1""";                  ValueType: string;  ValueName: "";

Like this?

[threeandthreee]

old client needs to be explicitly deleted in inno_setup.iss as well.

I could use some guidance on how to do this

[threeandthreee]

I think I worked it out.