Notice: "bam backup" must be renamed to "bam-backup" in order to work with this version of drush (Drush 3.0alpha). If you
are the maintainer for the module that defines this command, please rename it immediately. If you are a
user of this command, you may enable spaces in commands for a while by setting "allow-spaces-in-commands" in
your drush configuration file.

CommentFileSizeAuthor
#2 bam_drush3.patch9.94 KBkswan
#1 720580-drush-1.patch10.1 KBmikey_p
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mikey_p’s picture

mikey_p CreditAttribution: mikey_p commented
Status: Active » Needs review
FileSize
720580-drush-1.patch10.1 KB

This at least covers the DRUPAL-6--2.

Needs tested.

kswan’s picture

kswan CreditAttribution: kswan commented
FileSize
bam_drush3.patch9.94 KB

I tested your patch, but got an error when listing the available command using simply "drush". I found an error in backup_migrate_drush_command(). There was an incorrect line
$items['bam-destinations'] = arraybam-restore;

I fixed this line in the attached patch.

drush bam-destinations isn't returning any destinations, but I don't know if that is related to this issue.

kswan’s picture

kswan CreditAttribution: kswan commented
Title: Drush 3.0alpha won't allow spaces in backup requests » Drush 3.0 doesn't allow spaces in commands

I found that the empty bam-destinations issue is referenced in #715472: "drush bam destinations" is an empty list.

I also changed the title to more accurately describe the problem.

logickal’s picture

logickal CreditAttribution: logickal commented

Tested, seems to work great on my end. I rely pretty heavily on Drush so this fix was good news. I suppose the other workaround would be the setting to allow Drush to continue using spaces in command names....

dooug’s picture

dooug CreditAttribution: dooug commented

Tested and confirmed to work here. Thanks!

ronan’s picture

ronan CreditAttribution: ronan commented
Status: Needs review » Fixed

Awesome, thanks everybody.

Status: Fixed » Closed (fixed)

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

FiNeX’s picture

FiNeX CreditAttribution: FiNeX commented

Thanks for the fix. Does it exists a release plan for the next version which will include this patch?

Thanks.

jim0203’s picture

jim0203 CreditAttribution: jim0203 commented

Thanks from me, too - and I'd also be interested in knowing when this patch will roll into a release of backup and migrate. Great module, btw!

TravisCarden’s picture

TravisCarden CreditAttribution: TravisCarden commented

@jim0203, I share your interest in seeing this released.

cgmonroe’s picture

cgmonroe CreditAttribution: cgmonroe commented

FWIW - This is in the current dev version... now the question is.. to use the dev version in production or not...

ronan’s picture

ronan CreditAttribution: ronan commented

6--2.3 is released now.

Thanks all.