secondlife / viewer

🖥️ Second Life's official client
GNU Lesser General Public License v2.1
206 stars 52 forks source link

Sort out / maybe fix viewer "logfile" option #1681

Open simon-linden opened 2 months ago

simon-linden commented 2 months ago

Environment

Release viewer

Description

The viewer has a bunch of code for the "logfile" setting about logging to a custom file name. That, unfortunately, doesn't seem to work as expected. Look into it, sort out what's working and fix what is not, so that we can:

Reproduction steps

Use the log settings and look for files as you expect

akleshchev commented 2 months ago

Relevant code: LLAppViewer::initLoggingAndGetLastDuration

Also looks like it is supposed to be possible to change log file using command argument UserLogFile or a debug setting UserLogFile.

P.S. Currently viewer generates an uuid log for second instance, but that id isn't used anywhere else, probably should be used for crash reporting if we report crashes from second isntance