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.