I made the following somewhat dubious observation on #1152908: Remove calls to drush_backend_invoke() (Call to undefined function drush_backend_invoke() in features.drush.inc line 214):

#24 does work with both drush-5.x and drush 4.5; however, on drush 4.5 the output from the backend calls is not shown. Drush 4.x works better if #integrate is set; this is not necessary on 5.x, though.

The problem here is that integrate defaults to TRUE in backend_invoke in drush-4.x, and it also defaults to true in drush-5, but somehow it is defaulting to FALSE in drush-4.5. We should probably fix this for consistency in drush-4.6.

CommentFileSizeAuthor
#2 drush-4-integrate.patch2.99 KBgreg.1.anderson

Comments

greg.1.anderson’s picture

greg.1.anderson commented
Issue tags: +Release blocker

This is a change for drush-4, but I'm tagging it as a "Release blocker" for drush-5 so that we get this discrepancy worked out before shipping drush-5.

greg.1.anderson’s picture

greg.1.anderson commented
Status: Active » Needs review
StatusFileSize
new drush-4-integrate.patch2.99 KB

Here's a patch that makes drush-4's integrate default to TRUE, like it does in drush-5.

greg.1.anderson’s picture

greg.1.anderson commented
Assigned: greg.1.anderson » msonnabaum
eugenmayer’s picture

eugenmayer commented

just ran into that one. Thanks for the contribution greg, very helpful

eugenmayer’s picture

eugenmayer commented

Just confirmed, applied the patch on todays 4.6-dev ( which is not yet fixed ) and finally the #integrate options defaults properly, now seeing all those missing drush_invoke_process outputs. Thank you!

greg.1.anderson’s picture

greg.1.anderson commented
Status: Needs review » Fixed

Committed per moshe in IRC. 7aab176.

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