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
We have added composer support for modules in other issues, now it is time to add it to distributions.
Proposed resolution
(For now) add composer support using composer manager.
Remaining tasks
Patch- Review
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | add_composer_support-2842430-2.patch | 1.64 KB | yanniboi |
Comments
Comment #2
yanniboi CreditAttribution: yanniboi at FreelyGive commentedComment #3
yanniboi CreditAttribution: yanniboi at FreelyGive commentedThe current patch unfortunately duplicates the modules built using drush make. So if a distribution provides a make file and a composer.json then it will build both.
In the case of opensocial (https://www.drupal.org/project/social) it provides both a make file and a composer.json in which case it downloads a load of module dependencies to webroot/profiles/social/modules/contrib as well as webroot/modules/. I think this isn't the worst because the composer downloaded modules take preference over those provided by the profile, so the install works and so should demoing the project on simplytest.me, but ideally if composer is used then the make file should be ignored.
Comment #4
patrickd CreditAttribution: patrickd commentedYeah, it's kind of a dirty hack but it's basically the same way it works right now when using project type module instead of distribution ¯\_(ツ)_/¯
works, deployed!
Thanks!