Changes to required patch no longer apply

Created on 29 August 2024, 11 months ago
Updated 30 August 2024, 11 months ago

This patch has changed because someone pushed to the PR: https://git.drupalcode.org/project/default_content/-/merge_requests/5.patch

So, it no longer applies to the module when using composer install.

- Applying patches for drupal/default_content
https://git.drupalcode.org/project/default_content/-/merge_requests/5.patch (2698425 - Duplicate content issues in default content)
Could not apply patch! Skipping. The error was: Cannot apply patch https://git.drupalcode.org/project/default_content/-/merge_requests/5.patch

In Patches.php line 331:

Cannot apply patch 2698425 - Duplicate content issues in default content (https://git.drupalcode.org/project/default_content/-/merge_requests/5.patch)!

🐛 Bug report
Status

Needs review

Version

1.4

Component

Code

Created by

🇬🇧United Kingdom rjjakes

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @rjjakes
  • Pipeline finished with Failed
    11 months ago
    Total: 346s
    #268359
  • Merge request !3Updated to use new patch. → (Open) created by rjjakes
  • 🇬🇧United Kingdom rjjakes

    I raised MR3 which tested as applying.

  • Status changed to Needs review 11 months ago
  • 🇬🇧United Kingdom rjjakes
  • Pipeline finished with Failed
    11 months ago
    Total: 191s
    #268409
  • First commit to issue fork.
  • Pipeline finished with Failed
    11 months ago
    Total: 262s
    #268808
  • Pipeline finished with Failed
    11 months ago
    Total: 264s
    #268809
  • Pipeline finished with Failed
    11 months ago
    Total: 225s
    #268810
  • Pipeline finished with Failed
    11 months ago
    #268811
  • Pipeline finished with Failed
    11 months ago
    Total: 225s
    #268812
  • Pipeline finished with Failed
    11 months ago
    Total: 236s
    #268815
  • 🇦🇺Australia silverham

    I note that that the previous Merge request (MR !3) worked on old a version of default_content (2.0.0alpha2). But it still had the issue of using an MR which could be changed by anyone - so I have changed it to use a static patch file instead and added explicit dependency on 2.0.0alpha2.

    In the case that maintainers want to the use the newest version of default_content (2.0.0alpha3), i have created a new Merge request (MR !4). Same deal - using a static patch file instead and added explicit dependency on 2.0.0alpha3 or higher..

  • Pipeline finished with Failed
    11 months ago
    Total: 316s
    #269055
  • Pipeline finished with Failed
    11 months ago
    Total: 339s
    #269056
  • 🇮🇳India vishalkhode

    Thanks everyone for working on this. Fixed in 9ddfc and released as part of 1.4.5 .

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

Production build 0.71.5 2024