Update Symfony dependency [#1879410]

Problem/Motivation

I made some changes in Symfony (https://github.com/symfony/symfony/pull/6500, https://github.com/symfony/symfony/pull/6501, and https://github.com/symfony/symfony/pull/6503) to be able to remove some hacks that were done in Drupal when integrating Symfony. Updating the Symfony deps will allow a follow-up patch that will get rid of some of these hacks.

Proposed resolution

The patch is just an update of dependencies when running composer update.

Comment	File	Size	Author
#8	composer-update_8.patch	285.38 KB	fabpot
#8
#3	composer-update_3.patch	273.26 KB	fabpot
#3
	composer-update.patch	285.31 KB	fabpot

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

3 January 2013 at 23:41

Status:

Needs review

» Needs work

The last submitted patch, composer-update.patch, failed testing.

Comment #2

Berdir

German

Switzerland

CreditAttribution: Berdir commented 3 January 2013 at 23:45

You probably want to exclude guzzle from the update. The 3.0.7 release is broken as it references a plugin that is not a dependency.

It's already fixed but needs a new release before we can update it. There's a separate issue for it: #1875818: Update composer.json to use the latest stable 3.0.x version of Guzzle

Comment #3

fabpot CreditAttribution: fabpot commented 4 January 2013 at 08:54

Status:

Needs work

» Needs review

File	Size
composer-update_3.patch	273.26 KB

The new patch only updates Symfony

Comment #4

Crell CreditAttribution: Crell commented 4 January 2013 at 16:15

Status:

Needs review

» Reviewed & tested by the community

Spiffy.

Comment #5

webchick

she/they

English

Vancouver 🇨🇦

CreditAttribution: webchick commented 4 January 2013 at 16:26

#3: composer-update_3.patch queued for re-testing.

Comment #6

webchick

she/they

English

Vancouver 🇨🇦

CreditAttribution: webchick commented 4 January 2013 at 16:56

Category:

feature

» task

There's no reason for this to be blocked on thresholds, so re-assigning as a task.

Comment #7

webchick

she/they

English

Vancouver 🇨🇦

CreditAttribution: webchick commented 4 January 2013 at 17:48

Status:

Reviewed & tested by the community

» Needs work

This patch is no longer applying for me:

$ git apply --index composer-update_3.patch 
error: patch failed: core/vendor/autoload.php:4
error: core/vendor/autoload.php: patch does not apply
error: patch failed: core/vendor/composer/autoload_real.php:2
error: core/vendor/composer/autoload_real.php: patch does not apply
error: patch failed: core/vendor/composer/installed.json:674
error: core/vendor/composer/installed.json: patch does not apply

My, that drop moves fast. :)