Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
#2366043: Upgrade to Symfony 2.6.0-beta1 got us onto 2.6 beta.
The composer update introduced a new dependency on symfony/debug via http-kernel. Before updating to 2.6 we should understand a bit more about why this was added.
Comment | File | Size | Author |
---|---|---|---|
#13 | composer-symfony-debug.patch | 862 bytes | dawehner |
#6 | upgrade_to_symfony_2_6-2377281-6.patch | 146.69 KB | hussainweb |
#6 | composer-json-only-do-not-test.patch | 1.2 KB | hussainweb |
Comments
Comment #1
jibranJust got mail * Version 2.6 is going to be released.
Comment #2
webchickHm. I don't see evidence of this, neither at https://github.com/symfony/symfony/tags nor https://twitter.com/symfony nor http://symfony.com/download.
Restoring postponed status for now.
Comment #3
hussainwebMeanwhile, we can update to beta2. I have a new issue at #2381579: Upgrade to Symfony 2.6.0-beta2.
Comment #4
TJacksonVA CreditAttribution: TJacksonVA commentedAlthough I should note that Symfony folks are saying that the 2.6 stable might be released this week. The release notes dated November 24th to the 2.6 Beta2 say "Symfony 2.6.0-BETA2 has just been released. This is the last version before the 2.6 final release later this week." In addition, the weekly Symfony update dated November 23rd (relating to the week of Nov 17 - Nov 23) stated "This week, Symfony project focused on tweaking and polishing all its features before the big Symfony 2.6 launch of the next week."
http://symfony.com/blog/symfony-2-6-0-beta2-released
http://symfony.com/blog/a-week-of-symfony-412-17-23-november-2014
If so, we might want to forego upgrading to Symfony 2.6 beta2 and just wait a couple of days for 2.6 stable.
Comment #5
hussainwebAh, I didn't see that post. If it is really this week, then yes, there is not much sense in going to beta2. Anyway, all the tests with beta2 are passing which is good enough for now.
Comment #6
hussainwebSymfony 2.6.0 is out. I am attaching updates to composer.json and the complete patch. The command to update was
composer update symfony/*
Comment #7
catchLooks good.
Comment #8
dawehnerNice!
Comment #9
alexpottThis issue is a critical task and is allowed per https://www.drupal.org/core/beta-changes. Committed 93c75fa and pushed to 8.0.x. Thanks!
Comment #11
Berdirthe issue summary says:
Comment #12
catchWhoops, yes I think we should open a new issue to figure that out.
Comment #13
dawehnerNote: This got introduced when we update to the 2.6 beta, because composer complained. (afaik this was a bug earlier)
I don't really know why, but it seems to be that we can now drop that line again.
Once I updated the composer.json file, running
composer update symfony/http-kernel
(that one has the dependency to symfony/debug) succeeds again.Comment #14
BerdirShouldn't there be some core/vendor code that can be removed now?
Comment #15
catchYes there should, at least if #13 is correct.
Comment #16
dawehnerNo, that is the confusing part about this problem:
So HttpKernel has the following composer.json file:
For some reasons pointing to a beta version required symfony/debug to be explicitly defined, but symfony/debug is a requirement since 2.3
Comment #17
BerdirDid some digging, https://github.com/symfony/HttpKernel/commit/9517776f77e5488655838516fe6... moved symfony/debug from require-dev to require. Doesn't say why in that comment, though.
Anyway, looks like we're back to status quo then?
Comment #18
alexpottCommitted d037482 and pushed to 8.0.x. Thanks!
Comment #20
znerol CreditAttribution: znerol commentedFollow-up: #2385699: Use upstream AbstractEventDispatcherTest as the base class of ContainerAwareEventDispatcherTest
Comment #21
lussolucaPlease integrate also #2375339: Update DependencyInjection YamlFileLoader for Symfony 2.6 to fully use the new Symfony 2.6 functionalities