Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Problem/Motivation
aggregator_requirements() will block installation if cURL is not available. However, Guzzle does not have a hard dependency on cURL and can work without it.
Steps to reproduce
Proposed resolution
Remove the aggregator_requirements() check.
Remaining tasks
User interface changes
API changes
Data model changes
Release notes snippet
Comment | File | Size | Author |
---|---|---|---|
#2 | 3178175.patch | 1.09 KB | longwave |
Issue fork drupal-3178175
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
Comments
Comment #2
longwaveComment #3
Kristen PolThanks for the issue and patch!
1) I have confirmed that Guzzle does not need cURL. See https://docs.guzzlephp.org/en/stable/overview.html
2) The patch still applies cleanly to 9.2.
3) The code changes look good and addresses the issue in the issue summary.
4) Tests pass.
5) Manually tested by using with patch in place and installing, uninstalling, and reinstalling aggregator. There were no errors.
Looks good to me! Marking RTBC.
Comment #5
catchGood find.
Committed 8664915 and pushed to 9.2.x. Thanks!