ScrollsModLoader icon indicating copy to clipboard operation
ScrollsModLoader copied to clipboard

Auto repatch on file change

Open MaPePeR opened this issue 12 years ago • 5 comments

Detects if a mod.dll file is newer than the Assembly-CSharp.dll and will repatch automatically.

To be on the safe side there should be a way to restrict it to not do it twice...

MaPePeR avatar Sep 10 '13 18:09 MaPePeR

Thats a nice way to "fix" it. Although this is more a nice, new feature then a fix.

I cannot review code in holidays (laptop is at home). I will test and accept both of your pull requests, when I am back.

Drakulix avatar Sep 10 '13 21:09 Drakulix

Enjoy your holidays!

This one is a bit 'dangerous', because under weird circumstances(write times not saved in file system?) it may cause an endless repatch loop.

MaPePeR avatar Sep 11 '13 05:09 MaPePeR

The question is if Mono/.net implements it on all required platforms/filesystems.

We would need to cover NTFS, FAT32, HFS and HFS+ at least for Mac and Windows.

Drakulix avatar Sep 11 '13 06:09 Drakulix

I think I am not going to include this right now, because we have no tested this functionality on enough systems and it might be buggy in the mono runtime. Once you have a more elaborated version of this, that maybe saves the SHA1 hash of the mod file for change detection, I am happy to add this feature.

Drakulix avatar Oct 22 '13 20:10 Drakulix

Yea, i agree. It might be nice for development, but is not useful for the end user and only introduces more risks.

MaPePeR avatar Oct 23 '13 06:10 MaPePeR