kleidis / Troppical

Manage your emulators easily
MIT License
51 stars 3 forks source link

[BUG] Torzu installation replaced Sudachi with Torzu #8

Closed SilentSparrow02 closed 3 months ago

SilentSparrow02 commented 3 months ago

Describe the bug I tried to install Torzu into my emulators directory (which already had the Sudachi folder inside it). After installing I found that Torzu was installed in its own folder, but also had replaced Sudachi in the Sudachi folder.

To Reproduce Steps to reproduce the behavior:

  1. Have Torzu and Sudachi installed in the same parent directory
  2. Use Troppical to update Torzu (choosing the parent directory as target location)
  3. Sreenshot below

Screenshots/Videos image

Expected behavior Torzu shouldn't replace Sudachi in its folder

Windows version and build Windows 11 23H2

kleidis commented 3 months ago

It works fine on my end Sudachi was the first one that was installed as you can see on the screenshots

I installed torzu and it didn't replace anything

image

SilentSparrow02 commented 3 months ago

Some more details: In my case I already had Torzu installed in my emulators folder. After the Troppical update I tried to use it to update Torzu. But since Troppical did not recognize the pre-existing version (the update option was greyed out) I selected the proper directory and pressed install. I thought it would overwrite the existing Torzu version. However it ended up overwriting the Sudachi folder instead, as you can see from the picture I attached where the Torzu logo shows in both folders.

Sorry if my original issue wasn't clear enough!

kleidis commented 3 months ago

Are you using the release v1.1?

Or you downloads from the link on the readme / latest ci builds

SilentSparrow02 commented 3 months ago

Yes I am using v1.1

I downloaded it from the releases page yesterday (before you removed assets)

kleidis commented 3 months ago

Can you try the latest release

SilentSparrow02 commented 3 months ago

Thanks, I tried it the problem seems to be resolved now!

The only issue with the new version is that I have to disable controlled folder access on Windows every time I use the app 😅. I tried to whitelist it but it's not working since the process is titled "main.exe" rather than "init_troppical.exe". If it was possible to create a workaround that would be great! (Such as making the process name "troppical.exe" so we can whitelist it) image

Allowlist: image

kleidis commented 3 months ago

Yeah init_troppical.exe is just a small script that downloads the main file from Github and adds it to APPDATA temp directories, So I don't see how changing the name from main.exe to troppical.exe would fix this since the main.exe file gets deleted once you close the app

You can open another issue for this tho, I am going to see what I can do