Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
Problem/Motivation
In #3187857: [Symfony 6] Service deprecation messages require more fields we added code to support two different method signatures for Symfony\Component\DependencyInjection\Definition
- this allows Drupal 9 to run on Symfony 5 (after a manual compuser update) without changes.
In Drupal 10 we'll require Symfony 5 or higher, so can drop the compatibility layer for Symfony 4.
Steps to reproduce
Proposed resolution
Remaining tasks
User interface changes
API changes
Data model changes
Release notes snippet
Issue fork drupal-3197729
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
- 3197729-remove-forwardbc-layer changes, plain diff MR !1661
Comments
Comment #2
andypostComment #3
andypostComment #4
SpokjeUnpostponing since there's now a Drupal
10.0.x-dev
branchComment #5
SpokjeUnpostponing since there's now a Drupal
10.0.x-dev
branchComment #6
SpokjeNo, re-postponing, since the current
10.0.x-dev
is onComment #7
daffie CreditAttribution: daffie commented#3197482: Update Drupal 10 to depend on Symfony 5.4 (as a stepping stone to Symfony 6, for deprecation checking support) has landed.
Comment #9
SpokjeComment #10
longwaveLooks OK to me.
Could do with a followup to remove the
cache.null
service as per https://www.drupal.org/node/3066407 if an issue for this doesn't exist already.Comment #11
SpokjeThanks @longwave, couldn't find an issue for the removal of the
cache.null
service, so created a follow-up: #3258918: Remove deprecated 'cache.null' serviceComment #12
alexpottCommitted 6923e5e and pushed to 10.0.x. Thanks!