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.hi,
for the fluxkraft distro, we are using composer & composer manager to include external php libraries.
i just wanted to ask if you are planning to support composer for adding further dependencies besides what gets listed in .make files.
here are some reference links:
https://drupal.org/project/fluxkraft_distro
https://drupal.org/project/composer_manager
http://drupalcode.org/project/fluxkraft_distro.git/blob/refs/heads/7.x-1...
greetings,
dasjo
| Comment | File | Size | Author |
|---|---|---|---|
| #11 | composer_support-2069833-11.patch | 1.32 KB | yanniboi |
Comments
Comment #1
patrickd CreditAttribution: patrickd commentedWhile it's currently technically not possible it is definitely planned for the coming rewrite/relaunch of simplytest.me
Until then you can try to include a simplytest.make which first includes the build-*.make and then manually downloads the composer dependencies..
Sorry :/
Comment #2
patrickd CreditAttribution: patrickd commentedDue to (https://groups.drupal.org/node/508662) removal of dependencies from GIT, adding composer support is now a critical problem for testing patches of Drupal Core on simplytest.me.
I'll need to find some way to do it..
Comment #3
xjmThe issue was rolled back but recommitted now that github lifted the rate limits: https://groups.drupal.org/node/509719
So attempting the 8.1.x or 8.2.x branches on simplytest.me now fails.
Comment #4
nevergone CreditAttribution: nevergone commentedDrupal 8.1.0-beta2 is not working. Please enable composer support. Thanks!
Comment #5
patrickd CreditAttribution: patrickd commentedSorry that this took so long!
composer installis now executed for projects that have a composer.jsonDrupal core 8.1.x seems to work fine with the changes.
But I'm afraid this will not work properly with distributions and sub-dependencies yet.
Comment #6
patrickd CreditAttribution: patrickd commentedI'll need to see how I can use Composer manager on simplytest.me to get around these issues
Comment #7
rachel_norfolkpatrickd - I’m at DevDaysMilan and looking do do exactly this with a distribution (and needing dependencies for address) so very happy to test anything you might have so far...
Comment #8
tamasd CreditAttribution: tamasd at Pronovix commentedWhat is the status of the composer manager support? I would like to use a D7 module that has a composer dependency.
Comment #9
patrickd CreditAttribution: patrickd commentedstill not fully supported but composer install should be executed for your module. you could create a separate branch for your project and try it out from simplytest.me, please let me know how it goes.
sorry this is taking so long
Comment #10
m4oliveiHey there, thanks for your work on this. I'm getting an error when trying to setup with fitbit module. The message isn't very helpful, just "An error occurred while installing the environment.". The module includes a composer.json file, so I suspected that was the problem, hence the post here. I tried address module though and that worked out fine. Not sure what's wrong.
Comment #11
yanniboi CreditAttribution: yanniboi at FreelyGive commentedOk, so composer_manager is now "deprecated and no longer needed" according to the project page.
I spoke with @bojanz, and the ideal approach would be for simplytest.me to build drupal projects entirely using composer:
While this is a nice idea to aim for in the upcoming new simplytest.me, I had a quick look at just making composer dependencies work in a non fancy way for now.
My attached patch downloads composer_manager updates composer dependencies if the module being built has a composer.json.
Currently only works for building modules, not distributions. Not looked into distros yet.
Comment #12
patrickd CreditAttribution: patrickd commenteddeployed, thanks yanniboy!!!
please open a new ticket if you still find to have issues regarding composer support, thanks!