Running composer update does:

Loading composer repositories with package information
Updating dependencies (including require-dev)
  - Removing mikey179/vfsstream (v1.5.0)
  - Installing mikey179/vfsstream (v1.6.0)
    Downloading: 100%

  - Removing phpunit/php-code-coverage (2.2.3)
  - Installing phpunit/php-code-coverage (2.2.4)
    Downloading: 100%

  - Removing phpunit/phpunit (4.8.10)
  - Installing phpunit/phpunit (4.8.11)
    Downloading: 100%

  - Installing zendframework/zend-hydrator (1.0.0)
    Downloading: 100%

  - Removing zendframework/zend-stdlib (2.6.0)
  - Installing zendframework/zend-stdlib (2.7.3)
    Downloading: 100%

Why is this critical? Drupal\aggregator\Plugin\aggregator\parser\DefaultParser uses Zend\Feed\Reader\Reader<code> which uses <code>Zend\Stdlib\ErrorHandle - therefore Zend\Stdlib is used in run-time code.

zendframework/zend-hydrator is added because it is a new dependency of zendframework/zend-stdlib - in 2.6.0 it was part of zendframework/zend-stdlib.

CommentFileSizeAuthor
#2 2581683-2.patch564.18 KBalexpott
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

alexpott created an issue. See original summary.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott at Chapter Three commented
Status: Active » Needs review
FileSize
2581683-2.patch564.18 KB
alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott at Chapter Three commented
Parent issue: » #2400407: [meta] Ensure vendor (PHP) libraries are on latest stable release
alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott at Chapter Three commented
Issue summary: View changes
alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott at Chapter Three commented

hussainweb should be credited since he worked on #2400407: [meta] Ensure vendor (PHP) libraries are on latest stable release

pfrenssen’s picture

pfrenssen
Location Sofia
CreditAttribution: pfrenssen commented

Patch looks good, only touches composer.lock + the affected files in the vendor folder. Hopefully tests come back green.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott at Chapter Three commented
Issue summary: View changes
hussainweb’s picture

hussainweb
Primary language English
CreditAttribution: hussainweb at Axelerant commented
Status: Needs review » Reviewed & tested by the community

@alexpott: Thanks for remembering. :)

I applied the patch, and ran composer update:

hw@d8:/var/www/d8task-[git 8.0.x] $ composer update
Loading composer repositories with package information
Updating dependencies (including require-dev)
Nothing to install or update
Generating autoload files
> Drupal\Core\Composer\Composer::preAutoloadDump
> Drupal\Core\Composer\Composer::ensureHtaccess

No updates, and there were no changes to any of the files. RTBC once the tests are green.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott at Chapter Three commented

Adding @hussainweb to the people to credit.

catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch at Tag1 Consulting for Drupal Association commented

Committed/pushed to 8.0.x, thanks!

  • catch committed 18a5c79 on 8.0.x 
    Issue #2581683 by alexpott, hussainweb: Update PHP dependencies pre RC...
alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott at Chapter Three commented
Status: Reviewed & tested by the community » Fixed
ianthomas_uk’s picture

ianthomas_uk
he/him
Primary language English
Location Brighton, UK
CreditAttribution: ianthomas_uk as a volunteer commented

Yey! Here comes 8.0.0-RC1

Edit: Looks like Alex beat me to actually marking it fixed.

Status: Fixed » Needs work

The last submitted patch, 2: 2581683-2.patch, failed testing.

catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch at Tag1 Consulting for Drupal Association commented
Status: Needs work » Fixed

Yes PIFR, we know you're still there.

Status: Fixed » Closed (fixed)

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

Cameron Tod’s picture

Cameron Tod CreditAttribution: Cameron Tod commented
Issue summary: View changes