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).

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

PatchRanger’s picture

Issue summary: View changes

Added link to Composer module

PatchRanger’s picture

Status: Active » Needs review
FileSize
1021 bytes
PatchRanger’s picture

Issue summary: View changes

Added one more example issue

sun’s picture

Issue summary: View changes
Status: Needs review » Fixed

Fixed a little after commit.

  • Commit 30bdf1c on 7.x-3.x authored by PatchRanger, committed by sun:
    - #1980288 by PatchRanger: Add composer.json.
    

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

dandaman’s picture

The 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.

oadaeh’s picture

@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.

dandaman’s picture

Good 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.

roderik’s picture

@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/.