Problem/Motivation

1356276#686: Allow profiles to define a base/parent profile": "https://www.drupal.org/files/issues/2022-06-17/1356276-686.patch
Above patch failes when we are upgrading thre site to Drupal 10.

Steps to reproduce

* Try to update Drupal core to 10 (from an older version) with Latest pantheon_decoupled_profile

Proposed resolution

Use https://www.drupal.org/files/issues/2022-12-08/3266057-88.patch patch
Issue link - https://www.drupal.org/project/drupal/issues/3266057

Issue fork pantheon_decoupled_profile-3335126

Command icon 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:

About issue forks

Comments

sonvir249 created an issue. See original summary.

sonvir249 opened merge request !2

sonvir249’s picture

sonvir249 commented
Issue summary: View changes
sonvir249’s picture

sonvir249 commented
Status: Active » Needs review
brianperry’s picture

brianperry
he/him
commented

This patch will only apply in Drupal 10, correct? If that is the case, I think we'll have to create a 2.0.x branch with this patch that limits the core version requirement to 10. And on 1.0.x we'd probably want to remove the D10 requirement.

sonvir249’s picture

sonvir249 commented

@brianperry Yes, the patch is only for Drupal 10. Having a separate branch for Drupal 10 is a good idea. This way it will be compatible with both Drupal 9 and Drupal 10

brianperry opened merge request !3

  • brianperry committed bbc29a82 on 1.0.x 
    Issue #3335126: Composer breaks to install profile with Drupal 10...

brianperry’s picture

brianperry
he/him
commented
Status: Needs review » Fixed

Updated 1.0.x to restrict to Drupal 8 and 9.

Created 2.0.x which is Drupal 10 only, and integrated @sonvir249's MR.

Marking as fixed. Thanks @sonvir249!

Status: Fixed » Closed (fixed)

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