When attempting to generate a makefile for a drupal 6 site i'm given the error:
Fatal error: Call to undefined function drush_get_modules() in /var/www/drush/commands/drush_make/drush_make.generate.inc on line 35
I installed 'drush_make' directory in the /drush/commands/ directory.
| Comment | File | Size | Author |
|---|---|---|---|
| #10 | 659210.patch | 2.02 KB | jonhattan |
| #9 | 659210.patch | 1.98 KB | jonhattan |
| #6 | drush_make.drush_get_modules.patch | 694 bytes | becw |
Comments
Comment #1
buddaCaused by an out of date drush install on the source server.
Comment #2
anarcat commentedI'm seeing this here with drush 3.0-beta1, which does feature drush_get_modules()... Seems like we have a bootstrap problem of some sort here.
Comment #3
jwilson3I also just pulled straight from cvs drush-HEAD and installed drush_make 6.x-2.0-dev from cvs and get the same issue.
^ the acquia error bothers me too... at best it should be a warning.. perhaps that should be another issue?
Comment #4
redben commentedsubscribing
Comment #5
ldweeks commentedsubscribing (is there a better way to subscribe to stuff?)
Comment #6
becw commentedI'm using Drush 3.0 RC1 and Drush Make 2.0 Beta 6 and having this problem; the problem is still present when I grab the latest from CVS. I can fix it by adding
drush_include_engine('drupal', 'environment');before thedrush_get_modules()call. I don't know Drush well enough to know if this is the correct solution, but it works. Patch attached.Comment #7
jonhattan@bec this is a solution.
A better integration with drush is to just use drush_get_projects() for modules and themes. No need to include the environment engine.
Comment #8
becw commented@jonhattan drush_get_projects() just includes the environment engine, and then calls drush_get_modules() and drush_get_themes(). Since it returns a different set of results than only calling drush_get_modules(), the surrounding code would need to be refactored. It might be the right way to do things, though :)
Comment #9
jonhattan@bec: I know.. I wrote it :)
Here is a patch using drush_get_projects() and a rework of the subsequent loop.
Comment #10
jonhattanSorry. this is the correct patch:
Comment #11
becw commentedCool; works for me & looks good. Would it be jumping the gun to mark this RTBC?
Comment #12
jonhattanCan't we push it up beyond critical and RTBC? :)
Comment #13
hunmonk commenteddrush 3.0 will not be supported on drush make 6.x-2.x-dev. please run drush 2.x series. once we have the first stable release on the 6.x-2.x series, we'll branch and get compatibility with 3.x working.
Comment #14
dmitrig01 commentedfixed. thanks