When we added Guzzle, we had to use a git clone based version that included the very latest commits as there were required composer.json changes in Guzzle.

Since then, 3.0.6 and 7 have been released, containing these changes, so it should be save to go back to a version definition of "3.0.*@stable" or similar.

Files: 
CommentFileSizeAuthor
#8 guzzle-easyrdf-update-1875818-8.patch99.72 KBBerdir
PASSED: [[SimpleTest]]: [MySQL] 50,711 pass(es).
[ View ]
#3 guzzle-update-1875818-3.patch74.13 KBBerdir
PASSED: [[SimpleTest]]: [MySQL] 49,989 pass(es).
[ View ]
#1 guzzle-update-1875818-1.patch26.74 KBBerdir
FAILED: [[SimpleTest]]: [MySQL] 49,511 pass(es), 37 fail(s), and 0 exception(s).
[ View ]
#1 guzzle-update-1875818-1.patch26.74 KBBerdir
FAILED: [[SimpleTest]]: [MySQL] 49,619 pass(es), 36 fail(s), and 0 exception(s).
[ View ]

Comments

Berdir’s picture

Status:Active» Needs review
StatusFileSize
new26.74 KB
FAILED: [[SimpleTest]]: [MySQL] 49,619 pass(es), 36 fail(s), and 0 exception(s).
[ View ]
new26.74 KB
FAILED: [[SimpleTest]]: [MySQL] 49,511 pass(es), 37 fail(s), and 0 exception(s).
[ View ]

Another reason for this.

It was impossible to call composer update guzzle/http because composer died with a Runtime exception about a missing composer remote. I guess it assumed the existence of a .git repository within the guzzle directories, so I had to completely remove vendor/guzzle first.

Should otherwise be a straight forward thing to do...

Status:Needs review» Needs work

The last submitted patch, guzzle-update-1875818-1.patch, failed testing.

Berdir’s picture

Status:Needs work» Needs review
StatusFileSize
new74.13 KB
PASSED: [[SimpleTest]]: [MySQL] 49,989 pass(es).
[ View ]

Hah, or so you'd think.

Looks like the update introduced a dependency on the backoff plugin, which is however not defined in composer.

For demonstration purposes, I added it manually and confirmed that the tests then work again. But that adds a plugin that we don't need and also the Log component.

I think this shouldn't be hard dependency and CurlMulti.php should check the existence of that plugin...

Will open an upstream bug report.

Berdir’s picture

Berdir’s picture

The bug has been fixed but now we need to wait for 3.0.8 :)

YesCT’s picture

Berdir’s picture

Status:Needs review» Needs work

Hm, weird. Looks like that easyrdf stuff was renamed and we reference a version that does not exist?

Was not able to update just guzzle and ignore easyrdf and easyrdf seems to contain huge changes, a ton of test files, ... ?

Berdir’s picture

Status:Needs work» Needs review
StatusFileSize
new99.72 KB
PASSED: [[SimpleTest]]: [MySQL] 50,711 pass(es).
[ View ]

Ok, here is an update with easyrdf set to dev-master. Looks like most of the files are identical. Not sure if that is correct, we can always update that again.

I left the LICENSE and similar files in there, I think they might be relevant for licensing reason.

scor’s picture

thanks @Berdir, this update to easyrdf is fine, I think I used the wrong namespace in composer.json before. We will most likely update easyrdf in the future to keep up with its changes, so we can fine tune as necessary.

YesCT’s picture

Status:Needs review» Reviewed & tested by the community

If there is no manual test to do, I think this is RTBC.
@scor thought this looked ok.
The code changes look reasonable (they come from the new/updated files).
And the testbot is green.

webchick’s picture

Status:Reviewed & tested by the community» Fixed

Committed and pushed to 8.x. Thanks!

Status:Fixed» Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.