Closed (fixed)
Project:
Drush
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
11 Nov 2009 at 21:33 UTC
Updated:
20 Dec 2009 at 14:40 UTC
Just wanted to make a note that this needs to be fixed. It's a pretty simple fix on line 544 of pm.drush.inc . Currently if you use drush dl to download an installation profile it dl's it to an incorrect directory. Example my drupal install is located at /d6 when I `drush dl example_profile` it dl's to /d6profiles/ instead of /d6/profiles/. I imagine this bug hasn't popped up because, well lets face it no one uses install profiles because they aren't implemented very well (hopefully the packaging script stuff gets set up for d7). Any way the fix is simply to add the '/' to line 544 in pm.drush.inc I didn't create a patch cause that seams like overkill to me for simply adding the /.
<?php
function pm_dl_destination_lookup($type, $drupal_root, $sitepath, $create = FALSE) {
switch ($type) {
case 'module':
$destination = $sitepath . 'modules/';
break;
case 'theme':
$destination = $sitepath . 'themes/';
break;
case 'theme engine':
$destination = $sitepath . 'themes/engines/';
break;
case 'translation':
$destination = $drupal_root . '/';
break;
case 'profile':
$destination = $drupal_root . '/profiles/'; // <<<<------ADDED THE / Before 'profiles/' it fixed the issue
break;
}
?>
Comments
Comment #1
moshe weitzman commentedAbout to commit some integration with new profile packaging.