We don't use zendframework/zend-stdlib directly, but through zendframework/zend-feed. It is still a good idea to upgrade it though.

Beta phase evaluation

Reference: https://www.drupal.org/core/beta-changes
Issue category Task because it is an external lib upgrade
Issue priority Not critical because we don't use this directly
CommentFileSizeAuthor
#2 upgrade-2578819-2.patch216.46 KBhussainweb
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

hussainweb created an issue. See original summary.

hussainweb’s picture

hussainweb
Primary language English
CreditAttribution: hussainweb at Axelerant commented
Status: Active » Needs review
FileSize
upgrade-2578819-2.patch216.46 KB 
hw@d8:/var/www/d8task-[git 8.0.x] $ composer update zendframework/*
Loading composer repositories with package information
Updating dependencies (including require-dev)
  - Installing zendframework/zend-hydrator (1.0.0)
    Loading from cache

  - Removing zendframework/zend-stdlib (2.6.0)
  - Installing zendframework/zend-stdlib (2.7.3)
    Loading from cache

zendframework/zend-hydrator suggests installing zendframework/zend-eventmanager
(^2.5.1, to support aggregate hydrator usage)
zendframework/zend-hydrator suggests installing zendframework/zend-serializer (^2.5.1, to use the SerializableStrategy)
zendframework/zend-hydrator suggests installing zendframework/zend-servicemanager (^2.5.1, to support hydrator plugin manager usage)
zendframework/zend-hydrator suggests installing zendframework/zend-filter (^2.5.1, to support naming strategy hydrator usage)
Writing lock file
Generating autoload files
> Drupal\Core\Composer\Composer::preAutoloadDump
> Drupal\Core\Composer\Composer::ensureHtaccess

Status: Needs review » Needs work

The last submitted patch, 2: upgrade-2578819-2.patch, failed testing.

hussainweb’s picture

hussainweb
Primary language English
CreditAttribution: hussainweb at Axelerant commented

I am not sure if I trust the fails on PIFR. I don't think Views UI has anything to do with zend-stdlib. And CI is passing. Let's retest.

Status: Needs work » Needs review

hussainweb queued 2: upgrade-2578819-2.patch for re-testing.

Status: Needs review » Needs work

The last submitted patch, 2: upgrade-2578819-2.patch, failed testing.

hussainweb’s picture

hussainweb
Primary language English
CreditAttribution: hussainweb at Axelerant commented
Status: Needs work » Needs review

It is a completely different failure this time. I am just going to leave it for Needs review now. The CI testbot passes anyway.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott at Chapter Three commented
Status: Needs review » Closed (duplicate)
Related issues: +#2581683: Update PHP dependencies pre RC using composer

We're updating all the deps in one go on #2581683: Update PHP dependencies pre RC using composer - @hussainweb please comment on that issue so you can credit for the work here.