As reported by rfay at http://drupal.org/node/718058:

I just saw where the module got built... into sites/all/modulesmy_module_name. Missing the '/' between 'sites/all/modules' and 'my_module_name'.

cd sites/all/modules
drush mb junk menu --write
...
rfay@rfay-bigsony:~/workspace/d6git/sites/all/modules$ ls ..
libraries/  modules/  modulesjunk/  README.txt themes/

This is with current drush HEAD and with the latest from 6.x-2.x (pulled from git.drupal.org).

Confused me quite thoroughly. I had to strace it to find out where the module was going :-)

Thanks for maintaining this great module!

Comments

joachim’s picture

Status: Active » Fixed
StatusFileSize
new871 bytes

Urgh. This was because Drush changed what it returns from pm_dl_destination() on HEAD -- there used to be the trailing slash and now there isn't.

#951084 by joachim: Fixed Drush HEAD omitting the trailing slash after the module directory path.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

Anonymous’s picture

Version: 6.x-2.x-dev » 7.x-2.x-dev
Priority: Normal » Minor
Status: Closed (fixed) » Active

I've started using MB for 7.x (which may not be recommended since it isn't on the project page), but I just wanted to note here that this is still an issue in the dev version of 7.x

joachim’s picture

Oh the 7.x branch of module builder is horribly out of date. You can just use 6.x as a drush plugin and it should do the right thing for 7.x modules.

(Except code templates are wrong, because they're not versioned yet, not had time to write that.)

joachim’s picture

Status: Active » Patch (to be ported)

Changing status.

joachim’s picture

Status: Patch (to be ported) » Fixed

Should all be fixed now -- 6--2 and 7--2 both have the same code for Drush. Reopen if I missed something.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.