Closed (won't fix)
Project:
Drush
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
30 Jul 2009 at 05:41 UTC
Updated:
3 Mar 2010 at 00:18 UTC
package_handler_update_project() calls drush_op('chdir', $project), which changes the working directory to an arbitrary path. This causes subsequent calls to require() to fail because the items in PHPs include path (usually ".") are not Drupal's root directory anymore.
This patch fixes the issue by calling drush_op('chdir', drush_get_context('DRUSH_DRUPAL_ROOT')) (instead of just cd ..) after the cvs update command. Another solution could entail something like
set_include_path(get_include_path() . PATH_SEPARATOR . drush_get_context('DRUSH_DRUPAL_ROOT'));
(as per the PHP manual) when drush (or Drupal core) bootstraps or initializes. I have not tested that approach since that may have negative impacts elsewhere (but may also fix similar bugs in other parts of drush).
| Comment | File | Size | Author |
|---|---|---|---|
| drush.cvs_.up_.patch | 830 bytes | Bevan |
Comments
Comment #1
moshe weitzman commentedstill a problem with HEAD?
Comment #2
Bevan commentedI should have included more detail about the how to reproduce this. I am no longer able to test.