Squid 🦑 saveable logging and errors and crashes handling

[Pawel]

Thank you for the first reports! I’ll look into those

If this is only what you see in log:

INFO:   [crash]: [17:44:38] squid/squid.lua:121: CRASH: No crash dump found

this is normal, it’s an INFO trace, it means there were no crashes, perhaps I should change the message, but somehow I find it a useful information, so I don’t want to remove it

@NaakkaDev

WARNING:CRASH: Crashdump version or format does not match: Crash version: 1330007625.538976314 Tool Version: 2.51144

I also bumped into this, in different circumstances:

Crash dump after error occurence is nil?

I’m trying to collect a crash, but when it occurs after error: assert(false) crash.write_dump() The error is printed, but I get a warning in console after: WARNING:CRASH: Crashdump version or format does not match: Crash version: 1128354388.538982981 Tool Version: 2.51144 and then: local dump = crash.load_previous() dump is nil When there is no error (assert commented out) the dump is correct (not nil). Why is this happening?

But I don’t know what it is and why there is such a warning.

@NaakkaDev

When this happens squid writes its things into the newest existing log file, instead of creating a new one.
This could be a non-issue, but if the engine crashes too early then the log files might get messy.

Yeah, this is an issue

EDIT:
@NaakkaDev maybe you are opening and closing game in the very same minute twice or more?

If so it will be written in the same log file, because squid appends logs to a file with a name with a time stamp in format YYYY-MM-DD_hh_mm, so if you call squid.init() in the very same minute, e.g. 18:51, all logs will be written into one file. I think narrowing it down to a minute is enough for usual use cases, so I won’t be adding seconds to the time stamp

If you configure is_adding_timestamp in game.project to 0 it will be resulting in all logs being written to a single file always, but also bear in mind in such case, squid will not automatically remove old logs (because it is based on those time stamps in file names)