Explicit Version of monolog dependency for Symfony2.1 Bundle

[jpgerdeman]

The MonologBundle throws the following error, when including it in a Symfony 2.1 Application.

Fatal error: Can't inherit abstract function Symfony\Component\HttpKernel\Log\LoggerInterface::alert() (previously declared abstract in Psr\Log\LoggerInterface)
 in C:\projects\iRed\project\vendor\symfony\symfony\src\Symfony\Bridge\Monolog\Logger.php on line 23

Call Stack:
    0.0002     336760   1. {main}() C:\projects\iRed\project\app\console:0
    0.0287    2280256   2. Symfony\Component\Console\Application->run() C:\projects\iRed\project\app\console:22
    0.0346    2528360   3. Symfony\Bundle\FrameworkBundle\Console\Application->doRun() C:\projects\iRed\project\vendor\symfony\symfony\src\Symfony\Component\Console\Application.php:105
    0.0346    2528360   4. Symfony\Bundle\FrameworkBundle\Console\Application->registerCommands() C:\projects\iRed\project\vendor\symfony\symfony\src\Symfony\Bundle\FrameworkBundle\Console\Application.php:68
    0.0346    2528360   5. Symfony\Component\HttpKernel\Kernel->boot() C:\projects\iRed\project\vendor\symfony\symfony\src\Symfony\Bundle\FrameworkBundle\Console\Application.php:83
    0.0480    2770136   6. Symfony\Component\HttpKernel\Kernel->initializeContainer() C:\projects\iRed\project\app\bootstrap.php.cache:571
    0.0481    2770712   7. Symfony\Component\HttpKernel\Kernel->buildContainer() C:\projects\iRed\project\app\bootstrap.php.cache:859
    0.3368    7278408   8. Symfony\Component\DependencyInjection\ContainerBuilder->compile() C:\projects\iRed\project\app\bootstrap.php.cache:950
    0.3799    7307816   9. Symfony\Component\DependencyInjection\Compiler\Compiler->compile() C:\projects\iRed\project\vendor\symfony\symfony\src\Symfony\Component\DependencyInjection\ContainerBuilder.php:453
    3.3270   18675904  10. JMS\AopBundle\DependencyInjection\Compiler\PointcutMatchingPass->process() C:\projects\iRed\project\vendor\symfony\symfony\src\Symfony\Component\DependencyInjection\Compiler\Compiler.php:119
    3.6291   26451248  11. JMS\AopBundle\DependencyInjection\Compiler\PointcutMatchingPass->processDefinition() C:\projects\iRed\project\vendor\jms\aop-bundle\JMS\AopBundle\DependencyInjection\Compiler\PointcutMatchingPass.php:59
    3.6291   26451280  12. class_exists() C:\projects\iRed\project\vendor\jms\aop-bundle\JMS\AopBundle\DependencyInjection\Compiler\PointcutMatchingPass.php:96
    3.6291   26451560  13. Symfony\Component\ClassLoader\DebugClassLoader->loadClass() C:\projects\iRed\project\vendor\symfony\symfony\src\Symfony\Component\ClassLoader\DebugClassLoader.php:0
    3.6297   26461656  14. require('C:\projects\iRed\project\vendor\symfony\symfony\src\Symfony\Bridge\Monolog\Logger.php') C:\projects\iRed\project\vendor\symfony\symfony\src\Symfony\Component\ClassLoader\DebugClassLoader.php:82

Script Sensio\Bundle\DistributionBundle\Composer\ScriptHandler::clearCache handling the post-update-cmd event terminated with an exception

I assume that the inclusion of psr/log is meant for Symfony 2.2. Creating a 2.1 branch of the bundle - even if it is just for including the correct monolog version - is necessary as to not trip developers waiting before switching to 2.2, or cannot yet make that switch.

Symfony 2.1 works with Monolog 1.2.1

[stof]

This is wrong. It should not require 1.2.1. This is too specific.

Btw, we should not even need a requirement on Monolog here. We already have the constraint in the monolog bridge, which already contains the stricter requirement

[jpgerdeman]

The requirement of the bridge is "monolog/monolog": ">=1.0,<1.3-dev", which is to weak a constraint. So it should be "monolog/monolog": ">=1.0,<1.2.1 in the bridge, correct?

Should the inclusion of psr/log work with Symfony 2.1 at all?

Edit:
Also, shouldn't the monolog requirement be removed from the bundle? It already requires the bridge, which requires monolog and has a stricter constraint.

[stof]

it is not too weak. The requirement in the bridge is right in all branches (but the released version contain the old requirement)

[stof]

anyway, I merged #25 for now. I will see later if I remove the requirements entirely