It would make it easier to write Drush commands that work with both Drush-4 and Drush-5 if Drush-4 handled backend set results the same way that Drush-5 does. The attached patch adds this feature to Drush-4.

CommentFileSizeAuthor
#4 drush-4-integrate.patch2.99 KBgreg.1.anderson
backend-set-results.patch0 bytesgreg.1.anderson

Comments

greg.1.anderson’s picture

greg.1.anderson commented

See also #1434376: Errors not propagated, make silently "succeeds" but doesn't do anything.

dww’s picture

dww
we/he/they
commented
Status: Needs review » Needs work

http://drupal.org/files/backend-set-results.patch is empty. Makes it hard to review. ;)

dww’s picture

dww
we/he/they
commented
Title: Drush-4 should handle backend set results the same way that » Drush-4 should handle backend set results the same way that Drush-5 does
greg.1.anderson’s picture

greg.1.anderson commented
StatusFileSize
new drush-4-integrate.patch2.99 KB

Hm, yes. Here is a slightly longer version.

greg.1.anderson’s picture

greg.1.anderson commented
Status: Needs work » Needs review
greg.1.anderson’s picture

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

The patch in #4 actually belonged to #1313810: #integrate consistency betwen drush-4 and drush-5, which was already committed. Time to dig up the backend set results patch...

greg.1.anderson’s picture

greg.1.anderson commented

Committed 45c9479. Code straight from Drush-5; all tests pass.

greg.1.anderson’s picture

greg.1.anderson commented
Status: Active » Fixed

Status: Fixed » Closed (fixed)

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