Closed (fixed)
Project:
Drupal core
Version:
9.4.x-dev
Component:
base system
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
7 Nov 2020 at 22:10 UTC
Updated:
8 Feb 2022 at 11:49 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
andypostComment #3
Pooja Ganjage commentedHi,
Creating a patch for this issue.
Please review the patch.
Let me know if any suggestions.
Thanks.
Comment #4
Pooja Ganjage commentedComment #6
alexpott@Pooja Ganjage that's not the fix here. We can't remove the dependency - we still support PHP 7.3.
IN \Drupal\Core\DrupalKernel::boot() we only need to do this when on PHP 7.
Comment #7
andypostCurious about is there a way to skip unregister if already core's override applied
Comment #8
alexpott@andypost can you explain a bit more re #7.
Comment #9
Pooja Ganjage commentedComment #10
Pooja Ganjage commentedComment #12
andypost@alexpott sorry I was wrong. I thought about
initializeSettings()method which have own issue #2708827: Allows DrupalKernel::handle to handle more than one requests (rough support for PHP-PM)Here's a patch to skip whole block and todo to new issue #3210486: Remove typo3/phar-stream-wrapper and associated code
Comment #13
andypostComment #14
andypostFix broken test
Comment #15
andypostIt does not throw exception on PHP < 8
Comment #17
andypostUnrelated failure in
QuickStartTest::testQuickStartCommandComment #18
alexpott@andypost I think in #15 you meant to say... It does not throw exception on PHP >= 8 ... it certainly does throw an exception on PHP < 8.
What does happen with the regular phar handler in PHP 8 with files that are not phar files?
Comment #19
andypostYes, I mean
> 8does not throw as https://bugs.php.net/bug.php?id=76774The commit with details as RFC link https://github.com/php/php-src/commit/0c238ede019f6ffbe7c996ec1695a747f4...
Comment #20
andypostUpdated IS with links
According to tests on PHP 8 the used library does not throw https://github.com/ohader/phar-stream-wrapper/commit/3a25049ddd9f49e732d...
Comment #21
andypostExtended test for PHP 8
Comment #24
longwaveMakes sense to me to do this now more people are moving towards PHP 8.
Comment #25
alexpottCommitted and pushed 1a7b7f7219b to 10.0.x and 1311b624e7e to 9.4.x. Thanks!
Can we get a follow-up to remove the dependency and code on Drupal 10? Thanks!
Comment #28
longwaveFollowup is #3210486: Remove typo3/phar-stream-wrapper and associated code