Closed (fixed)
Project:
Drush Make
Version:
6.x-2.x-dev
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
14 Dec 2009 at 09:54 UTC
Updated:
3 Jan 2014 at 01:08 UTC
Jump to comment: Most recent, Most recent file
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