localgovdrupal / localgov_events

Events for LocalGov Drupal.
GNU General Public License v2.0
1 stars 1 forks source link

Please remove patch from composer.json #123

Open nccchris opened 1 year ago

nccchris commented 1 year ago

"Date validation #3154944": "https://www.drupal.org/files/issues/2020-06-25/alpha-modal-form-end-date-validation.patch" has been fixed and so can no longer be applied by composer :

"https://www.drupal.org/files/issues/2020-06-25/alpha-modal-form-end-date-validation.patch (Date validation #3154944) Could not apply patch! Skipping. The error was: Cannot apply patch https://www.drupal.org/files/issues/2020-06-25/alpha-modal-form-end-date-validation.patch"

andybroomfield commented 1 year ago

Are we sure the problem is fixed? It looks like its marked as a duplicate, so might need a rerolled patch from the duplicate issue.

@nccchris Does this prevent your deploys, or is it just warnings? Usually it should be possible to skip patches and the composer build continues, though that might be a setting in composer.json.

nccchris commented 1 year ago

It doesn't prevent deploys, it skips it with a red message. It's one of 12 such messages when I create a new install of microsites ( composer create-project --stability beta localgovdrupal/localgov_microsites_project TEST --no-install; cd TEST; composer install) which is a little off putting.

andybroomfield commented 1 year ago

I've opened a PR to try and remove the patch at #124. However I'm not seeing any patch conflict, although the issue is marked as a duiplicate.

Gathering patches for dependencies. This might take a minute.

Thats not to say we shouldn't re-roll it as the issue has been marked as a duplicate and so no further devleopment is being made to that patch and it's likley to conflict with others.

Are you using any other patches @nccchris? Does anyone from @localgovdrupal/core-team who works on Microsites know if other patches are used for date recur?

finnlewis commented 1 year ago

Original issue: https://www.drupal.org/node/3154944

Closed in favour of https://www.drupal.org/project/date_recur_modular/issues/3071666

Maybe we should test https://www.drupal.org/files/issues/2019-08-09/date_validation-3071666-09.patch

finnlewis commented 1 year ago

@nccchris

when I run

composer create-project --stability beta localgovdrupal/localgov_microsites_project TEST --no-install; cd TEST; composer install 

The patch does not fail for me.

I'm on :

PHP: 8.1.7 Composer: 2.5.1 Ubuntu : 23.04

What versions are you running on?

Could you provide the output of your composer install?

nccchris commented 1 year ago

PHP 8.1.24 Composer 2.6.5 Redhat : 8.8

Here's the warning I get about the patches. If you want the whole output just let me know.

finnlewis commented 9 months ago

Hey @nccchris

Just checking in on this issue.

The patch mentioned still applies for me:

Looking at your output, it looks like none of your patches were applying.

Is this still the case?

nccchris commented 9 months ago

Yes, I still get half a dozen errors per composer run on any of my LGD installs.

andybroomfield commented 9 months ago

Might be worth looking at patches-ignore if it's hampering your own installs. and we can review if this patch is still needed.

Is anyone else experincing this issue?