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

Game saves not being detected because of smart apostrophe #282

Closed
darklinkpower opened this issue Nov 23, 2023 · 6 comments
Closed

Game saves not being detected because of smart apostrophe #282

darklinkpower opened this issue Nov 23, 2023 · 6 comments
Labels
bug Something isn't working

Comments

@darklinkpower
Copy link

Ludusavi version

v0.21.0

Operating system

Windows

Installation method

Standalone

Description

Ludusavi is not detecting game saves for game The Witch's Love Diary on GOG and I'm not sure why. I could add a manual configuration but I think it would be better to identify the underlying issue causing this.

From PCGW page:

image

Configured roots in Ludusavi:

image

Game saves location:

image

Ludusavi Playnite notification when trying to backup:

image

Also not found on Ludusavi itself:

image

Logs

No response

@darklinkpower darklinkpower added the bug Something isn't working label Nov 23, 2023
@mtkennerly
Copy link
Owner

That's weird - I don't have the game myself, but when I add the folder for it, it's detected. If G: is Google Drive, I wonder if that has anything to do with it.

Could you run this and give me the log files? (example using Command Prompt)

set RUST_LOG=ludusavi=trace
ludusavi backup --force "The Witch's Love Diary"

@darklinkpower
Copy link
Author

Thanks for the swift response! Here are the logs:

ludusavi_rCURRENT.log

The weird thing is that the directory exists according to the logs but it's not seeing it for some reason:

[2023-11-23T06:32:11.565Z] TRACE [ludusavi::scan::launchers] launcher games found (Epic - G:/Games\PC\Epic Games): {}
[2023-11-23T06:32:11.566Z] DEBUG [ludusavi::scan::launchers::generic] ranking installations for GogGalaxy: G:/Games\PC\GOG Galaxy
[2023-11-23T06:32:11.566Z] DEBUG [ludusavi::scan::launchers::generic] actual install folders: A Space for the Unbound | Amatsutsumi | CHAOS HEAD NOAH | CHAOS HEAD NOAH Downloads | Control | Dungeon Siege III | Full Metal Daemon Muramasa | Grim Fandango Remastererd | Mirror's Edge | Spelunky | The Witch’s Love Diary
[2023-11-23T06:32:11.566Z] TRACE [ludusavi::scan::launchers::generic] [The Witch's Love Diary] looking for install dir: The Witch's Love Diary
[2023-11-23T06:32:11.566Z] TRACE [ludusavi::scan::launchers::generic] [The Witch's Love Diary] looking for install dir: The Witch's Love Diary
[2023-11-23T06:32:11.566Z] TRACE [ludusavi::scan::launchers] launcher games found (GogGalaxy - G:/Games\PC\GOG Galaxy): {}
[2023-11-23T06:32:11.566Z] DEBUG [ludusavi::scan::launchers::generic] ranking installations for Other: G:/Games\PC\itch.io\downloads

@darklinkpower
Copy link
Author

darklinkpower commented Nov 23, 2023

I see the issue. It's the apostrophe being different.

The install folder is The Witch’s Love Diary with
image

But It is looking for The Witch's Love Diary with an ' character. What do you think the best solution would be? That install folder is the default Gog Galaxy installed to and the name of the game is this one, on both Steam and GOG. I understand that it could be manually fixed but I wonder if an automatic solution would be possible, also considering that other games may suffer from this. I've noticed that GOG installers have certain limitations regarding the games install paths and a general solution could benefit other games. Other examples that I remember is GOG installers having trouble with the ; character.

Steam uses The Witch's Love Diary as the install directory so it won't have this issue.

@mtkennerly
Copy link
Owner

Ah, that's a tricky one. For now, I've added an override in the manifest: mtkennerly/ludusavi-manifest@b5392a4 . Long-term, maybe Ludusavi should convert smart apostrophes/quotes to normal ones when looking for a match.

@darklinkpower
Copy link
Author

Thank you, that fixed the issue! I see how that can be tricky to solve, specially considering that they may be cases where the apostrophe changes and some that it doesn't. I don't know if the issue should be closed or not to serve as a tracker for this so I won't do it but you can close it if that would be appropriate.

@mtkennerly mtkennerly changed the title Game saves not being detected Game saves not being detected because of smart apostrophe Dec 2, 2023
@mtkennerly
Copy link
Owner

I think I'll close this for now since there are very few manifest entries with smart apostrophes/quotes. If we find more cases that are incorrect like this one, we can adjust it in the manifest directly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants