Speech log filename: Allow date/time variables

[alisonjo315]

First of all, thank you for this addon!

I would love to be able to put date/time variables in the log filename. Or, a setting to create a new file each day or session or something -- I'd be fine with even a new file each session, if that's simpler/more doable than allowing date/time variables in the filename.

I'm afraid I don't have a clue how to do this! -- I googled a bit, this seeeeems potentially useful:
https://serverfault.com/questions/16706/current-date-in-the-file-name

In unix, for my bash histories filename, the setting goes like this:

HISTFILE=/home/$USER/.histories/.bash_history-$(date "+%Y%m%d")

If nothing else, hopefully this ^^ example helps clarify what I'm talking about.

Thanks again!

[XLTechie]

Hi Alison I'm very glad you find this add-on to be helpful, and thanks for the thoughtful feedback! It has been in the roadmap from day one, to have a log rotation feature (inspired by logrotate in Linux, of course), to swap logs out with a .number suffix or similar, on every NVDA restart. Unfortunately, I never quite got around to finishing that feature. Another feature I have in the todo file (no really, look at todo.md in the repo), though added more recently, is to allow each new NVDA session to list its date and time when starting in the log file. But again, I haven't worked on that one. Though that should just be a few lines to get going, and I should probably do it first just for simplicity's sake. I already process the filename for Windows variables such as %homepath%, %temp%, and the like. It shouldn't be difficult to run the filename through an strftime conversion as well. I'm not sure if there are any security implications in doing that, but that's easily researched.

[XLTechie]

I have tested implementing the feature requested in this issue, and while it can work, I ran into some problems in doing it safely. It is still my intention to make this available.

Regarding my mention above of logging the date and time in the log file: this was done as of version 23.1.0.