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

XP: game specific configs ignored in recent build ? #7967

Closed
Trokinos opened this issue Sep 13, 2015 · 24 comments
Closed

XP: game specific configs ignored in recent build ? #7967

Trokinos opened this issue Sep 13, 2015 · 24 comments

Comments

@Trokinos
Copy link

Was using PPSSPP 1.0.1-830-gc671b07 on win32 (it was from 2015-08-27 15:13:05 ) and had no such problem.

Today i decided to give a try to the latest build and downloaded PPSSPP 1.0.1-2654-g315ff81 ( from the 2015-09-13 21:31:12 ) and after moving the memstick folder and the games into that new PPSSPP directory, on the game list screen after launch i quickly noticed that the games i had a specific configs were missing their usual icon that tells they have a specific game config.

And holding a click to them confirmed that despite those specific configs are in the usual place , this version of PPSSPP was not considering there was a custom config as it asked if i wanted to create one.

I have used a lot of builds and that's the 1st time it's happening.

@Trokinos
Copy link
Author

checked a few builds and it appears the 1st one i have this problem on my system is
1.0.1-2631-g2eb45f5

with 1.0.1-2630-g487f184 being the last version that had the specific game config icon appearing on their related game picture.

edit
Just to be sure i'm explaining it well, with all the file in correct location in the memstick , in
1.0.1-2630-g487f184
I see immediately which game has a custom config with the small icon appearing in the game picture
http://i.imgur.com/tYOEHt2.jpg
And if i hold a click on it, the interface confirm it has its own config because it ask me if i want to delete it
http://i.imgur.com/NWnJH6l.jpg

While with 1.0.1-2654-g315ff81, with all the files correctly in their memstick location, the menu does not display the icon telling there's a game with a custom config
http://i.imgur.com/MhZRElZ.jpg

And indeed if i hold a click on the game picture, on the menu there's no more asking if i want to delete it, it ask me if i want to create one instead, despite there's already one.
http://i.imgur.com/dJ7YHtA.jpg

@Trokinos
Copy link
Author

After testing more, the problem is actually bigger than i thought , none of the games i launch are finding their save data (despite they're all there (i mean in ...\ppsspp\memstick\PSP\SAVEDATA\ )

I can't seem to find how to edit the title of this bug report as it's much bigger than only game-specific config.
1.0.1-2630-g487f184 seems to be the last version that work correctly on my win32 os (winxp)

@unknownbrackets
Copy link
Collaborator

Just to clarify, v1.0.1-2631-g2eb45f5 (2eb45f5) doesn't work?

Are you trying with PPSSPP in the exact same folder? You're sure you have free disk space?

-[Unknown]

@Trokinos
Copy link
Author

I'm sorry, only the detection of the "game specific" .ini file is last correctly detected by
1.0.1-2630-g487f184
but it does not detect the save data and starting with
v1.0.1-2631-g2eb45f5
the game specific ini files aren't detected anymore (and the save data are all undetected too)

files are all in the correct memstick directory that i transfer from builds to builds, doing this has always worked for all the builds i have downloaded so far since i'm using ppsspp.
Hardrive space isn't a problem on my system, plenty gb free.

I'm downloading some past builds to see which one started to not find the save data , so far v1.0.1-857-g08b340f is correct, all the files are detected.
Will update this report when i find

@Trokinos
Copy link
Author

Ok here is what i found after testing several builds and testing with svr 2011 that never had any kind of saving problem in the past (unlike some other games like gods eater burst).

Up to v1.0.1-891-g1c67de0 on win32 PPSSPP does find the save data, game specific ini are found
In v1.0.1-899-g1b4b60d, on win32 PPSSPP does NOT find the save data, game specific ini are found

unfortunately between those 2 builds there's no win32 ones (x86), so i can't pinpoint which one started exactly.
Then starting with v1.0.1-2631-g2eb45f5 , on win32 PPSSPP does not find game specific ini

@unknownbrackets
Copy link
Collaborator

Okay, well, that range is the update to MSVC 2015. Maybe there's a compatibility issue with Windows XP.

I think 2eb45f5 is a totally separate issue. Not discounting it, just saying it doesn't seem related to the MSVC 2015 problem and probably is not Windows XP specific.

-[Unknown]

@sum2012
Copy link
Collaborator

sum2012 commented Sep 15, 2015

I use vmware to confirm.
Yes cannot see "save" or "save status"
There is a file called _writable_test.$$$ in memstick folder after start ppsspp.

edit:vmware OS:Windows XP

@hrydgard
Copy link
Owner

Seems like File::Exists must return the wrong result, otherwise the file would have been deleted. Might be some strange unicode issue? Hm..

@sum2012
Copy link
Collaborator

sum2012 commented Sep 22, 2015

Let me try to rename the file in the code

hrydgard added a commit that referenced this issue Sep 22, 2015
Might possibly help #7967 (XP) by avoiding _wstat64? That would be a libc bug though.
@hrydgard
Copy link
Owner

@sum2012 can you try with #7994 ? Would be interesting if it worked.

@sum2012
Copy link
Collaborator

sum2012 commented Sep 22, 2015

@hrydgard ok,try your method first.
If fail,I try to rename _writable_test.$$$ to _writable_test.aaa

@hrydgard
Copy link
Owner

ok, worth trying that too.

@sum2012
Copy link
Collaborator

sum2012 commented Sep 22, 2015

@hrydgard Unfortunately,both method fail

@hrydgard
Copy link
Owner

Thanks for testing @sum2012 , this is very strange..

@hrydgard
Copy link
Owner

I have set up my own XP VM now, I will investigate in detail tomorrow. It seems IsDirectory is the problem.

@hrydgard
Copy link
Owner

https://connect.microsoft.com/VisualStudio/feedback/details/1557168/wstat64-returns-1-on-xp-always

That would explain it.

Will work around it tomorrow, by simply avoiding the function and calling native Windows functions instead.

@sum2012
Copy link
Collaborator

sum2012 commented Sep 22, 2015

@hrydgard maybe there is additional bug in native
log:
https://gist.github.com/sum2012/972b298375d1d0cbb4c0

733 37:16:750 user_main W[COMMON]: FileUtil.cpp:464 GetSize: failed C:\test\ppsspp-v1.0.1-2624-g936ccbf-windows-x86\ppsspp\memstick/PSP\SAVEDATA\ULJM05556\ICON0.PNG: No such file

734 37:16:750 user_main W[SCEGE]: Util\PPGeDraw.cpp:848 Bad PPGeImage - not a valid png
735 37:16:765 user_main W[COMMON]: FileUtil.cpp:179 IsDirectory: stat failed on C:\test\ppsspp-v1.0.1-2624-g936ccbf-windows-x86\ppsspp\memstick/PSP\SAVEDATA\ULJM05556\ICON0.PNG: 系統不支援這個功能。 (in english:system don't support this function)

@sum2012
Copy link
Collaborator

sum2012 commented Sep 22, 2015

@hrydgard oh,I don't see your last comment,anyway good luck

@hrydgard
Copy link
Owner

#7994 should now fix all these issues.

@hrydgard
Copy link
Owner

It passed all checks so I merged it.

@Trokinos , can you re-test with the latest build when it's done?

@hrydgard hrydgard changed the title game specific configs ignored in recent build ? XP: game specific configs ignored in recent build ? Sep 23, 2015
@Trokinos
Copy link
Author

Hello,

I gave a test to v1.0.1-2768-gd8b43dd today on my XP win32 and noticed that with this version PPSSPP correctly detect both game specific configs and the saved games again, so it seems the problems are fixed

Thank you very much.

@sum2012
Copy link
Collaborator

sum2012 commented Sep 24, 2015

@Trokinos
Now you can close this issue

@Trokinos
Copy link
Author

Didn't noticed i could do that, sorry, i thought it was only developers that closed issues.
Anyways, closing the issue(s) as they are now fixed :)

@hrydgard
Copy link
Owner

@Trokinos great, thank you for reporting the issue and testing the fix.

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

No branches or pull requests

4 participants