Roll on each process run

carlopantaleo commented 3 years ago

I added the option rollOnEachProcessRun to LoggerConfiguration which, if set to false, mimics the behaviour of log4net. That is:

Run 1 on day 1: logfile.log is created.
Run 2 on day 1: logs are appended to logfile.log
Run 1 on day 2: old log file is rolled to logfile20210521.log (or logfile20210521-001.log etc. if locked or if it already existed) and a new logfile.log is created.
Run 2 on day 2: logs are appended to day 2's logfile.log
...and so on.

Those changes should not be breaking, because the option rollOnEachProcessRun is defaulted to true which is the previous behaviour. I also added some unit tests for completeness.

carlopantaleo commented 3 years ago

AppVeyor build with Visual Studio 2017 fails because tests target also .NET Core. The build on Ubuntu succeeds, indeed...

numito commented 3 years ago

Ok thank you AppVeyor error is because if donet3.1 target which is not compatible with visual studio 2017. I will restaure target donet2.2

dfacto-lab / serilog-sinks-file

Roll on each process run #10