When the command-line version of PHP isn't sufficient, we get a helpful error message. Unfortunately, it uses the wrong exit code, making it difficult to react to this failure mode:

# drush
Your command line PHP installation is too old. Drush requires at least PHP 5.3.5
# echo $?
0
Files: 
CommentFileSizeAuthor
#2 drush-1821888.patch1.25 KBjonhattan
#1 drush-1821888.patch686 bytesjonhattan

Comments

Status:Active» Needs review
StatusFileSize
new686 bytes

Changed die() to drush_log() and exit(1). Perhaps drush_set_error() is preferable for remote calls ?

StatusFileSize
new1.25 KB

A better approach.

Status:Needs review» Reviewed & tested by the community

Could you change the error message so that Drush is capitalized. It should only be lowercase when referring to the executable shell script and not the Drush project in general.

Status:Reviewed & tested by the community» Fixed

done

Status:Fixed» Closed (fixed)

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