- First commit to issue fork.
- 🇮🇳India sheetal-wish Indore
I have applied this patch and now it's D10 compatible but I am not able to download AMP library via composer it's showing me this error:
lullabot/amp[2.0.1, ..., 2.0.3] require guzzlehttp/guzzle ~6.1 -> satisfiable by guzzlehttp/guzzle[6.1.0, ..., 6.5.x-dev].
- lullabot/amp 2.0.0 requires sebastian/diff ^1.2 || ^2 || ^3 -> found sebastian/diff[1.2.0, ..., 1.4.x-dev, 2.0.1, 2.0.x-dev, 3.0.0, ..., 3.0.x-dev] but these were not loaded, likely because it conflicts with another require.
- guzzlehttp/guzzle[6.4.0, ..., 6.5.7] require guzzlehttp/psr7 ^1.6.1 -> found guzzlehttp/psr7[1.6.1, ..., 1.x-dev] but these were not loaded, likely because it conflicts with another require.
- guzzlehttp/guzzle 6.5.4 requires symfony/polyfill-intl-idn 1.17.0 -> found symfony/polyfill-intl-idn[v1.17.0] but these were not loaded, likely because it conflicts with another require.
- guzzlehttp/guzzle[6.5.8, ..., 6.5.x-dev] require guzzlehttp/psr7 ^1.9 -> found guzzlehttp/psr7[1.9.0, 1.x-dev] but these were not loaded, likely because it conflicts with another require.
- Root composer.json requires lullabot/amp ^2.0 -> satisfiable by lullabot/amp[2.0.0, 2.0.1, 2.0.2, 2.0.3]Can someone please help me to resolve this.
Thank you - First commit to issue fork.
- e0ipso Can Picafort
@sheetal-wish it seems you have some other dependency incompatibility that entangles with Guzzle.
- 🇷🇺Russia pick_d
Probably dumb question, but
How does one upgrade to Drupal 10 when there is no suitable amp release? I mean, I patched the module, and upgrade_status shows that everything is fine and I should I be able to upgrade, however when I try to upgrade, I get this:
- drupal/amp 3.6.0 requires drupal/core ^8.7.7 || ^9 -> satisfiable by drupal/core[8.7.7, ..., 8.9.x-dev, 9.0.0-alpha1, ..., 9.5.x-dev].
- Open on Drupal.org →Core: 9.5.x + Environment: PHP 7.3 & MySQL 5.7last update
over 1 year ago Waiting for branch to pass - 🇧🇷Brazil adinancenci
A small modification to patch #10 to make AmpCssCollectionRenderer compatible with CssCollectionRenderer
- last update
over 1 year ago Patch Failed to Apply - 🇭🇺Hungary szato
@adinancenci
You have wrong references in #18 patch file, e.g.:
diff --git a/web/modules/contrib/amp/amp.info.yml b/web/modules/contrib/amp/amp.info.yml
should be:
diff --git a/amp.info.yml b/amp.info.yml
- last update
over 1 year ago Composer require failure - First commit to issue fork.
- Open on Drupal.org →Core: 9.5.x + Environment: PHP 7.3 & MySQL 5.7last update
over 1 year ago Waiting for branch to pass - @stborchert opened merge request.
- last update
over 1 year ago 34 pass, 12 fail The last submitted patch, 22: amp-3286062-22.patch, failed testing. View results →
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.- e0ipso Can Picafort
This looks good to me!
My main hesitation was
+ "lullabot/amp": "^1.1 || ^2.0 || dev-main",
, so I tagged a release in the library to avoid that. I will fix this on commit. -
e0ipso →
committed 178f1b16 on 8.x-3.x authored by
dakruchko →
Issue #3286062 by Project Update Bot, reenaraghavan, dakruchko, abhinesh...
-
e0ipso →
committed 178f1b16 on 8.x-3.x authored by
dakruchko →
- Status changed to Fixed
over 1 year ago 3:17pm 12 July 2023 Automatically closed - issue fixed for 2 weeks with no activity.