I removed SystemDaemonFormatter in favor of use LineFormatter with a different argument on constructor.

Not sure what the point is here. This forwards logs to a system daemon log it seems. Is there a real use case or is it just for BC support for old apps or something?

I have used this handler when writing/maintaining daemons with System_Daemon.
System_Daemon forces to define a file to write logs but it only write logs on this file. Using Monolog and defining System_Daemon as a handler:

• I have a better API;
• I don't need to call System_Daemon and Monolog to write the same message

I sent this pull request because I think other people may want it as me, but if you judge it only as a mine necessity I will understand.

The thing is I am trying to avoid new optional dependencies since I'd like to move all optional deps into separate packages for 2.0, so I think it'd be better if you create a new package for this, feel free to call it monolog/system-daemon-handler, and it can require monolog/monolog + pear/system_daemon explicitly, so there are no optional dependencies to add for users, they just require your handler package if they want to use it.

Looks great for me! Thanks.

I will copy Monolog\TestCase, but I think people will like if you move it to "src/" directory or create a Monolog\Test namespace on the future...

Yup it probably should be moved to the src dir to allow re-use. I'll add it to the 2.0 notes thanks.

@Seldaek one doubt when reading the v2 changes I came to here.

Does this means you are planning to move all tests to src folder ?

@harikt nope, just the TestCase since it could be reused by standalone handler libs.

Thank you @Seldaek .

Just like @harikt, just came here because I saw the #197 task list.

I don't really get the point of moving the TestCase in the src folder, as it is a helper for the tests and is already loadable by PSR-4 (Hence a correct namespace Monolog\TestCase)... If we leave it alone, I don't really see what is preventing standalone handler libs to use the TestCase, as it should already need to load monolog\monolog anyway ?