Problem/Motivation

Part of #3109885: [meta] Ensure compatibility of Drupal 9 with PHP 8.0 (as it evolves)

typo3/phar-stream-wrapper

is core's (and drupal/core-recommended) dependency package used to release PHP 8 compatibility in bug-fix release
I
https://github.com/TYPO3/phar-stream-wrapper/issues/64
https://github.com/TYPO3/phar-stream-wrapper/pull/65

Proposed resolution

Upgrade version from 3.1.5 to 3.1.6
This upgrade leaves only laminas dependencies blocking to declare core PHP 8.0 compatible

Remaining tasks

- review/commit - https://github.com/TYPO3/phar-stream-wrapper/releases/tag/v3.1.6
- backport to 9.1.x branch as this is bugfix release in terms of semver

User interface changes

no

API changes

no

Data model changes

no

Release notes snippet

typo3/phar-stream-wrapper updated to v3.1.6 for PHP 8 compatibility.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

andypost created an issue. See original summary.

andypost’s picture

Release notes https://github.com/TYPO3/phar-stream-wrapper/releases/tag/v3.1.6 states about possible incompatibility

andypost’s picture

Issue summary: View changes

Fixed IS as the package is not -dev dependency

andypost’s picture

checked upgrade for 9.1.x and it is exactly the same as for 9.2.x

COMPOSER_ROOT_VERSION=9.1.x-dev composer2 update typo3/phar-stream-wrapper

andypost’s picture

From https://github.com/TYPO3/phar-stream-wrapper/issues/64#issuecomment-7191...

The other option would be to release a new release which has the same API but does nothing other than passthrough to the PHP streamwrapper. But I think it's better for projects like Drupal to only register the wrapper when running on PHP 7.

Talked to @alexpott in slack and he said

We can leave that alone. It doesn’t matter that we also register it in PHP 8 - it shouldn;t be necessary because of fixes in PHP but we can handle that in a follow-up once PHP 8 testing is enabled

alexpott’s picture

longwave’s picture

Status: Needs review » Reviewed & tested by the community

Looks good to me!

alexpott’s picture

Version: 9.2.x-dev » 9.1.x-dev
Status: Reviewed & tested by the community » Fixed
Issue tags: -9.2.0 release notes +9.1.0 release notes

Committed and pushed 5e0b2d7a05 to 9.2.x and 2068480ad0 to 9.1.x. Thanks!

  • alexpott committed 5e0b2d7 on 9.2.x
    Issue #3181240 by andypost: Upgrade typo3/phar-stream-wrapper 3.1.6
    

  • alexpott committed 2068480 on 9.1.x
    Issue #3181240 by andypost: Upgrade typo3/phar-stream-wrapper 3.1.6
    
    (...

Status: Fixed » Closed (fixed)

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