Space Mode: Export to MiSTer copied everything then deleted it about ammister HOT 13 CLOSED

Doh, I can't attach builds here.

Ok, I think I have this working. I'll do more testing tomorrow and if all looks good, I'll cut a new build.

from ammister.

I think this is great, and will hopefully make multi-platform development easier going forward. Thanks for the updates, I'll try this on my windows machine hopefully tonight and let you know how it goes.

from ammister.

One last note for the day: just did an export using Speed mode, and a bunch of stuff is also missing and/or deleted. Hopefully easy to repro for you on Windows.

from ammister.

I haven't truly dug into this yet. But I did manage to squeeze in a couple of Windows fixes this morning.

If you have time, an export log would be helpful

1. install the latest version
2. Export a plan to a mister and have the problem happen
3. go to Catalog > Open Folder
4. go up one directory
5. zip up all the export-log-exportToMister... files in that directory and attach them here.

When doing this, the smaller the plan the better. Hopefully just a few games causes the problem. Please use space mode, otherwise the log will be gigantic.

And if you don't have time, no worries.

from ammister.

Just ran it twice, and both times it seemed to not delete stuff, but the weird thing this time is that even though it's space mode, it didn't delete the stuff that was already there that should have gotten deleted.

logs attached

export-log--exportToMister-New_Plan-2023-04-17T16_40_03.652Z.zip

from ammister.

Thanks. Just quickly looking at the logs I see several issues, they mostly boil down to windows using \ and mister using / for path separator.

from ammister.

I'm just going to switch Windows to /. Initially \ was simple and I chose it to make the app feel a little bit more familiar with Windows users. But it's just not worth it. Windows and Node on Windows both support /, and this will remove a lot of Windows specific code from the app.

I can always have it switch from / to \ in the details modal if needed.

from ammister.

That sounds wise. I didn't even notice slashes in the program interface tbh

from ammister.

Hoo this is painful. I'll always curse Microsoft for doing this :)

I think I have this working. A huge downside is all existing plans created on Windows will no longer work. I think it's best just to rip that band aid off though.

The upside is the code is still a little crazy, but not nearly as bad as it was before.

from ammister.

Yeah, it's early days, I don't think you should feel bad about breaking existing plans at all.

from ammister.

I have released 0.25.7. In testing I could not find anything wrong, but there may still be some minor issues.

catalog.json needs to be recreated for Windows users, noted in the release notes.

With this, AMMiSTer on Windows is now pretty much the same as on Linux and Mac. So one interesting upside: plan files can now work across OSes.

from ammister.

This should be fixed w 0.25.7

from ammister.

Confirming this is fixed for me Win 10 v0.26.0

from ammister.