- Issue created by @fraserthompson
- ๐ธ๐ฆSaudi Arabia martins.bruvelis Thuwal
I can confirm that adding the patch '
guzzle-description-loader-fix.patch
' usign composer.json, to arthurkirkosa/guzzle-description-loader module, resolves the issue:, "arthurkirkosa/guzzle-description-loader": { "3406108 - guzzle-description-loader dependency is broken": "https://www.drupal.org/files/issues/2023-12-05/guzzle-description-loader-fix.patch" }
- ๐บ๐ธUnited States datawench
Patch also works for me. Thanks for figuring this out so quickly.
Given that the library is so old, perhaps a module maintainer here might fork it, update it, and use it instead? - Status changed to Needs review
about 1 year ago 8:57am 22 December 2023 - ๐ฎ๐นItaly aronne
Hi guys,
I've just made a fork of that library and here you can find the patch that should fix the problem.Regards,
aronne -
aronne โ
authored b13dd86a on 3.x
Issue #3406108 by fraserthompson, aronne: guzzle-description-loader...
-
aronne โ
authored b13dd86a on 3.x
- Status changed to Needs work
about 1 year ago 9:17am 22 December 2023 - ๐ฎ๐นItaly aronne
Hmm no I have to figure out why composer canโt find the release
Iโll work on it later
- Status changed to Needs review
about 1 year ago 12:51pm 22 December 2023 - ๐ฎ๐นItaly aronne
Ok this patch should be the right one.
I've already submitted it on the dev version so please let me know if it works for you as well by doing
composer require 'drupal/http_client_manager:3.x-dev@dev'
Regards,
aronne -
aronne โ
authored c39f4279 on 3.x
Issue #3406108 by aronne: guzzle-description-loader dependency is broken
-
aronne โ
authored c39f4279 on 3.x
- ๐บ๐ธUnited States datawench
The #7 patch works great, thank you!
Sticking with the patch for now, until it gets merged into a release version.
- ๐ฎ๐นItaly trickfun
Instruction #2 works fine.
Patch #7 doesn't work. file composer.json doesn't change in module folder. - ๐ฎ๐นItaly aronne
@trickfun composer.json will never change if you run patch #7 in the drupal root instead of the module root.
- Status changed to RTBC
12 months ago 4:16pm 10 January 2024 - Status changed to Fixed
12 months ago 4:17pm 10 January 2024 - ๐บ๐ธUnited States datawench
@aronne I'm not sure I understand. Are you saying that the patch shouldn't apply if you add the appropriate element in the cweagans-style "patches" portion of the root composer.json?
"drupal/http_client_manager": { "guzzle-description-loader dependency is broken (see https://www.drupal.org/project/http_client_manager/issues/3406108)": "https://www.drupal.org/files/issues/2023-12-22/3406108-7.patch" }
- ๐ฎ๐นItaly aronne
Patch #7 doesn't work. file composer.json doesn't change in module folder.
No I'm just saying that if you put the patch in the drupal document root, as I think he did, and then you run
git apply 3406108-7.patch
it won't work.@aronne I'm not sure I understand. Are you saying that the patch shouldn't apply if you add the appropriate element in the cweagans-style "patches" portion of the root composer.json?
@atawench Adding the patch in the root composer.json is how it should be done properly.
Automatically closed - issue fixed for 2 weeks with no activity.