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.
Adding composer.json will allow to manage the dependency with the power of mighty Composer tool (see Drupal Composer module for details).
Marking as 'task' instead of just 'feature', because Drupal itself is going to switch (sooner or later) to Composer dependency handling (see #1398772: Replace .info.yml with composer.json for extensions and #1886820: Packagist for Drupal Projects, #542940: [meta] Support for downloading libraries via Composer).
Comment | File | Size | Author |
---|---|---|---|
#1 | phpmailer-add_composer.json-1980288-1.patch | 1021 bytes | PatchRanger |
Comments
Comment #0.0
PatchRanger CreditAttribution: PatchRanger commentedAdded link to Composer module
Comment #1
PatchRanger CreditAttribution: PatchRanger commentedComment #1.0
PatchRanger CreditAttribution: PatchRanger commentedAdded one more example issue
Comment #2
sunFixed a little after commit.
Comment #5
dandaman CreditAttribution: dandaman commentedThe presence of this composer.json file confuses Composer Manager. But as far as I can tell, this module doesn't know how to load something from the Vendor directory. So I'd vote that the composer.json is removed or it actually is updated so composer can be used actually.
Comment #6
oadaeh CreditAttribution: oadaeh as a volunteer and at Flatt and Sons commented@dandaman, this is a really old issue that has been fixed and closed, and your comment does not show up in the normal course of browsing through the issue queue. You should create a new issue for your comment.
Comment #7
dandaman CreditAttribution: dandaman commentedGood point, oadaeh. Now I cannot even remember what website I was having the issue on and if it really was the 7.x version or if it was an 8.x version which probably wouldn't apply anyways. If i find it, I might submit a separate ticket.
Comment #8
roderik@dandaman you are correct, it is an issue with the 7.x version. If you have unpatched phpmailer and composer_manager modules installed, and you don't see any warnings in the composer_manager admin screen, then... it's likely that you've installed phpmailer/phpmailer in the vendor directory by now. Which is unused.
I've tried in every way to see if there would be a reason for having this composer.json on D7 but I can't see it. So I filed #3108163: Remove composer.json and will patch my own system to run without the package in vendor/.