Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

dev-master does not fetch "psr\log" as Monolog's dependency #27

Closed
tiraeth opened this issue Jan 8, 2013 · 4 comments
Closed

dev-master does not fetch "psr\log" as Monolog's dependency #27

tiraeth opened this issue Jan 8, 2013 · 4 comments

Comments

@tiraeth
Copy link

tiraeth commented Jan 8, 2013

Using monolog/monolog (25a97a) (dev-master) for the development purposes with symfony/monolog-bundle @dev does not fetch psr/log dependency and results in rising exception right after composer update:

PHP Fatal error:  Interface 'Psr\Log\LoggerInterface' not found in 
/path/vendor/monolog/monolog/src/Monolog/Logger.php on line 28

Do you have any idea why does it not work properly? Fetching monolog/monolog and calling composer update works and psr is available under vendor folder.

@stof
Copy link
Member

stof commented Jan 8, 2013

dev-master of this bundle should not allow you to fetch a version of Monolog implementing PSR-3, so it is logical that you don't get psr/log

@tiraeth
Copy link
Author

tiraeth commented Jan 8, 2013

So why the PSR-3 version of Monolog is being downloaded if I have symfony/symfony as dev-master and everyone knows it wont work? Shouldn't it be done in a way that everything works? Assuming I work on 2.1.*@dev, I can't build the project. If I downgrade it to 2.1.6 - it works, mainly because older version of monolog/monolog is fetched.

@Seldaek
Copy link
Member

Seldaek commented Jan 8, 2013

I will send a patch to symfony 2.2/master later today to fix the whole thing. In the meantime please stick to monolog 1.2.*

@stof
Copy link
Member

stof commented Jan 8, 2013

@Seldaek how could this allow Monolog 1.3 ? https://github.com/symfony/symfony/blob/master/composer.json#L62

@Seldaek Seldaek closed this as completed Mar 12, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants