Problem/Motivation

But drupal/core-recommended 9.4.0 requires guzzlehttp/psr7 ~1.8.5

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

CommentFileSizeAuthor
#3 3291780-2-9.3.x.patch5.66 KBlongwave
#2 3291780-2-10.0.x.patch5.58 KBlongwave
#2 3291780-2-9.x.x.patch5.7 KBlongwave

Comments

heddn created an issue. See original summary.

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Status: Active » Needs review
StatusFileSize
new 3291780-2-9.x.x.patch5.7 KB
new 3291780-2-10.0.x.patch5.58 KB

HEAD is currently broken in 10.0.x so the expected result there is 70 fails.

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
StatusFileSize
new 3291780-2-9.3.x.patch5.66 KB
longwave’s picture

longwave
he/him
Primary language English
Location UK
commented

10.0.x patch seems like it will also run into #3285230: Migrate's DownloadFunctionalTest:: testExceptionThrow() is failing on guzzlehttp/psr7 2.3.0

The last submitted patch, 2: 3291780-2-10.0.x.patch, failed testing. View results

xjm’s picture

xjm
she/her
Primary language English
commented

Going to commit the 9.x patches from NR here. We can fix the blocker first on D10.

  • xjm committed b7c5fcb on 9.5.x 
    Issue #3291780 by longwave, xjm: guzzlehttp/guzzle 6.5.8 requires...

  • xjm committed 07d2501 on 9.4.x 
    Issue #3291780 by longwave, xjm: guzzlehttp/guzzle 6.5.8 requires...
xjm’s picture

xjm
she/her
Primary language English
commented
Title: guzzlehttp/guzzle 6.5.8 requires guzzlehttp/psr7 ^1.9 » [10.0.x only] guzzlehttp/guzzle 7.4.5 requires guzzlehttp/psr7 ^2.3 which breaks HEAD
Priority: Normal » Critical
Status: Needs review » Postponed
Related issues: +#3285230: Migrate's DownloadFunctionalTest:: testExceptionThrow() is failing on guzzlehttp/psr7 2.3.0

I went ahead and committed this to 9.5.x, 9.4.x, and 9.3.x even though the 10.0.x branch is blocked on the other critical. Rescoping and postponing accordingly.

Thanks @longwave!

  • xjm committed dcdc2f5 on 9.3.x 
    Issue #3291780 by longwave, xjm: guzzlehttp/guzzle 6.5.8 requires...
xjm’s picture

xjm
she/her
Primary language English
commented
Status: Postponed » Needs review

This is unblocked now.

spokje’s picture

spokje
Primary language Dutch
commented
Status: Needs review » Reviewed & tested by the community

- Patch updates:

$ composer-lock-diff --no-links
+--------------------+-------+-------+
| Production Changes | From  | To    |
+--------------------+-------+-------+
| guzzlehttp/guzzle  | 7.4.4 | 7.4.5 |
| guzzlehttp/psr7    | 2.2.1 | 2.4.0 |
+--------------------+-------+-------+

- Green TestBot on both "normal" and updated-dependencies CI,

RTBC

  • xjm committed 14a5bf5 on 10.0.x 
    Issue #3291780 by longwave, xjm, Spokje: guzzlehttp/guzzle 7.4.5...
xjm’s picture

xjm
she/her
Primary language English
commented
Title: [10.0.x only] guzzlehttp/guzzle 7.4.5 requires guzzlehttp/psr7 ^2.3 which breaks HEAD » guzzlehttp/guzzle 7.4.5 requires guzzlehttp/psr7 ^2.3 which breaks HEAD
Status: Reviewed & tested by the community » Fixed
Issue tags: +10.0.0 release notes

Tagging for the alpha6 release notes.

Thanks everyone!

Status: Fixed » Closed (fixed)

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