latest php-di seems to break piwik #7664
Comments
|
doing a "git checkout 56ac462" (e.g. before the above mentioned commit) in vendor/mnapoli/php-di seems to fix the issue |
|
Hi @arunpersaud, yes I worked on PHP-DI this week end and was planning to update Piwik today ;) However this problem shouldn't happen on your end because you shouldn't run |
|
no, I guess I have been using composer the wrong way and got lucky that it worked for the last years;) Will make sure that I run "composer install" in the future. Thanks |
|
No worries thanks! |
mnapoli
added
the
answered
|
@mnapoli why are we not using fixed versions of dependencies? It's really hard to maintain, especially when using dev-master versions. It's also confusing for the end users. |
|
@quba it shouldn't affect end users so it's not really a problem.
What is hard to maintain? We are using the currently version in development because it's a development of the next major version based on the experience we have in Piwik. If we don't do that, then I would have to imagine some features needed for Piwik and release a 5.0 beforehand (some feature require BC breaks), and then if I was wrong Piwik is screwed and PHP-DI is screwed. The current situation is some sort of joint development (it's been working great for the past 5 months, I've only seen twice people doing FYI I've worked intensely on PHP-DI the last week-end and I'm closing in on a 5.0 release, so hopefully it shouldn't be much longer. But again, I don't see where the problem is? |
|
@mnapoli I cloned a fresh Piwik copy yesterday and I got error in PHP-DI class not found error. I tried "composer install", it didn't work. Then removed vendor files and gave "composer udpate". It still doesn't work. Piwik is now broken; could you clarify what's the issue with new git clone? |
|
@saleemkce this is a different issue, I have created a new ticket: #7759 |
Hi
just upated to 2.12.1 and I get the following error:
Seems that php-di removed 'DI\Definition\Source\ChainableDefinitionSource'
in commit f54d3e8ac72a3cc60c31c4081f931495172d6766 [https://github.com/mnapoli/PHP-DI/commit/f54d3e8ac72a3cc60c31c4081f931495172d6766].
Don't really understand the code, but it seems that piwik needs to be updated for this, since
"composer update" is using php-di 5.0.x-dev, which seems to be the latest git-version with that file removed.
The text was updated successfully, but these errors were encountered: