DingoDjango / snAutosave

An autosave system for Subnautica
MIT License
4 stars 8 forks source link

bugfix/no-saves-on-steam-jul-2020 #7

Closed Bisa closed 4 years ago

Bisa commented 4 years ago

Hi, this ought to fix #6

Please note that in order for the project to build I updated the paths, you will need to set your old hard coded paths with the environmental SUBNAUTICA_PATH

I've also included the dll, remove it if you build it yourself (which I suppose you do).

Bisa commented 4 years ago

Revisiting the build options I realized that building with Configuration Release results in a non-working .dll, I'll just pull this one back for now until I figure that one out.

Bisa commented 4 years ago

Crisis averted. I panicked when the mod was not auto saving every 60 seconds - but this lead me to investigate the DEBUG options further and I realized there's some interesting injections going on with Harmony.

Building with Configuration=Release this time around and it works as intended, attached resulting .dll (you probably still build your one any way)

Bisa commented 4 years ago

I spent some time adjusting the build process after introducing the SUBNAUTICA_PATH earlier, this to allow me to build/debug locally as well as making the change backwards compatible with how I suspect your build environment looks.

I made a new section to the readme in an attempt to make it easier for others like me to start out when they want to build from source.

(the included .dll was built using this updated process)

DingoDjango commented 4 years ago

I don't have the game installed nor do I have access to my desktop PC at the moment.

Can you confirm the latest pull request works? Is the dll Release built and OK to upload to Nexus?

Bisa commented 4 years ago

@DingoDjango - I ran some tests with the debug build which looked to be working, however - just to make sure I did some additional testing with the Release configuration and it's still broken. I'll have to go back to the drawing board.

Be right back! :)

Bisa commented 4 years ago

@DingoDjango - It took another round of debugging but I've fixed (and tested the Release-build .dll) I've opened #8 instead, please disregard this one.