Changes to required patch no longer apply

Created on 29 August 2024, 3 months ago
Updated 30 August 2024, 3 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
    3 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 3 months ago
  • 🇬🇧United Kingdom rjjakes
  • Pipeline finished with Failed
    3 months ago
    Total: 191s
    #268409
  • First commit to issue fork.
  • Pipeline finished with Failed
    3 months ago
    Total: 262s
    #268808
  • Pipeline finished with Failed
    3 months ago
    Total: 264s
    #268809
  • Pipeline finished with Failed
    3 months ago
    Total: 225s
    #268810
  • Pipeline finished with Failed
    3 months ago
    #268811
  • Pipeline finished with Failed
    3 months ago
    Total: 225s
    #268812
  • Pipeline finished with Failed
    3 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
    3 months ago
    Total: 316s
    #269055
  • Pipeline finished with Failed
    3 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